cc3100-dev: To fix before making pull request

blmorris / micropython

The Micro Python project

MIT License

2 stars 1 forks source link

cc3100-dev: To fix before making pull request #1

Open blmorris opened 9 years ago

blmorris commented 9 years ago

Issues that I have found while working with the cc3100 driver:

[ ] Cannot connect to WEP network (can connect to WPA2, nto tested WPA)
[ ] Class initialization accepts pin arguments, but does not use them - connection is hard-coded to SPI(2), Y5, Y4, and Y3
[ ] Change class name from CC31K to CC3K1 (or to something else; I believe the CC3K1 pattern was suggested elsewhere in the MicroPython discussions but can't find it now.)

Feel free to add others.

danicampora commented 9 years ago

WEP doesn't work on the CC3200 either. I think is related to the fact that in only supports WEP in HEX mode. I am going to try to add a fix for that now. WPA/WPA2 and Open networks connect just fine.

blmorris commented 9 years ago

Thanks, that's good to know. All the more reason to support the CC3100 with one driver across both ports; fix it once and it's done.

danicampora commented 9 years ago

@blmorris this: https://github.com/micropython/micropython/commit/92ea99a0fb28ecfdf81f9af9456fa2b423d498cd fixes the problem with WEP. Please note that WEP secured networks use an hex key. The simplelink library needs the key for WEP networks to be passed as binary, not as a string like done for WPA/WPA2. Check the example at the end of this wiki: http://processors.wiki.ti.com/index.php/CC31xx_Programmers_Guide

danicampora commented 9 years ago

True. Now we need to merge the CC3200 WLAN module and the CC3k1 into one.

Sent from my iPhone

On Feb 21, 2015, at 10:24 PM, Bryan Morrissey notifications@github.com wrote:

Thanks, that's good to know. All the more reason to support the CC3100 with one driver across both ports; fix it once and it's done.

— Reply to this email directly or view it on GitHub.

blmorris commented 9 years ago

@danicampora - as you work on merging Nadim's version of the cc3100 driver with yours, I just wanted to call your attention to the changes that were made to modnetwork.[ch] and modusocket.c in addition to the changes to Makefile and qstrdefsport.h, I realized that those changes might have gotten lost in the mass of changes made by committing everything at once.

danicampora commented 9 years ago

Thanks, I will pay attention to that.