2014/1/17 Luca Coelho <l...@coelho.fi>:
> 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 <gamer...@gmail.com>
>> ---
>> 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.
_______________________________________________
b43-dev mailing list
b43-dev@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/b43-dev
