From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kalle Valo Subject: Re: [PATCH] net: brcm80211: add a check for the status of usb_register Date: Tue, 08 Jan 2019 18:43:53 +0200 Message-ID: <87zhsbqeuu.fsf@kamboji.qca.qualcomm.com> References: <20181226012227.70524-1-kjlu@umn.edu> Mime-Version: 1.0 Content-Type: text/plain Cc: pakki001@umn.edu, Arend van Spriel , Franky Lin , Hante Meuleman , Chi-Hsien Lin , Wright Feng , "David S. Miller" , Pieter-Paul Giesberts , linux-wireless@vger.kernel.org, brcm80211-dev-list.pdl@broadcom.com, brcm80211-dev-list@cypress.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: Kangjie Lu Return-path: In-Reply-To: <20181226012227.70524-1-kjlu@umn.edu> (Kangjie Lu's message of "Tue, 25 Dec 2018 19:22:24 -0600") Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org 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