On Sat, 30 Dec, 2023 12:04:02 -0600 "Larry Finger" <[email protected]> wrote:
> On 12/29/23 22:51, Rahul Rameshbabu wrote:
>> + if (dev->qos_enabled)
>> + ieee80211_stop_queue(dev->wl->hw,
>> + skb_get_queue_mapping(skb));
>> + else
>> + ieee80211_stop_queue(dev->wl->hw, 0);
>
> This code sequence occurs in several places. Would it be better to put this in a
> routine specific to b43, thus it would only be used once?
Right, I am waiting to converge on the discussion in the second patch in
this series before making this refactor, but I agree that this pattern
is prevelant and should be refactored if this is the approach taken.
>
> We certainly could try extracting firmware from a newer binary. Any suggestions?
Unfortunately, new firmware would not prevent the need to fix up the
code with regards to QoS being disabled via the kernel parameter or
using OpenFW. That said, new firmware could help us drop the fifth patch
in this series. I am thinking about using b43-fwcutter to extract
proprietary fw from a newer release of broadcom-wl to see if that makes
a difference. That said, I am a bit puzzled since the device I am
testing on released in early 2011, and I used firmware released in late
2012.
--
Thanks,
Rahul Rameshbabu