Skip to content
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

New WiFly compatible library to work with Arduino and Ninja Blocks #3

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

ozzieg
Copy link

@ozzieg ozzieg commented Oct 25, 2013

Added NinjaBlockWiFly class to encapsulate and implement the Wi-Fly version of the NinjaBlockEthernet library. It utilizes the WiFlyHQ library for communicating to the netherworld wirelessly.

I have had to update the existing code to reduce the memory footprint by utilizing PSTR, str*_P, and F macros. It allowed me to include other libraries as a result of this reduction. I have also had to change how the data is received (sending/posting end is almost exactly the same) because of how WiFly operates over the Serial port. I have had poll available() and then read one char at a time.

I have also added the libraries into this commit because of the memory reduction modifications. There is a lot of unneeded code inside RCSwitch which takes up valuable Flash space.

I also added a "Simple.ino" example to demonstrate the usage of Wi-Fly and even the RCSwitch library. My original intention of this library was to have a wireless hub for my tri-state RF electrical plugs and this does it really well.

Thanks to all of the original work, otherwise this would not have been easy. I hope you merge it in.

@wolfeidau
Copy link

Looks great, I will get @theojulienne to have a look over it when he gets a chance.

We are a bit snowed under at the moment sorry.

Thanks for your contribution.

@ozzieg
Copy link
Author

ozzieg commented Oct 30, 2013

Thanks!

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