[PATCH] brcm80211: remove multiple calls to wl_release_fw
Roland Vossen
rvossen at broadcom.com
Mon Feb 14 19:59:15 UTC 2011
Hi Stanislav,
this patch is ok with me. For future patches, can you post to
devel at linuxdriverproject.org since that is the main list for the
brcm80211 driver ?
I will align yet unsubmitted patches having to do with firmware handling
with your patch.
Thanks, Roland.
Reviewed-by: Roland Vossen <rvossen at broadcom.com>
On 02/12/2011 04:00 PM, Stanislav Fomichev wrote:
> This causes an oops when the bcm43xx_hdr-0.fw binary is not found.
>
> Signed-off-by: Stanislav Fomichev<s at fomichev.me>
> ---
> drivers/staging/brcm80211/sys/wl_mac80211.c | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/brcm80211/sys/wl_mac80211.c b/drivers/staging/brcm80211/sys/wl_mac80211.c
> index cd8392b..5faa521 100644
> --- a/drivers/staging/brcm80211/sys/wl_mac80211.c
> +++ b/drivers/staging/brcm80211/sys/wl_mac80211.c
> @@ -1767,7 +1767,6 @@ static int wl_request_fw(struct wl_info *wl, struct pci_dev *pdev)
> if (status) {
> printf("%s: fail to load firmware %s\n",
> KBUILD_MODNAME, fw_name);
> - wl_release_fw(wl);
> return status;
> }
> WL_NONE("request fw %s\n", fw_name);
> @@ -1777,7 +1776,6 @@ static int wl_request_fw(struct wl_info *wl, struct pci_dev *pdev)
> if (status) {
> printf("%s: fail to load firmware %s\n",
> KBUILD_MODNAME, fw_name);
> - wl_release_fw(wl);
> return status;
> }
> wl->fw.hdr_num_entries[i] =
More information about the devel
mailing list