wip(pio): Local build for platformIO seems okay

Author: tyeth

src/Wippersnapper_Boards.h

@@ -170,6 +170,9 @@
 #define BOARD_ID "arduino-nesso-n1"
 #define USE_LITTLEFS
 #define USE_STATUS_LED
+#ifdef STATUS_LED_PIN
+#undef STATUS_LED_PIN
+#endif
 #define STATUS_LED_PIN LED_BUILTIN
 #define STATUS_LED_INVERTED
 #define USE_PSRAM ///< Board has PSRAM, use it for dynamic memory allocation

src/components/digitalIO/Wippersnapper_DigitalGPIO.cpp

@@ -66,20 +66,28 @@ void Wippersnapper_DigitalGPIO::initDigitalPin(
       wippersnapper_pin_v1_ConfigurePinRequest_Direction_DIRECTION_OUTPUT) {
 
 #ifdef STATUS_LED_PIN
+#if !defined(ARDUINO_ARDUINO_NESSO_N1)
+//     if (String("D") + pinName == STATUS_LED_PIN.pin)
+// #else
     // deinit status led, use it as a dio component instead
     if (pinName == STATUS_LED_PIN)
       releaseStatusLED();
+#endif
 #endif
     pinMode(pinName, OUTPUT);
 
     WS_DEBUG_PRINT("Configured digital output pin on D");
     WS_DEBUG_PRINTLN(pinName);
 
 // Initialize LOW
-#if defined(ARDUINO_ESP8266_ADAFRUIT_HUZZAH) || defined(STATUS_LED_INVERTED)
-    // The Adafruit Feather ESP8266's built-in LED is reverse wired so setting
-    // the pin LOW will turn the LED on.
-    digitalWrite(STATUS_LED_PIN, !0);
+#if defined(ARDUINO_ESP8266_ADAFRUIT_HUZZAH) // not until we support ExpanderPins || defined(STATUS_LED_INVERTED)
+    if (pinName == STATUS_LED_PIN) {
+      // The Adafruit Feather ESP8266's built-in LED is reverse wired so setting
+      // the pin LOW will turn the LED on.
+      digitalWrite(STATUS_LED_PIN, !0);
+    } else {
+      digitalWrite(pinName, LOW);
+    }
 #else
     pinMode(pinName, OUTPUT);
     digitalWrite(pinName, LOW); // initialize LOW
@@ -155,9 +163,11 @@ void Wippersnapper_DigitalGPIO::deinitDigitalPin(
 
 // if prv. in-use by DIO, release pin back to application
 #ifdef STATUS_LED_PIN
+#if !defined(ARDUINO_ARDUINO_NESSO_N1) // not until we support ExpanderPins
   if (pinName == STATUS_LED_PIN)
     initStatusLED();
 #endif
+#endif
 }
 
 /********************************************************************/
@@ -175,6 +185,40 @@ int Wippersnapper_DigitalGPIO::digitalReadSvc(int pinName) {
   return pinVal;
 }
 
+#if defined(ARDUINO_ARDUINO_NESSO_N1)
+/********************************************************************/
+/*!
+    @brief    High-level digitalRead service impl. which performs a
+                digitalRead.
+    @param    pin
+                The ExpanderPin instance
+    @returns  The pin's value.
+*/
+/********************************************************************/
+int Wippersnapper_DigitalGPIO::digitalReadSvc(ExpanderPin pin) {
+  // Service using arduino `digitalRead`
+  int pinVal = digitalRead(pin);
+  return pinVal;
+}
+
+/*******************************************************************************/
+/*!
+    @brief  Writes a value to a pin.
+    @param  pinName
+                The pin's name.
+    @param  pinValue
+                The pin's value.
+*/
+/*******************************************************************************/
+void Wippersnapper_DigitalGPIO::digitalWriteSvc(ExpanderPin pin, int pinValue) {
+  WS_DEBUG_PRINT("Digital Pin Event: Set ");
+  WS_DEBUG_PRINT(pin.pin);
+  WS_DEBUG_PRINT(" to ");
+  WS_DEBUG_PRINTLN(pinValue);
+  digitalWrite(pin, pinValue);
+}
+#endif
+
 /*******************************************************************************/
 /*!
     @brief  Writes a value to a pin.

src/components/digitalIO/Wippersnapper_DigitalGPIO.h

@@ -50,6 +50,18 @@ class Wippersnapper_DigitalGPIO {
 
   int digitalReadSvc(int pinName);
   void digitalWriteSvc(uint8_t pinName, int pinValue);
+#if defined(ARDUINO_ARDUINO_NESSO_N1)
+  // void
+  // initDigitalPin(wippersnapper_pin_v1_ConfigurePinRequest_Direction direction,
+  //                ExpanderPin pinName, float period,
+  //                wippersnapper_pin_v1_ConfigurePinRequest_Pull pull);
+  // void
+  // deinitDigitalPin(wippersnapper_pin_v1_ConfigurePinRequest_Direction direction,
+  //                  ExpanderPin pinName);
+
+  int digitalReadSvc(ExpanderPin pinName);
+  void digitalWriteSvc(ExpanderPin pinName, int pinValue);
+#endif
   void processDigitalInputs();
 
   digitalInputPin *_digital_input_pins; /*!< Array of gpio pin objects */

src/components/statusLED/Wippersnapper_StatusLED.cpp

@@ -82,6 +82,8 @@ void initStatusLED() {
 // Turn off LED initially
 #if defined(ARDUINO_ESP8266_ADAFRUIT_HUZZAH)
   analogWrite(STATUS_LED_PIN, 255);
+#elif defined(ARDUINO_ARDUINO_NESSO_N1) || defined(STATUS_LED_INVERTED)
+  digitalWrite(STATUS_LED_PIN, HIGH);
 #elif defined(ARDUINO_ARCH_ESP32)
   WS._pwmComponent->attach(STATUS_LED_PIN, LEDC_BASE_FREQ, LEDC_TIMER_12_BIT);
   WS._pwmComponent->writeDutyCycle(STATUS_LED_PIN, 
@@ -205,7 +207,7 @@ void setStatusLEDColor(uint32_t color) {
 #ifdef USE_STATUS_LED
   if (!WS.lockStatusLED)
     return; // status pixel is in-use elsewhere
-#ifdef ARDUINO_ARCH_RP2040
+#if defined(ARDUINO_ARCH_RP2040) || defined(ARDUINO_ARDUINO_NESSO_N1) || defined(STATUS_LED_INVERTED)
   digitalWrite(STATUS_LED_PIN, 
 #if defined(STATUS_LED_INVERTED)
     !(color > 0)
@@ -282,7 +284,7 @@ void setStatusLEDColor(uint32_t color, int brightness) {
   if (!WS.lockStatusLED)
     return;
 
-#ifdef ARDUINO_ARCH_RP2040
+#if defined(ARDUINO_ARCH_RP2040) || defined(ARDUINO_ARDUINO_NESSO_N1)
   digitalWrite(STATUS_LED_PIN,
 #if defined(STATUS_LED_INVERTED)
     !(color > 0)