Return-path: Received: from mail-ob0-f173.google.com ([209.85.214.173]:35352 "EHLO mail-ob0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751092AbaAQHLf (ORCPT ); Fri, 17 Jan 2014 02:11:35 -0500 Received: by mail-ob0-f173.google.com with SMTP id vb8so3939882obc.32 for ; Thu, 16 Jan 2014 23:11:35 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <1389939896.8062.53.camel@porter.coelho.fi> References: <1389939896.8062.53.camel@porter.coelho.fi> Date: Fri, 17 Jan 2014 08:11:34 +0100 Message-ID: (sfid-20140117_081139_505666_4EA379B4) Subject: Re: [PATCH 1/2] b43: fix the wrong assignment of status.freq in b43_rx() From: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= To: Luca Coelho Cc: ZHAO Gang , "linux-wireless@vger.kernel.org" , b43-dev , Stefano Brivio Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: 2014/1/17 Luca Coelho : > On Fri, 2014-01-17 at 13:27 +0800, ZHAO Gang wrote: >> In following patch, replace b43 specific helper function with kernel >> api to reduce code duplication. >> >> Signed-off-by: ZHAO Gang >> --- >> drivers/net/wireless/b43/xmit.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/net/wireless/b43/xmit.c b/drivers/net/wireless/b43/xmit.c >> index 4ae63f4..50e5ddb 100644 >> --- a/drivers/net/wireless/b43/xmit.c >> +++ b/drivers/net/wireless/b43/xmit.c >> @@ -821,10 +821,10 @@ void b43_rx(struct b43_wldev *dev, struct sk_buff *skb, const void *_rxhdr) >> * channel number in b43. */ >> if (chanstat & B43_RX_CHAN_5GHZ) { >> status.band = IEEE80211_BAND_5GHZ; >> - status.freq = b43_freq_to_channel_5ghz(chanid); >> + status.freq = b43_channel_to_freq_5ghz(chanid); >> } else { >> status.band = IEEE80211_BAND_2GHZ; >> - status.freq = b43_freq_to_channel_2ghz(chanid); >> + status.freq = b43_channel_to_freq_2ghz(chanid); >> } >> break; >> default: > > Why do you need this patch if you're going to remove these calls in the > next patch anyway? I was thinking about this for a moment too. You could just make a one patch and note in commit message that "translation" was reversed.