On Thu, Jun 18, 2020 at 04:10:02PM +0200, Shihab Rashed wrote: > Fixed two brace coding style issues and one comment style issue found by > checkpatch. > > Signed-off-by: Shihab Rashed <shihab.rashe...@gmail.com> > --- > drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c | 15 +++++++-------- > 1 file changed, 7 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c > b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c > index 2fb80b6eb51d..3b06b60cdfd1 100644 > --- a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c > +++ b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c > @@ -117,15 +117,12 @@ static struct ieee80211_supported_band > *rtw_spt_band_alloc( > struct ieee80211_supported_band *spt_band = NULL; > int n_channels, n_bitrates; > > - if (band == NL80211_BAND_2GHZ) > - { > + if (band == NL80211_BAND_2GHZ) { ^^ too many spaces.
> n_channels = RTW_2G_CHANNELS_NUM; > n_bitrates = RTW_G_RATES_NUM; > - } > - else > - { > + } else { ^^ too many spaces. > goto exit; > - } > + } ^ Indented too far. > > spt_band = rtw_zmalloc(sizeof(struct ieee80211_supported_band) + > sizeof(struct ieee80211_channel) * n_channels + regards, dan carpenter