Commit 7e0395ec authored by Roland Vossen's avatar Roland Vossen Committed by Greg Kroah-Hartman

staging: brcm80211: fixed checkpatch warnings for brcmutil dir

Most of them being 'line longer than 80 chars' type of warning.
Reviewed-by: default avatarPieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: default avatarArend van Spriel <arend@broadcom.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 54d7879c
...@@ -586,7 +586,10 @@ brcmu_format_flags(const struct brcmu_bit_desc *bd, u32 flags, char *buf, ...@@ -586,7 +586,10 @@ brcmu_format_flags(const struct brcmu_bit_desc *bd, u32 flags, char *buf,
} }
EXPORT_SYMBOL(brcmu_format_flags); EXPORT_SYMBOL(brcmu_format_flags);
/* print bytes formatted as hex to a string. return the resulting string length */ /*
* print bytes formatted as hex to a string. return the resulting
* string length
*/
int brcmu_format_hex(char *str, const void *bytes, int len) int brcmu_format_hex(char *str, const void *bytes, int len)
{ {
int i; int i;
......
...@@ -43,9 +43,9 @@ bool brcmu_chspec_malformed(u16 chanspec) ...@@ -43,9 +43,9 @@ bool brcmu_chspec_malformed(u16 chanspec)
EXPORT_SYMBOL(brcmu_chspec_malformed); EXPORT_SYMBOL(brcmu_chspec_malformed);
/* /*
* This function returns the channel number that control traffic is being sent on, for legacy * This function returns the channel number that control traffic is being sent
* channels this is just the channel number, for 40MHZ channels it is the upper or lowre 20MHZ * on, for legacy channels this is just the channel number, for 40MHZ channels
* sideband depending on the chanspec selected * it is the upper or lower 20MHZ sideband depending on the chanspec selected.
*/ */
u8 brcmu_chspec_ctlchan(u16 chspec) u8 brcmu_chspec_ctlchan(u16 chspec)
{ {
...@@ -55,15 +55,22 @@ u8 brcmu_chspec_ctlchan(u16 chspec) ...@@ -55,15 +55,22 @@ u8 brcmu_chspec_ctlchan(u16 chspec)
if (CHSPEC_CTL_SB(chspec) == WL_CHANSPEC_CTL_SB_NONE) { if (CHSPEC_CTL_SB(chspec) == WL_CHANSPEC_CTL_SB_NONE) {
return CHSPEC_CHANNEL(chspec); return CHSPEC_CHANNEL(chspec);
} else { } else {
/* we only support 40MHZ with sidebands */ /*
/* chanspec channel holds the centre frequency, use that and the * we only support 40MHZ with sidebands. chanspec channel holds
* side band information to reconstruct the control channel number * the centre frequency, use that and the side band information
* to reconstruct the control channel number
*/ */
if (CHSPEC_CTL_SB(chspec) == WL_CHANSPEC_CTL_SB_UPPER) if (CHSPEC_CTL_SB(chspec) == WL_CHANSPEC_CTL_SB_UPPER)
/* control chan is the upper 20 MHZ SB of the 40MHZ channel */ /*
* control chan is the upper 20 MHZ SB of the
* 40MHZ channel
*/
ctl_chan = UPPER_20_SB(CHSPEC_CHANNEL(chspec)); ctl_chan = UPPER_20_SB(CHSPEC_CHANNEL(chspec));
else else
/* control chan is the lower 20 MHZ SB of the 40MHZ channel */ /*
* control chan is the lower 20 MHZ SB of the
* 40MHZ channel
*/
ctl_chan = LOWER_20_SB(CHSPEC_CHANNEL(chspec)); ctl_chan = LOWER_20_SB(CHSPEC_CHANNEL(chspec));
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment