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

[Luka Gejak]

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