Return-path: Received: from s3.sipsolutions.net ([5.9.151.49]:42991 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752264AbcFWLfp (ORCPT ); Thu, 23 Jun 2016 07:35:45 -0400 Message-ID: <1466681739.11061.3.camel@sipsolutions.net> (sfid-20160623_133620_451943_39C29DC6) Subject: Re: [PATCH] wlcore: time sync : add support for 64 bit clock From: Johannes Berg To: "Machani, Yaniv" , "linux-kernel@vger.kernel.org" Cc: Kalle Valo , "Mishol, Guy" , Eliad Peller , "linux-wireless@vger.kernel.org" , "netdev@vger.kernel.org" Date: Thu, 23 Jun 2016 13:35:39 +0200 In-Reply-To: References: <20160623111332.23326-1-yanivma@ti.com> (sfid-20160623_131215_419855_E4550AFE) <1466680680.11061.2.camel@sipsolutions.net> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: > > Additionally, this looks like it changes the firmware API, so that  > > older firmware images will no longer work? > > It is backwards compatible,  > although it changes a API structure, older firmware are using only > u16 for the field so there is no impact on that. > Oh, ok. I had also thought that the size changed, but missed that you replaced a u32 with two u16. Thanks for checking :) johannes