Merge branch 'main' into dev-container-nightly

networkfusion · web-flow · commit ed07ab33dce6 · 2025-11-07T12:33:00.000Z
diff --git a/.github/workflows/devcontainer-all.yaml b/.github/workflows/devcontainer-all.yaml
@@ -26,7 +26,7 @@ jobs:
     steps:
 
     - name: Checkout Repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
       
     - name: Free Disk Space (Ubuntu)
       uses: jlumbroso/free-disk-space@main
@@ -55,7 +55,7 @@ jobs:
         password: ${{ secrets.CONTAINER_BUILD_TOKEN }}
 
     - name: Build and Push Docker Image
-      uses: docker/build-push-action@v5
+      uses: docker/build-push-action@v6
       with:
         file: ${{ env.CONTAINER_SRC_FILE }}
         push: true # Will only build if this is not here
diff --git a/.github/workflows/devcontainer-azurertos.yaml b/.github/workflows/devcontainer-azurertos.yaml
@@ -26,7 +26,7 @@ jobs:
     steps:
     
     - name: Checkout Repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
 
     - name: Free Disk Space (Ubuntu)
       uses: jlumbroso/free-disk-space@main
@@ -55,7 +55,7 @@ jobs:
         password: ${{ secrets.CONTAINER_BUILD_TOKEN }}
 
     - name: Build and Push Docker Image
-      uses: docker/build-push-action@v5
+      uses: docker/build-push-action@v6
       with:
         file: ${{ env.CONTAINER_SRC_FILE }}
         push: true # Will only build if this is not here
diff --git a/.github/workflows/devcontainer-chibios.yaml b/.github/workflows/devcontainer-chibios.yaml
@@ -26,7 +26,7 @@ jobs:
     steps:
 
     - name: Checkout Repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
 
     - name: Free Disk Space (Ubuntu)
       uses: jlumbroso/free-disk-space@main
@@ -55,7 +55,7 @@ jobs:
         password: ${{ secrets.CONTAINER_BUILD_TOKEN }}
 
     - name: Build and Push Docker Image
-      uses: docker/build-push-action@v5
+      uses: docker/build-push-action@v6
       with:
         file: ${{ env.CONTAINER_SRC_FILE }}
         push: true # Will only build if this is not here
diff --git a/.github/workflows/devcontainer-esp32.yml b/.github/workflows/devcontainer-esp32.yml
@@ -26,7 +26,7 @@ jobs:
     steps:
 
     - name: Checkout Repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
 
     - name: Free Disk Space (Ubuntu)
       uses: jlumbroso/free-disk-space@main
@@ -55,7 +55,7 @@ jobs:
         password: ${{ secrets.CONTAINER_BUILD_TOKEN }}
 
     - name: Build and Push Docker Image
-      uses: docker/build-push-action@v5
+      uses: docker/build-push-action@v6
       with:
         file: ${{ env.CONTAINER_SRC_FILE }}
         push: true # Will only build if this is not here
diff --git a/.github/workflows/devcontainer-freertos-nxp.yaml b/.github/workflows/devcontainer-freertos-nxp.yaml
@@ -26,7 +26,7 @@ jobs:
     steps:
 
     - name: Checkout Repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
     
     - name: Free Disk Space (Ubuntu)
       uses: jlumbroso/free-disk-space@main
@@ -55,7 +55,7 @@ jobs:
         password: ${{ secrets.CONTAINER_BUILD_TOKEN }}
 
     - name: Build and Push Docker Image
-      uses: docker/build-push-action@v5
+      uses: docker/build-push-action@v6
       with:
         file: ${{ env.CONTAINER_SRC_FILE }}
         push: true # Will only build if this is not here
diff --git a/.github/workflows/devcontainer-smoketest.yaml b/.github/workflows/devcontainer-smoketest.yaml
@@ -36,7 +36,7 @@ jobs:
       contents: read
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v5
         with:
           submodules: true
 
diff --git a/.github/workflows/devcontainer-ti.yaml b/.github/workflows/devcontainer-ti.yaml
@@ -26,7 +26,7 @@ jobs:
     steps:
 
     - name: Checkout Repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v5
 
     - name: Free Disk Space (Ubuntu)
       uses: jlumbroso/free-disk-space@main
@@ -55,7 +55,7 @@ jobs:
         password: ${{ secrets.CONTAINER_BUILD_TOKEN }}
 
     - name: Build and Push Docker Image
-      uses: docker/build-push-action@v5
+      uses: docker/build-push-action@v6
       with:
         file: ${{ env.CONTAINER_SRC_FILE }}
         push: true # Will only build if this is not here
diff --git a/targets/AzureRTOS/Maxim/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp b/targets/AzureRTOS/Maxim/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp
@@ -38,6 +38,8 @@ struct gpio_input_state : public HAL_DblLinkedNode<gpio_input_state>
     void *param;
     // Expected state for debounce handler
     GPIO_PinState expected;
+    // Current pin state
+    GPIO_PinState current;
     // True if waiting for debounce timer to complete
     bool waitingDebounce;
 };
