Re: [PATCH] staging: rtl8723bs: remove unnecessary parentheses

[Luka Gejak]

October 15, 2026 at 10:11 PM, "Luka Gejak" <luka.gejak@xxxxxxxxx> wrote:


> 
> October 15, 2026 at 9:45 PM, "Dan Carpenter" <error27@xxxxxxxxx> wrote:
> 
> > 
> > On Thu, Oct 15, 2026 at 10:41:53AM -0500, Ethan Tidmore wrote:
> >  
> >  
> >  On Thu Oct 15, 2026 at 9:11 AM CDT, Luka Gejak wrote:
> >  October 15, 2026 at 2:06 PM, pavankumaryalagada@xxxxxxxxx wrote:
> >  
> >  
> >  From: Yalagada Pavan Kumar <pavankumaryalagada@xxxxxxxxx>
> >  
> >  Remove redundant parentheses around the address-of operator to
> >  follow kernel coding style.
> >  
> >  Signed-off-by: Yalagada Pavan Kumar <pavankumaryalagada@xxxxxxxxx>
> >  ---
> >  
> >  Hi Yalagada,
> >  
> >  I'm just letting you know that this change is no longer needed as the
> >  entire staging/rtl8723bs driver is being deprecated and removed.
> >  Support for RTL8723BS has been ported to rtw88 subsystem.
> >  
> >  So rtl8723bs patches aren't being accepted anymore? I saw Dan reviewing
> >  one earlier, just wondering.
> >  
> >  I didn't know it was being removed. My expectation is that we'll
> >  probably keep merging patches until it's actually removed. It's sometimes
> >  easier to just merge easy uncontroversial things without trying to track
> >  everything happening in the kernel. We should obviously delete the driver
> >  as soon as the other patch is merged.
> >  
> >  regards,
> >  dan carpenter
> > 
> But won't that cause a conflict? PR has been sent today, carrying my
> patch that removes whole staging/rtl8723bs into net-next and if
> another patch is merged into staging-next, whose time of commit is
> later than the time I made my commit localy it will try to exit a
> non-existent file.
> 
> And since both net-next tree and staging-next tree are merged at the
> same merge window this will cause merge conflict. Correct me if I am
> wrong, but this is what I think would happen.
> 
> Best regards,
> Luka Gejak
>

To clarify something: new driver, which replaces staging rtl8723bs and
the mentioned patch which removes the rtl8723bs from staging, have
been merged into  wireless-next and are awaiting to merged into the
net-next and eventually mainline when 7.4 merge window opens.

Best regards,
Luka Gejak