Fedor Pchelkin <pchelkin@ispras.ru> wrote:
> The first two patches concern memory leak issues found during testing.
>
> The other ones implement TX completion functionality missing for the USB
> part of rtw89 driver, suggested by Bitterblue Smith [1]. This will allow
> handling TX wait skbs and the ones flagged with IEEE80211_TX_CTL_REQ_TX_STATUS
> correctly.
>
> rtw89 has several ways of handling TX status report events. The first one
> is based on RPP feature which is used by PCIe HCI. The other one depends
> on firmware sending a corresponding C2H message, quite similar to what
> rtw88 has. RTL8851BU vendor driver [2] was taken for reference.
>
> [1]: https://lore.kernel.org/linux-wireless/0cb4d19b-94c7-450e-ac56-8b0d4a1d889f@gmail.com/
> [2]: https://github.com/fofajardo/rtl8851bu.git
>
> Series has been tested to work with RTL8851BU (USB) and RTL8852BE (PCIe)
> devices.
>
> Sorry for the inconvenience with the timing when the series is sent. It's
> not extremely urgent, and I'd gladly appreciate if it'd be reviewed for
> any issues I'm not aware of. Testing with other USB chips would be great,
> too. Thanks!
Thanks for your patches and description that I understand what you are going
to do. I'd like give people enough time to review this patchset, so I don't
merge this immediately, and also I can review them deeply days later.