@@ -76,14 +78,15 @@ static void DebounceTimerCallback(uint32_t id)
     // get current pin state
     bool actual = HAL_GPIO_ReadPin(port, GPIO_PIN(pState->pinNumber));
 
-    if (actual == pState->expected)
+    if (actual == pState->current)
     {
         pState->isrPtr(pState->pinNumber, actual, pState->param);
+
         if (pState->mode == GPIO_INT_EDGE_BOTH)
         {
             // both edges
-            // update expected state
-            pState->expected = (GPIO_PinState)(pState->expected ^ GPIO_PIN_SET);
+            // update current state
+            pState->current = (GPIO_PinState)(pState->current ^ GPIO_PIN_SET);
         }
     }
 
@@ -123,10 +126,14 @@ void HAL_GPIO_EXTI_Callback(uint16_t gpioPin)
             {
                 TX_RESTORE
 
-                pGpio->isrPtr(
-                    pGpio->pinNumber,
-                    HAL_GPIO_ReadPin(GPIO_PORT(pGpio->pinNumber), GPIO_PIN(pGpio->pinNumber)),
-                    pGpio->param);
+                bool level = HAL_GPIO_ReadPin(GPIO_PORT(pGpio->pinNumber), GPIO_PIN(pGpio->pinNumber));
+
+                if (level != pGpio->current)
+                {
+                    pGpio->current = level;
+
+                    pGpio->isrPtr(pGpio->pinNumber, level, pGpio->param);
+                }
 
                 TX_DISABLE
             }
