The usb wifi device AVM Fritz Stick N v2b (USB 057c:8501) is listed as supported by rt2800usb, but loading the driver fails for me (kernel 3.15-rc7 and older).
ieee80211 phy0: rt2x00_set_rt: Info - RT chipset 5592, rev 0222 detected
ieee80211 phy0: rt2x00usb_vendor_request: Error - Vendor Request 0x09 failed for offset 0x0000 with error -32
ieee80211 phy0: rt2x00lib_probe_dev: Error - Failed to allocate device
rt2800usb: probe of 1-1.3:1.0 failed with error -32
usbcore: registered new interface driver rt2800usb
>From searching the web, it looks like other users using the same device are hit by this error too, and some users of similar devices using the same driver experience similar errors.
I've tested these patches using a USB 057c:8501 wifi device in client mode (scanning + connect to WPA protected BSS + network access). With the patches applied, the device works fine. Without, it fails.
I've also tested that USB 148f:5370 (which is another usb wifi device using the same driver), which worked also without these patches applied, still works after applying these patches.
Regards,
M. Braun
---
Michael Braun (2):
rt2800usb:fix efuse detection
rt2800usb:fix hang during firmware load
drivers/net/wireless/rt2x00/rt2800usb.c | 30 +++++++++++++++++++++++++++---
1 file changed, 27 insertions(+), 3 deletions(-)
The device 057c:8501 (AVM Fritz! WLAN v2 rev. B) currently does not
load. One thing observed is that the vendors driver detects EFUSE mode
for this device, but rt2800usb does not. This is due to rt2800usb
lacking a check for the firmware mode present in the vendors driver,
that this patch adopts for rt2800usb.
With this patch applied, the 'RF chipset' detection does no longer fail.
Signed-off-by: Michael Braun <[email protected]>
---
drivers/net/wireless/rt2x00/rt2800usb.c | 16 +++++++++++++++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
index a49c3d7..b601422 100644
--- a/drivers/net/wireless/rt2x00/rt2800usb.c
+++ b/drivers/net/wireless/rt2x00/rt2800usb.c
@@ -735,11 +735,25 @@ static void rt2800usb_fill_rxdone(struct queue_entry *entry,
/*
* Device probe functions.
*/
+static int rt2800usb_efuse_detect(struct rt2x00_dev *rt2x00dev)
+{
+ __le32 fwMode;
+
+ rt2x00usb_vendor_request(rt2x00dev, USB_DEVICE_MODE,
+ USB_VENDOR_REQUEST_IN, 0, 0x11, &fwMode,
+ sizeof(fwMode), REGISTER_TIMEOUT_FIRMWARE);
+
+ if ((fwMode & 0x00000003) == 2)
+ return 1;
+
+ return rt2800_efuse_detect(rt2x00dev);
+}
+
static int rt2800usb_read_eeprom(struct rt2x00_dev *rt2x00dev)
{
int retval;
- if (rt2800_efuse_detect(rt2x00dev))
+ if (rt2800usb_efuse_detect(rt2x00dev))
retval = rt2800_read_eeprom_efuse(rt2x00dev);
else
retval = rt2x00usb_eeprom_read(rt2x00dev, rt2x00dev->eeprom,
On 06/01/2014 09:52 PM, Michael Braun wrote:
> The device 057c:8501 (AVM Fritz! WLAN v2 rev. B) boots into a state that does not actually require loading a firmware file.
Wrap the changelog lines on at least 80 chars please.
> The vendors driver finds out about this by checking a firmware state
> register, so this patch adopts this here.
> Finally, with this patch applied, my wifi dongle actually becomes
> usefull (scan + connect to wpa network works).
s/usefull/useful/.
> Signed-off-by: Michael Braun <[email protected]>
> ---
> drivers/net/wireless/rt2x00/rt2800usb.c | 14 ++++++++++++--
> 1 file changed, 12 insertions(+), 2 deletions(-)
> diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
> index b601422..71bf101 100644
> --- a/drivers/net/wireless/rt2x00/rt2800usb.c
> +++ b/drivers/net/wireless/rt2x00/rt2800usb.c
> @@ -240,6 +240,7 @@ static int rt2800usb_write_firmware(struct rt2x00_dev *rt2x00dev,
> int status;
> u32 offset;
> u32 length;
> + __le32 fwMode;
>
> /*
> * Check which section of the firmware we need.
> @@ -257,8 +258,17 @@ static int rt2800usb_write_firmware(struct rt2x00_dev *rt2x00dev,
> /*
> * Write firmware to device.
> */
> - rt2x00usb_register_multiwrite(rt2x00dev, FIRMWARE_IMAGE_BASE,
> - data + offset, length);
> + rt2x00usb_vendor_request(rt2x00dev, USB_DEVICE_MODE,
> + USB_VENDOR_REQUEST_IN, 0, 0x11, &fwMode,
> + sizeof(fwMode), REGISTER_TIMEOUT_FIRMWARE);
> +
> + if ((fwMode & 0x00000003) == 2) {
Same comment as for the patch #1.
> + rt2x00_info(rt2x00dev,
> + "Firmware loading not required - NIC in AutoRun mode\n");
This line should start under 'rt2xx00dev', like below.
> + } else {
> + rt2x00usb_register_multiwrite(rt2x00dev, FIRMWARE_IMAGE_BASE,
> + data + offset, length);
> + }
WBR, Sergei
Hello.
On 06/01/2014 09:52 PM, Michael Braun wrote:
> The device 057c:8501 (AVM Fritz! WLAN v2 rev. B) currently does not
> load. One thing observed is that the vendors driver detects EFUSE mode
> for this device, but rt2800usb does not. This is due to rt2800usb
> lacking a check for the firmware mode present in the vendors driver,
> that this patch adopts for rt2800usb.
> With this patch applied, the 'RF chipset' detection does no longer fail.
> Signed-off-by: Michael Braun <[email protected]>
> ---
> drivers/net/wireless/rt2x00/rt2800usb.c | 16 +++++++++++++++-
> 1 file changed, 15 insertions(+), 1 deletion(-)
> diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
> index a49c3d7..b601422 100644
> --- a/drivers/net/wireless/rt2x00/rt2800usb.c
> +++ b/drivers/net/wireless/rt2x00/rt2800usb.c
> @@ -735,11 +735,25 @@ static void rt2800usb_fill_rxdone(struct queue_entry *entry,
> /*
> * Device probe functions.
> */
> +static int rt2800usb_efuse_detect(struct rt2x00_dev *rt2x00dev)
> +{
> + __le32 fwMode;
> +
> + rt2x00usb_vendor_request(rt2x00dev, USB_DEVICE_MODE,
> + USB_VENDOR_REQUEST_IN, 0, 0x11, &fwMode,
> + sizeof(fwMode), REGISTER_TIMEOUT_FIRMWARE);
> +
> + if ((fwMode & 0x00000003) == 2)
Operating directly on '__le32'? That doesn't look right.
> + return 1;
> +
> + return rt2800_efuse_detect(rt2x00dev);
> +}
> +
WBR, Sergei
On Sun, Jun 01, 2014 at 11:06:11PM +0400, Sergei Shtylyov wrote:
> >+ rt2x00usb_vendor_request(rt2x00dev, USB_DEVICE_MODE,
> >+ USB_VENDOR_REQUEST_IN, 0, 0x11, &fwMode,
> >+ sizeof(fwMode), REGISTER_TIMEOUT_FIRMWARE);
> >+
> >+ if ((fwMode & 0x00000003) == 2)
>
> Operating directly on '__le32'? That doesn't look right.
rt2x00usb_register_read() helper can be used to do conversion. I also
dislike using 0x11 number instead of register name and prefer kernel
like variable names (i.e. fw_mode instead of fwMode).
Thanks
Stanislaw
The device 057c:8501 (AVM Fritz! WLAN v2 rev. B) boots into a state that does not actually require loading a firmware file.
The vendors driver finds out about this by checking a firmware state
register, so this patch adopts this here.
Finally, with this patch applied, my wifi dongle actually becomes
usefull (scan + connect to wpa network works).
Signed-off-by: Michael Braun <[email protected]>
---
drivers/net/wireless/rt2x00/rt2800usb.c | 14 ++++++++++++--
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
index b601422..71bf101 100644
--- a/drivers/net/wireless/rt2x00/rt2800usb.c
+++ b/drivers/net/wireless/rt2x00/rt2800usb.c
@@ -240,6 +240,7 @@ static int rt2800usb_write_firmware(struct rt2x00_dev *rt2x00dev,
int status;
u32 offset;
u32 length;
+ __le32 fwMode;
/*
* Check which section of the firmware we need.
@@ -257,8 +258,17 @@ static int rt2800usb_write_firmware(struct rt2x00_dev *rt2x00dev,
/*
* Write firmware to device.
*/
- rt2x00usb_register_multiwrite(rt2x00dev, FIRMWARE_IMAGE_BASE,
- data + offset, length);
+ rt2x00usb_vendor_request(rt2x00dev, USB_DEVICE_MODE,
+ USB_VENDOR_REQUEST_IN, 0, 0x11, &fwMode,
+ sizeof(fwMode), REGISTER_TIMEOUT_FIRMWARE);
+
+ if ((fwMode & 0x00000003) == 2) {
+ rt2x00_info(rt2x00dev,
+ "Firmware loading not required - NIC in AutoRun mode\n");
+ } else {
+ rt2x00usb_register_multiwrite(rt2x00dev, FIRMWARE_IMAGE_BASE,
+ data + offset, length);
+ }
rt2x00usb_register_write(rt2x00dev, H2M_MAILBOX_CID, ~0);
rt2x00usb_register_write(rt2x00dev, H2M_MAILBOX_STATUS, ~0);