2012-12-12 08:46:11

by Luciano Coelho

[permalink] [raw]
Subject: Re: wl12xx over spi with no scan results

On Wed, 2012-12-12 at 08:18 +0000, Alberto Garau wrote:
> Hi Luca,
>
> yes that's a great news for us.
>
> Yesterday afternoon with "iwlist wlan0 scan" (not "iw" yet) I saw also other two networks essids (for a small amount of seconds only)
> which I'm interested in.
>
> This seems to say that the networks I would like to connect to are available to the wl1271.
>
> In the meantime that I'm cross building "iw", I would like to connect to an unprotected network called "Inpeco-test"(no WEP, no WPA).
>
> Would you be so kind to suggest me the serie of commands I have to launch to connect to this network ? I would
> like then to ping outside of course.

ifconfig wlan0 up
iw wlan0 connect -w Inpeco-test

Then, to ping, you need to set your IP address, either with a DHCP
client or statically.

> I have got also one technical question. In the source code of the driver I have found for wl1271_spi.c in kernel
> 2.6.37 I have interrupts set in RISING_EDGE while the datasheet (in attachment: 4.1.18 at pag 19) talks about FALLING_EDGE.
> Which is the right setting to be applied and why ?

We currently use level and high with one shot as default. There are
some platforms that use rising, though, so we have a platform_quirk
value that enables us to toggle that.

You could be missing some interrupts by using rising and this could be
the reason why you don't see the SSIDs consistently.

--
Luca.



2012-12-12 12:18:28

by Alberto Garau

[permalink] [raw]
Subject: RE: wl12xx over spi with no scan results