@@ -167,6 +174,7 @@ gpio_input_state *AllocateGpioInputState(GPIO_PIN pinNumber)
         {
             memset(ptr, 0, sizeof(gpio_input_state));
             ptr->pinNumber = pinNumber;
+            ptr->current = HAL_GPIO_ReadPin(GPIO_PORT(pGpio->pinNumber), GPIO_PIN(pGpio->pinNumber));
 
             tx_timer_create(
                 &ptr->debounceTimer,
@@ -490,7 +498,7 @@ void CPU_GPIO_DisablePin(GPIO_PIN pinNumber, GpioPinDriveMode driveMode, uint32_
 
     GPIO_TypeDef *port = GPIO_PORT(pinNumber);
     uint32_t pin = GPIO_PIN(pinNumber);
-    
+
     GPIO_InitTypeDef gpio_init_structure;
     gpio_init_structure.Pin = pin;
 
diff --git a/targets/AzureRTOS/ST/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp b/targets/AzureRTOS/ST/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp
@@ -79,6 +79,12 @@ static void DebounceTimerCallback(ULONG pinState)
     {
         // call ISR
         pState->isrPtr(pState->pinNumber, actual, pState->param);
+
+        if (pState->mode == GPIO_INT_EDGE_BOTH)
+        {
+            // update expected state
+            pState->expected ^= 1;
+        }
     }
 
     // reset flag
diff --git a/targets/ChibiOS/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp b/targets/ChibiOS/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp
@@ -23,6 +23,7 @@ struct gpio_input_state : public HAL_DblLinkedNode<gpio_input_state>
     uint8_t mode;                          // Interrupt mode
     void *param;                           // Param to user isr call
     bool expected;                         // Expected state for debounce handler
+    bool current;                          // Current pin state
     bool waitingDebounce;                  // True if waiting for debounce timer to complete
 };
 
@@ -53,10 +54,11 @@ static void DebounceTimerCallback(virtual_timer_t *vtp, void *arg)
     if (actual == pState->expected)
     {
         pState->isrPtr(pState->pinNumber, actual, pState->param);
+
         if (pState->mode == GPIO_INT_EDGE_BOTH)
         {
-            // both edges
-            pState->expected ^= 1; // update expected state
+            // update expected state
+            pState->expected ^= 1;
         }
     }
 
@@ -92,9 +94,14 @@ static void GpioEventCallback(void *arg)
         // get IoLine from pin number
         ioline_t ioLine = GetIoLine(pGpio->pinNumber);
 
-        chSysUnlockFromISR();
-        pGpio->isrPtr(pGpio->pinNumber, palReadLine(ioLine), pGpio->param);
-        chSysLockFromISR();
+        bool level = palReadLine(ioLine);
+
+        if (level != pGpio->current)
+        {
+            pGpio->current = level;
+
+            pGpio->isrPtr(pGpio->pinNumber, level, pGpio->param);
+        }
     }
 
     chSysUnlockFromISR();
@@ -131,6 +138,7 @@ gpio_input_state *AllocateGpioInputState(GPIO_PIN pinNumber)
         {
             memset(ptr, 0, sizeof(gpio_input_state));
             ptr->pinNumber = pinNumber;
+            ptr->current = palReadLine(GetIoLine(pinNumber));
 
             chVTObjectInit(&ptr->debounceTimer);
 
diff --git a/targets/ESP32/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp b/targets/ESP32/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp
@@ -141,7 +141,7 @@ static void Esp_Gpio_DebounceHandler(TimerHandle_t timer)
     if (state != NULL)
     {
         // get current pin state
-        bool actual = CPU_GPIO_GetPinState(state->pinNumber);
+        GpioPinValue actual = CPU_GPIO_GetPinState(state->pinNumber);
 
         if (actual == state->expected)
         {
@@ -317,9 +317,6 @@ static void gpio_isr(void *arg)
         return;
     }
 
-    // get current pin state
-    bool actual = CPU_GPIO_GetPinState(state->pinNumber);
-
     if (state->debounceMs > 0)
     {
         // flag waiting for debounce timeout
@@ -338,9 +335,15 @@ static void gpio_isr(void *arg)
     }
     else
     {
-        state->current = actual;
-        // just fire event
-        state->isrPtr(state->pinNumber, actual, state->param);
+        // get current pin state
+        GpioPinValue actual = gpio_get_level((gpio_num_t)state->pinNumber) ? GpioPinValue_High : GpioPinValue_Low;
+
+        if (actual != state->current)
+        {
+            state->current = actual;
+
+            state->isrPtr(state->pinNumber, actual, state->param);
+        }
     }
 
     NATIVE_INTERRUPT_END
@@ -361,6 +364,8 @@ bool CPU_GPIO_EnableInputPin(
     if (driveMode >= (int)PinMode_Output)
         return false;
 
+    gpio_reset_pin((gpio_num_t)pinNumber);
+
     // Set as Input GPIO_INT_EDGE intEdge, GPIO_RESISTOR ResistorState
     if (!CPU_GPIO_SetDriveMode(pinNumber, driveMode))
     {
@@ -456,6 +461,8 @@ bool CPU_GPIO_EnableOutputPin(GPIO_PIN pinNumber, GpioPinValue InitialState, Pin
     // If this is currently an input pin then clean up
     DeleteInputState(pinNumber);
 
+    gpio_reset_pin((gpio_num_t)pinNumber);
+
     if (CPU_GPIO_SetDriveMode(pinNumber, driveMode) == false)
     {
         return false;
diff --git a/targets/FreeRTOS/NXP/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp b/targets/FreeRTOS/NXP/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp
@@ -28,6 +28,7 @@ struct gpio_input_state
     uint8_t mode;                          // Interrupt mode
     void *param;                           // Param to user isr call
     bool expected;                         // Expected state for debounce handler
+    bool current;                          // Current pin state
     bool waitingDebounce;                  // True if waiting for debounce timer to complete
 };
 
@@ -60,8 +61,9 @@ void Gpio_DebounceHandler(TimerHandle_t xTimer)
         pState->isrPtr(pState->pinNumber, actual, pState->param);
 
         if (pState->mode == GPIO_INT_EDGE_BOTH)
-        {                          // both edges
-            pState->expected ^= 1; // update expected state
+        {
+            // update expected state
+            pState->expected ^= 1;
         }
     }
 
@@ -121,13 +123,19 @@ void GPIO_Main_IRQHandler(int portIndex, GPIO_Type *portBase)
                         }
                         else
                         {
-                            GpioPinValue PinState =
-                                (GpioPinValue)GPIO_PinRead(GPIO_BASE(pState->pinNumber), GPIO_PIN(pState->pinNumber));
-                            pState->isrPtr(pState->pinNumber, PinState, pState->param);
+                            bool pinState =
+                                GPIO_PinRead(GPIO_BASE(pState->pinNumber), GPIO_PIN(pState->pinNumber)) != 0;
+
+                            if (pinState != pState->current)
+                            {
+                                pState->current = pinState;
+
+                                pState->isrPtr(pState->pinNumber, pinState, pState->param);
+                            }
                         }
                     }
                 } // if pin setup in nanoFramework
-            }     // if interrupt
+            } // if interrupt
 
             intPins >>= 1;
             bitNumber++;
@@ -237,6 +245,7 @@ gpio_input_state *AllocateGpioInputState(GPIO_PIN pinNumber)
         pState = (gpio_input_state *)platform_malloc(sizeof(gpio_input_state));
         memset(pState, 0, sizeof(gpio_input_state));
         pState->pinNumber = pinNumber;
+        pState->current = GPIO_PinRead(GPIO_BASE(pinNumber), GPIO_PIN(pinNumber));
         (*inputStates)[bit] = pState;
     }
     return pState;
diff --git a/targets/TI_SimpleLink/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp b/targets/TI_SimpleLink/_nanoCLR/System.Device.Gpio/cpu_gpio.cpp

Original file line number	Diff line number	Diff line change
`@@ -79,6 +79,12 @@ static void DebounceTimerCallback(ULONG pinState)`
`79`	`79`	`{`
`80`	`80`	`// call ISR`
`81`	`81`	`pState->isrPtr(pState->pinNumber, actual, pState->param);`
	`82`	`+`
	`83`	`+ if (pState->mode == GPIO_INT_EDGE_BOTH)`
	`84`	`+ {`
	`85`	`+ // update expected state`
	`86`	`+ pState->expected ^= 1;`
	`87`	`+ }`
`82`	`88`	`}`
`83`	`89`
`84`	`90`	`// reset flag`