[PATCH 1/2] b43: N-PHY: band width setting with dumb clock control call
Rafał Miłecki
zajec5 at gmail.com
Sun Aug 22 19:35:14 EDT 2010
2010/8/23 Julian Calaby <julian.calaby at gmail.com>:
> Rafal,
>
> Couple of minor style issues, if you're re-submitting.
Thanks for reviewing! :)
> 2010/8/23 Rafał Miłecki <zajec5 at gmail.com>:
>> Signed-off-by: Rafał Miłecki <zajec5 at gmail.com>
>> ---
>> drivers/net/wireless/b43/phy_common.c | 20 ++++++++++++++++++++
>> drivers/net/wireless/b43/phy_common.h | 11 +++++++++++
>> drivers/net/wireless/b43/phy_n.c | 4 ++--
>> drivers/net/wireless/b43/phy_n.h | 1 -
>> 4 files changed, 33 insertions(+), 3 deletions(-)
>>
>> diff --git a/drivers/net/wireless/b43/phy_common.c b/drivers/net/wireless/b43/phy_common.c
>> index 8f7d7ef..b06e3f0 100644
>> --- a/drivers/net/wireless/b43/phy_common.c
>> +++ b/drivers/net/wireless/b43/phy_common.c
>> @@ -466,3 +466,23 @@ struct b43_c32 b43_cordic(int theta)
>>
>> return ret;
>> }
>> +
>> +/* http://bcm-v4.sipsolutions.net/802.11/PHY/ClkCtlClk */
>> +static void b43_clock_control(struct b43_wldev *dev, u32 mode)
>> +{
>> + ; /* TODO */
>
> You don't need the semicolon here, as I understand it.
Yeah, could be skipped without causing warning. However it was
replaced anyway with 2/2. And this code doesn't come from the patch
you comment in this thread ;)
>> +}
>> +
>> +/* http://bcm-v4.sipsolutions.net/802.11/PHY/BmacBwSet */
>> +void b43_bmac_set_b_width(struct b43_wldev *dev, u8 b_width)
>> +{
>> + bool fast = dev->phy.forcefastclk;;
>
> Double semicolon - and is it necessary to assign a local variable here?
Bingo for double semicolon. We need to keep copy of "forcefastclk" as
this can be changed in sub calls.
--
Rafał
More information about the b43-dev
mailing list