SGkgTHVjYSwNCg0KSSdtIHRyeWluZyB0byBjYWxpYnJhdGUgbXkgZGV2aWNlIGFzIHN1Z2dlc3Rl
ZCwgZm9sbG93aW5nIHRoZSBndWlkZSBmb3VuZCBhdDogaHR0cDovL3Byb2Nlc3NvcnMud2lraS50
aS5jb20vaW5kZXgucGhwL09NQVBfV2lyZWxlc3NfQ29ubmVjdGl2aXR5X0NhbGlicmF0aW9uX1By
b2NjZXNzX05MQ1ANCg0KSW4gcGFydGljdWxhciBJJ20gdHJ5aW5nIHRvIHJlcGxpY2F0ZSB0aGUg
ZXhhbXBsZSBhdCB0aGUgZW5kIG9mIHRoZSBkb2N1bWVudCBhbmQgYXQgc3RlcCAzIG9mIHRoZSBl
eGFtcGxlIEknbSBvYnRhaW5pbmc6DQoNCnJvb3RAKG5vbmUpOn4vdGktdXRpbHMjIC4vY2FsaWJy
YXRvciBwbHQgY2FsaWJyYXRlDQp3bDEyNzE6IHBvd2VyIHVwDQpjaGlwX2lkIHJlYWQgZnJvbSBt
ZW1vcnk6IDA0MDMwMTExDQp3bDEyNzFfZmV0Y2hfZmlybXdhcmUgLSBmZXRjaGluZyBmaXJtd2Fy
ZSB3bDEyNzEtZncuYmluDQp3bDEyNzFfZmV0Y2hfbnZzIC0gZmV0Y2hpbmcgZmlybXdhcmUgd2wx
MjcxLW52cy5iaW4NCmVuYWJsZV9pbnRlcnJ1cHRzX3dsMTI3MQ0Kd2wxMjcxOiBmaXJtd2FyZSBi
b290ZWQgaW4gUExUIG1vZGUgKFJldiA2LjEuMC41MC4zNTApDQpGYWlsIHRvIGNhbGlicmF0d2wx
MjcxOiBwb3dlciBkb3duDQplIGl0aCByYWRpbyBzdGF0ZGlzYWJsZV9pbnRlcnJ1cHRzX3dsMTI3
MQ0KdXMgKC04MCkNCnJvb3RAKG5vbmUpOn4vdGktdXRpbHMjDQoNCg0KQW55IGlkZWEgYWJvdXQg
d2h5IG15IGNhbGlicmF0aW9uIHByb2Nlc3MgaXMgZmFpbGluZyA/IEkgd2FzIGFibGUgdG8gY29t
cGxldGUgdGhlIGZpcnN0IHN0ZXBzIGNyZWF0aW5nIHRoZSBudnMgZnJvbSBpbmkgZmlsZXMgDQpC
dXQgSSdtIGZhaWxpbmcgaW4gImNhbGlicmF0b3IgcGx0IGNhbGlicmF0ZSIuIA0KDQpUaGFua3Mg
aW4gYWR2YW5jZSANCg0KQWxiZXJ0bw0KDQotLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KRnJv
bTogTHVjaWFubyBDb2VsaG8gW21haWx0bzpjb2VsaG9AdGkuY29tXSANClNlbnQ6IG1lcmNvbGVk
w6wgMTIgZGljZW1icmUgMjAxMiAxMToxNw0KVG86IEFsYmVydG8gR2FyYXUNCkNjOiBsaW51eC13
aXJlbGVzc0B2Z2VyLmtlcm5lbC5vcmc7IElkbyBZYXJpdjsgQXJpayBOZW10c292OyBBZHJpYW5v
IE1lbGlzOyBMdWNhIEZyZXNpDQpTdWJqZWN0OiBSZTogd2wxMnh4IG92ZXIgc3BpIHdpdGggbm8g
c2NhbiByZXN1bHRzDQoNCk9uIFdlZCwgMjAxMi0xMi0xMiBhdCAxMDoxMyArMDAwMCwgQWxiZXJ0
byBHYXJhdSB3cm90ZToNCj4gVGhhbmtzIEx1Y2EsDQo+IA0KPiB3ZSB0cmllZCB3aXRoIEZBTExJ
TkdfRURHRSBmb3IgdGhlIGludGVycnVwdCBidXQgdGhlIGRyaXZlciBnb3Qgc3R1Y2tlZCBzbyBJ
IGhhZCB0byBwdXQgYWdhaW4gUklTSU5HX0VER0UuDQo+IA0KPiBXZSBzdWNjZXNzZnVsbHkgY3Jv
c3MgY29tcGlsZWQgaXcgYnV0IGFsc28gd2l0aCB0aGF0IHRvb2wgd2UgYWx3YXlzIA0KPiBzZWUg
b25seSBvbiBFU1NJRCBmcm9tIG9uZSBzcGVjaWZpYyBBUA0KDQpJIGRpZG4ndCBleHBlY3QgdGhl
IHNjYW4gdG8gaW1wcm92ZSB3aXRoIGl3LCBidXQgYXQgbGVhc3Qgbm93IHlvdSdyZSB1c2luZyB0
aGUgcmlnaHQgdG9vbC4gOikNCg0KDQo+IHdoaWxlIHRoZSBvdGhlcnMgRVNTSUQgZnJvbSB0aGUg
c2Vjb25kIEFQIGFyZSBhdmFpbGFibGUgb25seSBzb21ldGltZS4gDQo+IA0KPiBUaGF0J3MgdmVy
eSBzdHJhbmdlLg0KPiANCj4gRG8geW91IHRoaW5rIHdlIGFyZSBmYWNpbmcgY2FsaWJyYXRpb24g
cHJvYmxlbXMgPw0KDQpZZXMsIEkgZ3Vlc3MgY2FsaWJyYXRpb24gaXMgdGhlIG1vc3QgbGlrZWx5
IGNhdXNlIG5vdy4NCg0KDQo+IEluIG91ciBDTUQwIHdlIGhhdmUNCj4gDQo+IGJpdCAxMyA9IDEg
SUIgaW50ZXJydXB0IGlzIGFjdGl2ZSBoaWdoDQo+IA0KPiBiaXQgMTQgPSAxIElCIGludGVycnVw
dCBpcyBvcGVuIGRyYWluDQo+IA0KPiBEbyB5b3UgdGhpbmsgaXMgcmlnaHQgYWNjb3JkaW5nIGRv
IFJJU0lOR19FREdFIGludGVycnVwdCA/DQoNCkkgdGhpbmsgdGhpcyBpcyBva2F5LiAgSSBkb24n
dCB0aGluayB0aGUgcHJvYmxlbSBpcyB0aGUgaW50ZXJydXB0IG5vdy4NCg0KWW91IHNob3VsZCB0
cnkgdG8gY2FsaWJyYXRlIHlvdXIgZGV2aWNlIGFuZCBnZW5lcmF0ZSBhIHByb3BlciBOVlMgZmls
ZS4NCg0KLS0NCkx1Y2EuDQoNCg0K


