Hi Stanislav,
this patch is ok with me. For future patches, can you post to
devel@xxxxxxxxxxxxxxxxxxxxxx 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@xxxxxxxxxxxx>
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@xxxxxxxxxxx>
---
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] =
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
