Add Sonoff TH10/16 #476
Add Sonoff TH10/16 #476
Conversation
|
Will require the board adding to the CI system, both in this repo (doxy yaml), and ci-arduino |
| @@ -68,6 +68,7 @@ monitor_filters = esp32_exception_decoder, time | |||
| [common:esp8266] | |||
| platform = espressif8266 | |||
| lib_ignore = WiFiNINA, Adafruit TinyUSB Library | |||
| board_build.filesystem = littlefs | |||
There was a problem hiding this comment.
I am curious - why is this flag required for the sonoff_th but not for the huzzah ESP8266?
There was a problem hiding this comment.
I think maybe it is required for both. I couldn't push the filesystem image (platformIO) for the ESP8266 huzzah without this, I kept forgetting as it was part of my stash with secrets data file. Maybe I've got something configured wrong, but I think it was defaulting to spiffs rather than littlefs. I did the initial sonoff work just before the huzzah arrived later in the day and I was then able to get the secrets across and see debug with both sonoff + huzzah and moved the line to the common:esp8266 definition instead of the sonoff one.
I'll try to check on a fresh machine at some point soon.
There was a problem hiding this comment.
I couldn't push the filesystem image (platformIO) for the ESP8266 huzzah without this, I kept forgetting as it was part of my stash with secrets data file. Maybe I've got something configured wrong, but I think it was defaulting to spiffs rather than littlefs
Interesting! Could you try in the near the board define, USE_LITTLEFS too?
| #define BOARD_ID "sonoff-th" | ||
| #define USE_LITTLEFS | ||
| #define USE_STATUS_LED | ||
| #define STATUS_LED_PIN 13 // pull down |
There was a problem hiding this comment.
Is this pin pulled down by default?
| @@ -82,7 +82,7 @@ void Wippersnapper_DigitalGPIO::initDigitalPin( | |||
| #endif | |||
|
|
|||
| // Initialize LOW | |||
| #if defined(ARDUINO_ESP8266_ADAFRUIT_HUZZAH) | |||
| #if defined(ARDUINO_ESP8266_ADAFRUIT_HUZZAH) || defined(ESP8266_SONOFF_TH) | |||
There was a problem hiding this comment.
This applies to every ifdef like this, would we want to detect the ESP8266 arch instead of individual boards? Both boards we're using here (and elsewhere) seem to take the same pinout/pd resistor
This adds the initial provisional ALPHA quality release for the Sonoff TH10/16 Relays, using the ESP8266.