Moved checks for invalid pin to common-hal/microcontroller/Pin.c

2 files changed, 7 insertions, 12 deletions

diff --git a/ports/nrf/common-hal/busio/SPI.c b/ports/nrf/common-hal/busio/SPI.c
index d7d0e0fe8..1cf074955 100644
--- a/ports/nrf/common-hal/busio/SPI.c
+++ b/ports/nrf/common-hal/busio/SPI.c
@@ -104,12 +104,8 @@ void common_hal_busio_spi_never_reset(busio_spi_obj_t *self) {
             never_reset[i] = true;
 
             never_reset_pin_number(self->clock_pin_number);
-            if ( self->MOSI_pin_number != NO_PIN) {
-                never_reset_pin_number(self->MOSI_pin_number);
-            }
-            if ( self->MISO_pin_number != NO_PIN) {
-                never_reset_pin_number(self->MISO_pin_number);
-            }
+            never_reset_pin_number(self->MOSI_pin_number);
+            never_reset_pin_number(self->MISO_pin_number);
             break;
         }
     }
@@ -208,12 +204,8 @@ void common_hal_busio_spi_deinit(busio_spi_obj_t *self) {
     nrfx_spim_uninit(&self->spim_peripheral->spim);
 
     reset_pin_number(self->clock_pin_number);
-    if ( self->MOSI_pin_number != NO_PIN) {
-        reset_pin_number(self->MOSI_pin_number);
-    }
-    if ( self->MISO_pin_number != NO_PIN) {
-        reset_pin_number(self->MISO_pin_number);
-    }
+    reset_pin_number(self->MOSI_pin_number);
+    reset_pin_number(self->MISO_pin_number);
 }
 
 bool common_hal_busio_spi_configure(busio_spi_obj_t *self, uint32_t baudrate, uint8_t polarity, uint8_t phase, uint8_t bits) {
diff --git a/ports/nrf/common-hal/microcontroller/Pin.c b/ports/nrf/common-hal/microcontroller/Pin.c
index d294c1dd5..bc7ff831a 100644
--- a/ports/nrf/common-hal/microcontroller/Pin.c
+++ b/ports/nrf/common-hal/microcontroller/Pin.c
@@ -123,6 +123,9 @@ void reset_pin_number(uint8_t pin_number) {
 
 
 void never_reset_pin_number(uint8_t pin_number) {
+    if (pin_number == NO_PIN) {
+        return;
+    }
     never_reset_pins[nrf_pin_port(pin_number)] |= 1 << nrf_relative_pin_number(pin_number);
 }