Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230619175703.18826-1-franziska.naepelt@gmail.com>
 <20230619180753.18998-1-franziska.naepelt@gmail.com> <2023062012-regain-vintage-2c6d@gregkh>
In-Reply-To: <2023062012-regain-vintage-2c6d@gregkh>
From:   =?UTF-8?Q?Franziska_N=C3=A4pelt?= 
        <franziska.naepelt@googlemail.com>
Date:   Wed, 21 Jun 2023 21:02:56 +0200
Message-ID: <CAAUT3iN6r2KHxMO=mF0AjV_7eGvm8ecq3tahvB_yDB+0HQ1Xpg@mail.gmail.com>
Subject: Re: [PATCH 2/5] staging: rtl8723bs: Fix space issues
To:     Greg KH <gregkh@linuxfoundation.org>
Cc:     eperi1024@gmail.com, hdegoede@redhat.com, johannes.berg@intel.com,
        linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev,
        quic_vjakkam@quicinc.com, tegongkang@gmail.com
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

Hi Greg,

Am Di., 20. Juni 2023 um 16:12 Uhr schrieb Greg KH <gregkh@linuxfoundation.org>:
>
> On Mon, Jun 19, 2023 at 08:07:53PM +0200, Franziska Naepelt wrote:
> > Fix the following checkpatch space issues:
> > - CHECK: spaces preferred around that '*' (ctx:VxV)
> > - CHECK: spaces preferred around that '+' (ctx:VxV)
> > - CHECK: spaces preferred around that '-' (ctx:VxV)
> > - CHECK: spaces preferred around that '|' (ctx:VxV)
> > - CHECK: No space is necessary after a cast
> > - WARNING: please, no spaces at the start of a line
> >
> > Signed-off-by: Franziska Naepelt <franziska.naepelt@gmail.com>
> > ---
> >  .../staging/rtl8723bs/os_dep/ioctl_cfg80211.c | 77 +++++++++----------
> >  1 file changed, 38 insertions(+), 39 deletions(-)
> >
> > diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
> > index 1afd1a93bcee..abda4e0f0bf5 100644
> > --- a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
> > +++ b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
> > @@ -95,14 +95,14 @@ static struct ieee80211_channel rtw_2ghz_channels[] = {
> >  static void rtw_2g_channels_init(struct ieee80211_channel *channels)
> >  {
> >       memcpy((void *)channels, (void *)rtw_2ghz_channels,
> > -             sizeof(struct ieee80211_channel)*RTW_2G_CHANNELS_NUM
> > +             sizeof(struct ieee80211_channel) * RTW_2G_CHANNELS_NUM
> >       );
> >  }
> >
> >  static void rtw_2g_rates_init(struct ieee80211_rate *rates)
> >  {
> >       memcpy(rates, rtw_g_rates,
> > -             sizeof(struct ieee80211_rate)*RTW_G_RATES_NUM
> > +             sizeof(struct ieee80211_rate) * RTW_G_RATES_NUM
> >       );
> >  }
> >
> > @@ -126,8 +126,8 @@ static struct ieee80211_supported_band *rtw_spt_band_alloc(
> >       if (!spt_band)
> >               goto exit;
> >
> > -     spt_band->channels = (struct ieee80211_channel *)(((u8 *)spt_band)+sizeof(struct ieee80211_supported_band));
> > -     spt_band->bitrates = (struct ieee80211_rate *)(((u8 *)spt_band->channels)+sizeof(struct ieee80211_channel)*n_channels);
> > +     spt_band->channels = (struct ieee80211_channel *)(((u8 *)spt_band) + sizeof(struct ieee80211_supported_band));
> > +     spt_band->bitrates = (struct ieee80211_rate *)(((u8 *)spt_band->channels) + sizeof(struct ieee80211_channel) * n_channels);
> >       spt_band->band = band;
> >       spt_band->n_channels = n_channels;
> >       spt_band->n_bitrates = n_bitrates;
> > @@ -247,10 +247,10 @@ struct cfg80211_bss *rtw_cfg80211_inform_bss(struct adapter *padapter, struct wl
> >               u32 wpsielen = 0;
> >               u8 *wpsie = NULL;
> >
> > -             wpsie = rtw_get_wps_ie(pnetwork->network.ies+_FIXED_IE_LENGTH_, pnetwork->network.ie_length-_FIXED_IE_LENGTH_, NULL, &wpsielen);
> > +             wpsie = rtw_get_wps_ie(pnetwork->network.ies + _FIXED_IE_LENGTH_, pnetwork->network.ie_length - _FIXED_IE_LENGTH_, NULL, &wpsielen);
> >
> >               if (wpsie && wpsielen > 0)
> > -                     psr = rtw_get_wps_attr_content(wpsie,  wpsielen, WPS_ATTR_SELECTED_REGISTRAR, (u8 *)(&sr), NULL);
> > +                     psr = rtw_get_wps_attr_content(wpsie, wpsielen, WPS_ATTR_SELECTED_REGISTRAR, (u8 *)(&sr), NULL);
> >
> >               if (sr != 0) {
> >                       /* it means under processing WPS */
> > @@ -266,7 +266,6 @@ struct cfg80211_bss *rtw_cfg80211_inform_bss(struct adapter *padapter, struct wl
> >       }
> >       /* spin_unlock_bh(&pwdev_priv->scan_req_lock); */
> >
> > -
>
> This was an extra blank line issue that you did not mention in the
> changelog :(
>
> Please be more careful.
>
> thanks,
>
> greg k-h

Thanks for the heads up, it's fixed in v2.
https://lore.kernel.org/linux-staging/20230621184635.25064-1-franziska.naepelt@gmail.com/

---
Franziska