Skip to content

Adding StandardFirmataWiFi and required utility classes #262

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Jan 9, 2016

Conversation

turkycat
Copy link
Contributor

@turkycat turkycat commented Jan 7, 2016

Current support is for the legacy WiFi library used by the Arduino WiFi shield and the WiFi 101 library used by the WiFi Shield 101 and boards which have this chip built in (such as MKR1000).

…y from Arduino.

compilation problem exists when both WiFi101Stream and WiFiStream exist, as both compile regardless if they are included or not. This is an Arduino IDE problem and must be resolved. Problem is that one includes <WiFi.h> and the other includes <WiFI101.h>, which both define WiFiClass.
…esolving a major compilation issue caused by similarities between the two WiFi libraries.

Removed StandardFirmataWiFi101, StandardFirmataWiFi will now work for either WiFi or WiFi101.
- Pin 5 must be ignored by WiFI101 shield
- broke out the config steps into a separate header (wifiConfig.h) file that shows up as a 2nd tab in the Arduino IDE
- added a couple of macros to the end of wifiConfig.h to help sort out the pin ignore step
- allowing 3 attempts before failure
- changed the comment for the Static IP to clarify it is optional rather than required.
- Added a message to the readme section (maybe better broken out into a separate readme doc?) about needing to use a board with > 35k of Flash. Also added description for which pins cannot be used if using the 101 shield.
… print for library type, cleaned up other messages.
…r for WPA and OPEN security types.

Also successfully tested WEP connection types with the latest fixes applied to the WiFi101 library.
soundanalogous added a commit that referenced this pull request Jan 9, 2016
Adding StandardFirmataWiFi and required utility classes
@soundanalogous soundanalogous merged commit a779755 into firmata:master Jan 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants