Re: [PATCH] net: brcm80211: add a check for the status of usb_register
Arend Van Spriel writes: > On 1/8/2019 5:43 PM, Kalle Valo wrote: >> Kangjie Lu writes: >> >>> usb_register() may fail, so let's check its status and issue an error >>> message if it fails. >>> >>> Signed-off-by: Kangjie Lu >>> --- >>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c | 6 +- >> >> The title prefix should be "brcmfmac: " but I can fix that. > > Hi Kalle, > > I wanted to ack this patch, but can not locate it in my email not in > linux-wireless patchwork. Was it posted to the networking list only? I do see it on linux-wireless patchwork: https://patchwork.kernel.org/patch/10742769/ > Using scripts/get_maintainer.pl with the patch would reveal the lists > to post to in order of preference. For wireless driver changes I > refrain to linux-wireless only, but that is a personal choice I guess. Yeah, I guess in general wireless patches are just noise for netdev people. -- Kalle Valo
Re: [PATCH] net: brcm80211: add a check for the status of usb_register
On 1/8/2019 5:43 PM, Kalle Valo wrote: Kangjie Lu writes: usb_register() may fail, so let's check its status and issue an error message if it fails. Signed-off-by: Kangjie Lu --- drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c | 6 +- The title prefix should be "brcmfmac: " but I can fix that. Hi Kalle, I wanted to ack this patch, but can not locate it in my email not in linux-wireless patchwork. Was it posted to the networking list only? Using scripts/get_maintainer.pl with the patch would reveal the lists to post to in order of preference. For wireless driver changes I refrain to linux-wireless only, but that is a personal choice I guess. Regards, Arend https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#commit_title_is_wrong
Re: [PATCH] net: brcm80211: add a check for the status of usb_register
Kangjie Lu writes: > usb_register() may fail, so let's check its status and issue an error > message if it fails. > > Signed-off-by: Kangjie Lu > --- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/usb.c | 6 +- The title prefix should be "brcmfmac: " but I can fix that. https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#commit_title_is_wrong -- Kalle Valo
