RE: [PATCH rtw-next v2] wifi: rtl8xxxu: Detect the maximum supported channel width

Thorsten Leemhuis <[email protected]> wrote:
> On 5/6/26 09:57, Ping-Ke Shih wrote:
> > Bitterblue Smith <[email protected]> wrote:
> >
> >> Some devices malfunction when connected to a network with 40 MHz channel
> >> width, because they don't support that.
> >>
> >> RTL8188FU, RTL8192FU, and RTL8710BU (RTL8188GU) have a way to signal
> >> this (and some other capabilities) to the driver. Get this information
> >> from the hardware and advertise 40 MHz support only when the hardware
> >> can handle it. We assume the other chips can always handle it.
> >>
> >> RTL8710BU needs a different way to retrieve this information, which will
> >> be implemented some other time.
> >>
> >> Fixes: dbf9b7bb0edf ("wifi: rtl8xxxu: Enable 40 MHz width by default")
> >> Cc: [email protected]
> >> Closes: https://bugzilla.kernel.org/show_bug.cgi?id=221394
> >> Signed-off-by: Bitterblue Smith <[email protected]>
> >> Reviewed-by: Ping-Ke Shih <[email protected]>
> 
> Thx for fixing this!
> 
> > 1 patch(es) applied to rtw-next branch of rtw.git, thanks.
> > ef771eabc79d wifi: rtl8xxxu: Detect the maximum supported channel width
> > https://github.com/pkshih/rtw.git
> 
> rtw-next sounds like it aims for the next merge window; and it seems the
> fix hasn't even hit -next yet. This is slightly unfortunate, as this
> afaics is a fix for a recent regression -- so it ideally should head
> towards mainline by now[1], as Linus' the rule of thumb is to "generally
> fix regressions "within a week", preferably before the next rc"[1].
> 
> Or am I missing something? That might very well be the case, so do not
> hesitate to tell me!

As this patch applied to public rtw tree, and people who encountered the
problem in bugzilla can work again. To prevent breaking the public tree,
I'd keep it as was.

I will be carefully to handle this kind of patches later. 

Ping-Ke