Message ID | 20240528110246.477321-1-marcin.slusarz@gmail.com |
---|---|
State | New |
Headers | show |
Series | wifi: rtw88: schedule rx work after everything is set up | expand |
Marcin Ślusarz <marcin.slusarz@gmail.com> wrote: > From: Marcin Ślusarz <mslusarz@renau.com> > > Reported-by: Tim K <tpkuester@gmail.com> > Closes: > https://lore.kernel.org/linux-wireless/CA+shoWQ7P49jhQasofDcTdQhiuarPTjYEDa--NiVVx494WcuQw@mail.gmail. > com/ I gave this suggestions too early, since we have not gotten test result from Tim. I will change them to "Link:" if no ACK from Tim while merging. > Signed-off-by: Marcin Ślusarz <mslusarz@renau.com> > Cc: Ping-Ke Shih <pkshih@realtek.com> > Cc: Larry Finger <Larry.Finger@lwfinger.net> > Cc: Kalle Valo <kvalo@kernel.org> > Cc: linux-wireless@vger.kernel.org > Cc: linux-kernel@vger.kernel.org This is v2 version, so mail subject should be "[PATCH v2] ....", and add change log here, like: --- (delimiter is important here) v2: add Reported-by and Closes. > --- > drivers/net/wireless/realtek/rtw88/usb.c | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > Also I would prefer to point out "usb" in subject, please use "wifi: rtw88: usb: " as prefix. [...]
> I gave this suggestions too early, since we have not gotten test result from Tim. > I will change them to "Link:" if no ACK from Tim while merging. Hey all, thanks for reaching out! Sadly I'm not able to work on this project right now, but I've forwarded this email to a few colleagues to bring them in the loop. Did you have a timeline you were looking at to close this off? - Tim
Tim K <tpkuester@gmail.com> wrote: > > > I gave this suggestions too early, since we have not gotten test result from Tim. > > I will change them to "Link:" if no ACK from Tim while merging. > > Hey all, thanks for reaching out! > > Sadly I'm not able to work on this project right now, but I've > forwarded this email to a few colleagues to bring them in the loop. > Did you have a timeline you were looking at to close this off? Thanks for your reply. I would take this patch to 6.11, so you have about 4-5 weeks. Is it enough to you? Ping-Ke
diff --git a/drivers/net/wireless/realtek/rtw88/usb.c b/drivers/net/wireless/realtek/rtw88/usb.c index a0188511099a..98f81e3ae13e 100644 --- a/drivers/net/wireless/realtek/rtw88/usb.c +++ b/drivers/net/wireless/realtek/rtw88/usb.c @@ -740,7 +740,6 @@ static struct rtw_hci_ops rtw_usb_ops = { static int rtw_usb_init_rx(struct rtw_dev *rtwdev) { struct rtw_usb *rtwusb = rtw_get_usb_priv(rtwdev); - int i; rtwusb->rxwq = create_singlethread_workqueue("rtw88_usb: rx wq"); if (!rtwusb->rxwq) { @@ -752,13 +751,19 @@ static int rtw_usb_init_rx(struct rtw_dev *rtwdev) INIT_WORK(&rtwusb->rx_work, rtw_usb_rx_handler); + return 0; +} + +static void rtw_usb_setup_rx(struct rtw_dev *rtwdev) +{ + struct rtw_usb *rtwusb = rtw_get_usb_priv(rtwdev); + int i; + for (i = 0; i < RTW_USB_RXCB_NUM; i++) { struct rx_usb_ctrl_block *rxcb = &rtwusb->rx_cb[i]; rtw_usb_rx_resubmit(rtwusb, rxcb); } - - return 0; } static void rtw_usb_deinit_rx(struct rtw_dev *rtwdev) @@ -895,6 +900,8 @@ int rtw_usb_probe(struct usb_interface *intf, const struct usb_device_id *id) goto err_destroy_rxwq; } + rtw_usb_setup_rx(rtwdev); + return 0; err_destroy_rxwq: