Re: [PATCH] brcm80211: fmac: Add error handling for brcmf_usb_dl_writeimage()
Markus Elfring <[email protected]>
| Newsgroups | dev.linux.lists.brcm80211,org.kernel.vger.linux-kernel,org.kernel.vger.linux-wireless,org.kernel.vger.stable |
|---|---|
| Message-ID | <[email protected]> |
> The function brcmf_usb_dl_writeimage() calls the function > brcmf_usb_dl_cmd() but dose not check its return value. The * Wording suggestion: The brcmf_usb_dl_cmd() function is called here. Unfortunately, its return value was not checked so far. * You may occasionally put more than 60 characters into text lines of such a change description. * I propose to replace the word “for” by “in” (before function names) in some summary phrases. Regards, Markus