Re: [PATCH] [Outreachy kernel] staging: wfx: alignment should match open parenthesis

[Stefano Brivio <sbrivio@redhat.com>]

On Mon, 9 Mar 2020 17:23:42 +0530
Payal Kshirsagar <payalskshirsagar1234@gmail.com> wrote:

> On Mon, Mar 9, 2020, 5:11 PM Stefano Brivio <sbrivio@redhat.com> wrote:
> 
> > On Mon,  9 Mar 2020 00:25:18 +0530
> > Payal Kshirsagar <payalskshirsagar1234@gmail.com> wrote:
> >  
> > > Align code to open parenthesis.
> > >
> > > Signed-off-by: Payal Kshirsagar <payalskshirsagar1234@gmail.com>
> > > ---
> > >  drivers/staging/wfx/data_tx.c | 6 +++---
> > >  1 file changed, 3 insertions(+), 3 deletions(-)
> > >
> > > diff --git a/drivers/staging/wfx/data_tx.c  
> > b/drivers/staging/wfx/data_tx.c  
> > > index 8b0c52392319..0038dc7666cf 100644
> > > --- a/drivers/staging/wfx/data_tx.c
> > > +++ b/drivers/staging/wfx/data_tx.c
> > > @@ -301,8 +301,8 @@ static void wfx_tx_manage_pm(struct wfx_vif *wvif,  
> > struct ieee80211_hdr *hdr,  
> > >  }
> > >
> > >  static u8 wfx_tx_get_raw_link_id(struct wfx_vif *wvif,
> > > -                                   struct ieee80211_sta *sta,
> > > -                                   struct ieee80211_hdr *hdr)
> > > +                              struct ieee80211_sta *sta,
> > > +                              struct ieee80211_hdr *hdr)
> > >  {
> > >       struct wfx_sta_priv *sta_priv =
> > >               sta ? (struct wfx_sta_priv *) &sta->drv_priv : NULL;
> > > @@ -369,7 +369,7 @@ static void wfx_tx_fixup_rates(struct  
> > ieee80211_tx_rate *rates)  
> > >  }
> > >
> > >  static u8 wfx_tx_get_rate_id(struct wfx_vif *wvif,
> > > -                               struct ieee80211_tx_info *tx_info)
> > > +                          struct ieee80211_tx_info *tx_info)
> > >  {
> > >       bool tx_policy_renew = false;
> > >       u8 rate_id;  
> >
> > Change looks good, more Cc:'s needed.
> >  
> 
> Is is needed to resend all these patches to send them to the other
> maintainers?

I think it would be nice, yes.

-- 
Stefano