Re: [1/3] rt2x00usb: initialize the read value in case of failure

[Stanislaw Gruszka <sgruszka@redhat.com>]

On Tue, Mar 17, 2015 at 09:34:14AM +0100, Sebastian Andrzej Siewior wrote:
> I have no idea what AutoRun mode is (it seems that the stick already
> has a firmware and is happy with it) and I have no clue why the firmware
> decides to lie about it. I looked at the vendor driver I found at github
> (which seems to be from 2012-10-22, DPO_RT5572_LinuxSTA_2.6.1.3) and
> well, I look again if someone says it is worth looking… They seem to do
> some kind "firmware is comming" magic which might be already done in the
> current driver at a different spot but I think the main question is why
> request is answered wrong. I *think* the vendor driver sends the
> USB_MODE_AUTORUN request only once but it is hard to tell…

Does the below patch solve the issue ?

diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
index 9a2f44a..3a6cae8 100644
--- a/drivers/net/wireless/rt2x00/rt2800usb.c
+++ b/drivers/net/wireless/rt2x00/rt2800usb.c
@@ -293,6 +293,7 @@ static int rt2800usb_write_firmware(struct rt2x00_dev *rt2x00dev,
 	if (retval) {
 		rt2x00_info(rt2x00dev,
 			    "Firmware loading not required - NIC in AutoRun mode\n");
+		__clear_bit(REQUIRE_FIRMWARE, &rt2x00dev->cap_flags);
 	} else {
 		rt2x00usb_register_multiwrite(rt2x00dev, FIRMWARE_IMAGE_BASE,
 					      data + offset, length);