added esp8266 support

[konsgn]

It seems to work for the esp8266 as well as the teensy2.0, don't have any standard arduinos to test, but it is basically pulled from a working adafruit library (st7735 driver).

[Rotzbua]

This library already works with the esp8266 compiled with the Adruino IDE plugin https://github.com/esp8266/Arduino .
I think your changes were only needed if you do not use the Arduino IDE. So I would refuse this pr, because the main target is the Arduino IDE.
But should I add a link to your patch in the readme, for ppl who do not use the IDE? @konsgn

[konsgn]

Yup, Sorry about that, I think I was having a slightly different error that was causing it not to work for esp8266.

Science!

---

From: Rotzbua notifications@github.com
Sent: Wednesday, August 31, 2016 11:56:28 AM
To: miguelbalboa/rfid
Cc: konsgn; Mention
Subject: Re: [miguelbalboa/rfid] added esp8266 support (#235)

This library already works with the esp8266 compiled with the Adruino IDE plugin https://github.com/esp8266/Arduino .
I think your changes were only needed if you do not use the Arduino IDE. So I would refuse this pr, because the main target is the Arduino IDE.
But should I add a link to your patch in the readme, for ppl who do not use the IDE? @konsgnhttps://github.com/konsgn

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com//pull/235#issuecomment-243810828, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE-UYrGTp3NKHwZwk8dOswPPWs9pdypqks5qlaQsgaJpZM4JvoCD.

[konsgn]

Don't bother with the patch, Let my try the original again and figure it out.

Science!

---

From: Konstantin Avdashchenko konsgn@hotmail.com
Sent: Wednesday, August 31, 2016 12:24:44 PM
To: miguelbalboa/rfid; miguelbalboa/rfid
Cc: Mention
Subject: Re: [miguelbalboa/rfid] added esp8266 support (#235)

Yup, Sorry about that, I think I was having a slightly different error that was causing it not to work for esp8266.

Science!

---

From: Rotzbua notifications@github.com
Sent: Wednesday, August 31, 2016 11:56:28 AM
To: miguelbalboa/rfid
Cc: konsgn; Mention
Subject: Re: [miguelbalboa/rfid] added esp8266 support (#235)

This library already works with the esp8266 compiled with the Adruino IDE plugin https://github.com/esp8266/Arduino .
I think your changes were only needed if you do not use the Arduino IDE. So I would refuse this pr, because the main target is the Arduino IDE.
But should I add a link to your patch in the readme, for ppl who do not use the IDE? @konsgnhttps://github.com/konsgn

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com//pull/235#issuecomment-243810828, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE-UYrGTp3NKHwZwk8dOswPPWs9pdypqks5qlaQsgaJpZM4JvoCD.

[mlinaje]

by the way and due to the relation between esp8266 and esp32, I must say the library works with esp32 too: https://www.youtube.com/watch?v=j2vT5eBpp4Q