Skip to content

Commit

Permalink
staging: brcm80211: fixed checkpatch warnings for brcmutil dir
Browse files Browse the repository at this point in the history
Most of them being 'line longer than 80 chars' type of warning.

Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Roland Vossen authored and Greg Kroah-Hartman committed Aug 23, 2011
1 parent 54d7879 commit 7e0395e
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 11 deletions.
9 changes: 6 additions & 3 deletions drivers/staging/brcm80211/brcmutil/utils.c
Original file line number Diff line number Diff line change
Expand Up @@ -489,8 +489,8 @@ static const u8 crc8_table[256] = {
};

u8 brcmu_crc8(u8 *pdata, /* pointer to array of data to process */
uint nbytes, /* number of input data bytes to process */
u8 crc /* either CRC8_INIT_VALUE or previous return value */
uint nbytes, /* number of input data bytes to process */
u8 crc /* either CRC8_INIT_VALUE or previous return value */
) {
/* loop over the buffer data */
while (nbytes-- > 0)
Expand Down Expand Up @@ -586,7 +586,10 @@ brcmu_format_flags(const struct brcmu_bit_desc *bd, u32 flags, char *buf,
}
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 i;
Expand Down
23 changes: 15 additions & 8 deletions drivers/staging/brcm80211/brcmutil/wifi.c
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,9 @@ bool brcmu_chspec_malformed(u16 chanspec)
EXPORT_SYMBOL(brcmu_chspec_malformed);

/*
* This function returns the channel number that control traffic is being sent on, for legacy
* channels this is just the channel number, for 40MHZ channels it is the upper or lowre 20MHZ
* sideband depending on the chanspec selected
* This function returns the channel number that control traffic is being sent
* on, for legacy channels this is just the channel number, for 40MHZ channels
* it is the upper or lower 20MHZ sideband depending on the chanspec selected.
*/
u8 brcmu_chspec_ctlchan(u16 chspec)
{
Expand All @@ -55,15 +55,22 @@ u8 brcmu_chspec_ctlchan(u16 chspec)
if (CHSPEC_CTL_SB(chspec) == WL_CHANSPEC_CTL_SB_NONE) {
return CHSPEC_CHANNEL(chspec);
} else {
/* we only support 40MHZ with sidebands */
/* chanspec channel holds the centre frequency, use that and the
* side band information to reconstruct the control channel number
/*
* we only support 40MHZ with sidebands. chanspec channel holds
* 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)
/* 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));
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));
}

Expand Down

0 comments on commit 7e0395e

Please sign in to comment.