2012-12-12 10:17:33

by Luciano Coelho

[permalink] [raw]
Subject: Re: wl12xx over spi with no scan results

On Wed, 2012-12-12 at 10:13 +0000, Alberto Garau wrote:
> Thanks Luca,
>
> we tried with FALLING_EDGE for the interrupt but the driver got stucked so I had to put again RISING_EDGE.
>
> We successfully cross compiled iw but also with that tool we always see only on ESSID from one specific AP

I didn't expect the scan to improve with iw, but at least now you're
using the right tool. :)


> while the others ESSID from the second AP are available only sometime.
>
> That's very strange.
>
> Do you think we are facing calibration problems ?

Yes, I guess calibration is the most likely cause now.


> In our CMD0 we have
>
> bit 13 = 1 IB interrupt is active high
>
> bit 14 = 1 IB interrupt is open drain
>
> Do you think is right according do RISING_EDGE interrupt ?

I think this is okay. I don't think the problem is the interrupt now.

You should try to calibrate your device and generate a proper NVS file.

--
Luca.


2012-12-12 10:13:38

by Alberto Garau

[permalink] [raw]
Subject: RE: wl12xx over spi with no scan results

VGhhbmtzIEx1Y2EsDQoNCndlIHRyaWVkIHdpdGggRkFMTElOR19FREdFIGZvciB0aGUgaW50ZXJy
dXB0IGJ1dCB0aGUgZHJpdmVyIGdvdCBzdHVja2VkIHNvIEkgaGFkIHRvIHB1dCBhZ2FpbiBSSVNJ
TkdfRURHRS4NCg0KV2Ugc3VjY2Vzc2Z1bGx5IGNyb3NzIGNvbXBpbGVkIGl3IGJ1dCBhbHNvIHdp
dGggdGhhdCB0b29sIHdlIGFsd2F5cyBzZWUgb25seSBvbiBFU1NJRCBmcm9tIG9uZSBzcGVjaWZp
YyBBUA0KDQp3aGlsZSB0aGUgb3RoZXJzIEVTU0lEIGZyb20gdGhlIHNlY29uZCBBUCBhcmUgYXZh
aWxhYmxlIG9ubHkgc29tZXRpbWUuIA0KDQpUaGF0J3MgdmVyeSBzdHJhbmdlLg0KDQpEbyB5b3Ug
dGhpbmsgd2UgYXJlIGZhY2luZyBjYWxpYnJhdGlvbiBwcm9ibGVtcyA/DQoNCkluIG91ciBDTUQw
IHdlIGhhdmUNCg0KYml0IDEzID0gMSBJQiBpbnRlcnJ1cHQgaXMgYWN0aXZlIGhpZ2gNCg0KYml0
IDE0ID0gMSBJQiBpbnRlcnJ1cHQgaXMgb3BlbiBkcmFpbiANCg0KRG8geW91IHRoaW5rIGlzIHJp
Z2h0IGFjY29yZGluZyBkbyBSSVNJTkdfRURHRSBpbnRlcnJ1cHQgPw0KDQpUaGFua3MNCg0KQWxi
ZXJ0bw0KDQotLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KRnJvbTogTHVjaWFubyBDb2VsaG8g
W21haWx0bzpjb2VsaG9AdGkuY29tXSANClNlbnQ6IG1lcmNvbGVkw6wgMTIgZGljZW1icmUgMjAx
MiAwOTo0NQ0KVG86IEFsYmVydG8gR2FyYXUNCkNjOiBsaW51eC13aXJlbGVzc0B2Z2VyLmtlcm5l
bC5vcmc7IElkbyBZYXJpdjsgQXJpayBOZW10c292OyBBZHJpYW5vIE1lbGlzOyBMdWNhIEZyZXNp
DQpTdWJqZWN0OiBSZTogd2wxMnh4IG92ZXIgc3BpIHdpdGggbm8gc2NhbiByZXN1bHRzDQoNCk9u
IFdlZCwgMjAxMi0xMi0xMiBhdCAwODoxOCArMDAwMCwgQWxiZXJ0byBHYXJhdSB3cm90ZToNCj4g
SGkgTHVjYSwNCj4gDQo+IHllcyB0aGF0J3MgYSBncmVhdCBuZXdzIGZvciB1cy4NCj4gDQo+IFll
c3RlcmRheSBhZnRlcm5vb24gd2l0aCAiaXdsaXN0IHdsYW4wIHNjYW4iIChub3QgIml3IiB5ZXQp
IEkgc2F3IGFsc28gDQo+IG90aGVyIHR3byBuZXR3b3JrcyBlc3NpZHMgKGZvciBhIHNtYWxsIGFt
b3VudCBvZiBzZWNvbmRzIG9ubHkpIHdoaWNoIEknbSBpbnRlcmVzdGVkIGluLg0KPiANCj4gVGhp
cyBzZWVtcyB0byBzYXkgdGhhdCB0aGUgbmV0d29ya3MgSSB3b3VsZCBsaWtlIHRvIGNvbm5lY3Qg
dG8gYXJlIGF2YWlsYWJsZSB0byB0aGUgd2wxMjcxLiANCj4gDQo+IEluIHRoZSBtZWFudGltZSB0
aGF0IEknbSBjcm9zcyBidWlsZGluZyAiaXciLCBJIHdvdWxkIGxpa2UgdG8gY29ubmVjdCB0byBh
biB1bnByb3RlY3RlZCBuZXR3b3JrIGNhbGxlZCAiSW5wZWNvLXRlc3QiKG5vIFdFUCwgbm8gV1BB
KS4NCj4gDQo+IFdvdWxkIHlvdSBiZSBzbyBraW5kIHRvIHN1Z2dlc3QgbWUgdGhlIHNlcmllIG9m
IGNvbW1hbmRzIEkgaGF2ZSB0byANCj4gbGF1bmNoIHRvIGNvbm5lY3QgdG8gdGhpcyBuZXR3b3Jr
ID8gSSB3b3VsZCBsaWtlIHRoZW4gdG8gcGluZyBvdXRzaWRlIG9mIGNvdXJzZS4NCg0KaWZjb25m
aWcgd2xhbjAgdXANCml3IHdsYW4wIGNvbm5lY3QgLXcgSW5wZWNvLXRlc3QNCg0KVGhlbiwgdG8g
cGluZywgeW91IG5lZWQgdG8gc2V0IHlvdXIgSVAgYWRkcmVzcywgZWl0aGVyIHdpdGggYSBESENQ
IGNsaWVudCBvciBzdGF0aWNhbGx5Lg0KDQo+IEkgaGF2ZSBnb3QgYWxzbyBvbmUgdGVjaG5pY2Fs
IHF1ZXN0aW9uLiBJbiB0aGUgc291cmNlIGNvZGUgb2YgdGhlIA0KPiBkcml2ZXIgSSBoYXZlIGZv
dW5kIGZvciB3bDEyNzFfc3BpLmMgaW4ga2VybmVsDQo+IDIuNi4zNyBJIGhhdmUgaW50ZXJydXB0
cyBzZXQgaW4gUklTSU5HX0VER0Ugd2hpbGUgdGhlIGRhdGFzaGVldCAoaW4gYXR0YWNobWVudDog
NC4xLjE4IGF0IHBhZyAxOSkgdGFsa3MgYWJvdXQgRkFMTElOR19FREdFLg0KPiBXaGljaCBpcyB0
aGUgcmlnaHQgc2V0dGluZyB0byBiZSBhcHBsaWVkIGFuZCB3aHkgPw0KDQpXZSBjdXJyZW50bHkg
dXNlIGxldmVsIGFuZCBoaWdoIHdpdGggb25lIHNob3QgYXMgZGVmYXVsdC4gIFRoZXJlIGFyZSBz
b21lIHBsYXRmb3JtcyB0aGF0IHVzZSByaXNpbmcsIHRob3VnaCwgc28gd2UgaGF2ZSBhIHBsYXRm
b3JtX3F1aXJrIHZhbHVlIHRoYXQgZW5hYmxlcyB1cyB0byB0b2dnbGUgdGhhdC4NCg0KWW91IGNv
dWxkIGJlIG1pc3Npbmcgc29tZSBpbnRlcnJ1cHRzIGJ5IHVzaW5nIHJpc2luZyBhbmQgdGhpcyBj
b3VsZCBiZSB0aGUgcmVhc29uIHdoeSB5b3UgZG9uJ3Qgc2VlIHRoZSBTU0lEcyBjb25zaXN0ZW50
bHkuDQoNCi0tDQpMdWNhLg0KDQoNCg==