Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> writes: > Replace two instances of bare pr_info with dev_info. > > Also make their messages a little more informative. > > Signed-off-by: Bitterblue Smith <rtl8821cerfe2@xxxxxxxxx> [...] > @@ -5420,7 +5421,8 @@ static void rtl8xxxu_rx_urb_work(struct work_struct *work) > rtl8xxxu_queue_rx_urb(priv, rx_urb); > break; > default: > - pr_info("failed to requeue urb %i\n", ret); > + dev_info(&priv->udev->dev, > + "failed to requeue urb with error %i\n", ret); Should this be dev_warn()? It looks like a warning. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches