2012-05-11 19:50:05

by Bouterse, Robert

Subject: Re: ath9k_htc fails with

Sujith Manoharan <c_manoha@...> writes:

>
> David Madore wrote:
> > In other words: loading the module seems to place the dongle in a bad
> > state that goes away after ifconfig up (but not by removing the
> > modules), and if a reboot happens while the dongle is in that bad
> > state, it will fail permanently.
>
> Right, there's a difference here, a 'reboot' of the card is done only when
> if the interface is brought up. The device status from the USB subsystem
> is used to handle this - it works on x86.
>
> On your platform, I think that a full reboot is required even when doing just
> a simple load/unload cycle. I'll come up with a patch to test this.
>
> Thanks for the detailed log.
>
> Sujith
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo@...
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
Hi Sujith,
I seem to running into this same issue, mine is that the driver emits the
"Target unresponsive" error if the unit is powered-up with the TL-WN721N plugged
in already. I think my issue is that the USB subsytem event is not recieved by
the atk9k_htc driver, and therefore the hardware is not reset.

Has this ben fixed already? Can you give me the commit sha1 in wireless-testing?
I am using compat-wireless, so I am lagging master a bit.

If not, can you point me at where in the driver code the reset happens? Maybe I
can come up with a patch.

2012-05-14 09:20:23

by Sujith Manoharan

Subject: Re: ath9k_htc fails with

Robert Bouterse wrote:
> I seem to running into this same issue, mine is that the driver emits the
> "Target unresponsive" error if the unit is powered-up with the TL-WN721N plugged
> in already. I think my issue is that the USB subsytem event is not recieved by
> the atk9k_htc driver, and therefore the hardware is not reset.
>
> Has this ben fixed already? Can you give me the commit sha1 in wireless-testing?
> I am using compat-wireless, so I am lagging master a bit.
>
> If not, can you point me at where in the driver code the reset happens? Maybe I
> can come up with a patch.

You can play around with the function ath9k_hif_usb_disconnect() in
drivers/net/wireless/ath/ath9k/hif_usb.c which has the deinit/reboot code. :)

Sujith

2012-05-14 22:23:42

by Bouterse, Robert

Subject: RE: ath9k_htc fails with

RllJLCBJZiB5b3UgYXJlIGludGVyZXN0ZWQgaW4gd2hlcmUgSSBoYXZlIGdvdHRlbiB3aXRoIHRo
aXMsIGxldCBtZSBleHBsYWluOg0KSSBoYXZlIHRoZSBUTC1XTjcyMU4gcGx1Z2dlZCBpbnRvIGEg
c2VsZiBwb3dlcmVkIGh1Yi4NCkkgYW0gcnVubmluZyBvbiBhIEZyZWVzY2FsZSBpTVg1MF9yZDMg
RVZNLg0KDQpJIGhhdmUgZm91bmQgdGhhdCB0aGUgdXNiIGRpc2Nvbm5lY3QgY2FsbGJhY2sgZG9l
cyBub3QgYWx3YXlzIGdldCBjYWxsZWQsIGZvciBpbnN0YW5jZSBvbiBzaHV0ZG93bi4NCg0KV2hl
biB0aGUgZGV2aWNlIGlzIGluIHRoZSBVbnJlc3BvbnNpdmUgc3RhdGUsIEkgZ2V0IHRoZSBmb2xs
aW5nIHN0YXRlIG91dHB1dCBvbiBtb2R1bGUgbG9hZDoNCg0KCXVzYiAyLTEuMzogYXRoOWtfaHRj
OiBUcmFuc2ZlcnJlZCBGVzogaHRjXzkyNzEuZncsIHNpemU6IDUxMjcyICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICANCglhdGg5a19odGMgMi0xLjM6MS4wOiBhdGg5
a19odGM6IFRhcmdldCBpcyB1bnJlc3BvbnNpdmUgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgDQoJRmFpbGVkIHRvIGluaXRpYWxpemUgdGhlIGRldmlj
ZSAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgIA0KCXVzYiAyLTEuMzogYXRoOWtfaGlmX3VzYl9kZXZfZGVpbml0
OjEwNjUgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICANCglhdGg5a19oaWZfdXNiX2Rldl9kZWluaXQ6MTA2NiAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgDQoJdXNiIDItMS4zOiBhdGg5a19odGM6IFVTQiBTdGF0ZSA3LCB1bnBsdWdnZWQg
PSAwLCBoaWZfZGV2LT5mbGFncyA9IDAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgIA0KCXVzYiAyLTEuMzogYXRoOWtfaHRjOiBVU0IgbGF5ZXIgZGVpbml0aWFsaXplZA0KDQpJ
IGFkZGVkIHRoZSBsb2cgc3RhdGVtZW50IHRvIGdldCB0aGUgc3RhdGUganVzdCBiZWZvcmUgdGhl
IGlmIHN0YXRlbWVudCBnYXRpbmcgdGhlIGNhbGwgdG8gcmVib290Og0KQXMgeW91IGNhbiBzZWUg
dGhlIHVucGx1Z2dlZCBpZiBmYWxzZSBidXQgSElGX1VTQl9TVEFSVCBpcyBub3QgdHJ1ZSwgdGhl
cmVmb3JlIHRoZSByZWJvb3QgaXMgbm90IGNhbGxlZC4gSSBhbSBzdHVjayBpbiB0aGlzIHN0YXRl
IHVudGlsIEkgdW5wbHVnIHRoZSBkZXZpY2UuDQpUaGUgcHJvYmxlbSBpcyB0aGF0IEkgY2Fubm90
IGNhbGwgcmVib290IHVudGlsIGhpZl91c2Jfc3RhcnQgaXMgY2FsbGVkLg0KSSB0cmllZCBzaW1w
bGlmeWluZyB0aGUgY29uZGl0aW9uYWwgdG86DQoNCglkZXZfaW5mbygmdWRldi0+ZGV2LCAiYXRo
OWtfaHRjOiBVU0IgU3RhdGUgJWQsIHVucGx1Z2dlZCA9ICVkLCBoaWZfZGV2LT5mbGFncyA9ICVk
XG4iLCB1ZGV2LT5zdGF0ZSwgdW5wbHVnZ2VkLCBoaWZfZGV2LT5mbGFncyk7DQoJaWYgKCF1bnBs
dWdnZWQpew0KCQlhdGg5a19oaWZfdXNiX3JlYm9vdCh1ZGV2KTsNCgl9DQoNCmJ1dCB0aGF0IGp1
c3QgZ290IG1lOg0KDQoJdXNiIDItMS4zOiBhdGg5a19odGM6IFVTQiBTdGF0ZSA3LCB1bnBsdWdn
ZWQgPSAwLCBoaWZfZGV2LT5mbGFncyA9IDAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgIA0KCWF0aDlrX2h0YzogYXRoOWtfaGlmX3VzYl9yZWJvb3Q6MTIwMCAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICANCgl1c2IgMi0xLjM6IGF0aDlrX2h0YzogVVNCIHJlYm9vdCBmYWlsZWQgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgDQoJ
dXNiIDItMS4zOiBhdGg5a19odGM6IFVTQiBsYXllciBkZWluaXRpYWxpemVkICANCg0KU28sIGl0
IG1heSBiZSB0aGF0IHRoZSBkZXZpY2UgaXMgbm90IGNhcGFibGUgb2YgcHJvY2Vzc2luZyB0aGUg
cmVib290IGNvbW1hbmQgaW4gdGhpcyBzdGF0ZS4NCg0KSSBhcHBvbG9naXplIGZvciBteSBpbmV4
cGVyaWVuY2Ugd2l0aCB0aGUgdXNiIGFwaXMuDQoNCi8qLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t
LS0tLS0tLS0tLS0tLS0tLSovDQpCb2IgQm91dGVyc2UNClNlbmlvciBTb2Z0d2FyZSBFbmdpbmVl
cg0KTXVsdGltZWRpYSBEZXZlbG9wbWVudCBDZW50ZXINCkQmTSBIb2xkaW5ncyBJbmMuDQpodHRw
Oi8vd3d3LmRtLWhvbGRpbmdzLmNvbQ0KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fXw0KRnJvbTogU3VqaXRoIE1hbm9oYXJhbiBbY19tYW5vaGFAcWNhLnF1YWxjb21tLmNv
bV0NClNlbnQ6IE1vbmRheSwgTWF5IDE0LCAyMDEyIDU6MTkgQU0NClRvOiBCb3V0ZXJzZSwgUm9i
ZXJ0DQpDYzogbGludXgtd2lyZWxlc3NAdmdlci5rZXJuZWwub3JnDQpTdWJqZWN0OiBSZTogYXRo
OWtfaHRjIGZhaWxzIHdpdGgNCg0KUm9iZXJ0IEJvdXRlcnNlIHdyb3RlOg0KPiBJIHNlZW0gdG8g
cnVubmluZyBpbnRvIHRoaXMgc2FtZSBpc3N1ZSwgbWluZSBpcyB0aGF0IHRoZSBkcml2ZXIgZW1p
dHMgdGhlDQo+ICJUYXJnZXQgdW5yZXNwb25zaXZlIiBlcnJvciBpZiB0aGUgdW5pdCBpcyBwb3dl
cmVkLXVwIHdpdGggdGhlIFRMLVdONzIxTiBwbHVnZ2VkDQo+IGluIGFscmVhZHkuIEkgdGhpbmsg
bXkgaXNzdWUgaXMgdGhhdCB0aGUgVVNCIHN1YnN5dGVtIGV2ZW50IGlzIG5vdCByZWNpZXZlZCBi
eQ0KPiB0aGUgYXRrOWtfaHRjIGRyaXZlciwgYW5kIHRoZXJlZm9yZSB0aGUgaGFyZHdhcmUgaXMg
bm90IHJlc2V0Lg0KPg0KPiBIYXMgdGhpcyBiZW4gZml4ZWQgYWxyZWFkeT8gQ2FuIHlvdSBnaXZl
IG1lIHRoZSBjb21taXQgc2hhMSBpbiB3aXJlbGVzcy10ZXN0aW5nPw0KPiBJIGFtIHVzaW5nIGNv
bXBhdC13aXJlbGVzcywgc28gSSBhbSBsYWdnaW5nIG1hc3RlciBhIGJpdC4NCj4NCj4gSWYgbm90
LCBjYW4geW91IHBvaW50IG1lIGF0IHdoZXJlIGluIHRoZSBkcml2ZXIgY29kZSB0aGUgcmVzZXQg
aGFwcGVucz8gTWF5YmUgSQ0KPiBjYW4gY29tZSB1cCB3aXRoIGEgcGF0Y2guDQoNCllvdSBjYW4g
cGxheSBhcm91bmQgd2l0aCB0aGUgZnVuY3Rpb24gYXRoOWtfaGlmX3VzYl9kaXNjb25uZWN0KCkg
aW4NCmRyaXZlcnMvbmV0L3dpcmVsZXNzL2F0aC9hdGg5ay9oaWZfdXNiLmMgd2hpY2ggaGFzIHRo
ZSBkZWluaXQvcmVib290IGNvZGUuIDopDQoNClN1aml0aA0KDQo=