drivers/periph/gpio: make gpio_write() take a bool

benpicco · benpicco · commit 4627f66caad1 · 2024-10-22T16:39:48.000+02:00
diff --git a/cpu/atmega_common/periph/gpio.c b/cpu/atmega_common/periph/gpio.c
@@ -179,7 +179,7 @@ void gpio_toggle(gpio_t pin)
     }
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         gpio_set(pin);
diff --git a/cpu/atxmega/periph/gpio.c b/cpu/atxmega/periph/gpio.c
@@ -306,7 +306,7 @@ void gpio_toggle(gpio_t pin)
     }
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     DEBUG("gpio_write pin = 0x%04x, value = 0x%02x \n", pin, value);
 
diff --git a/cpu/cc2538/periph/gpio.c b/cpu/cc2538/periph/gpio.c
@@ -142,7 +142,7 @@ void gpio_toggle(gpio_t pin)
     gpio(pin)->DATA ^= _pin_mask(pin);
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         gpio(pin)->DATA |= _pin_mask(pin);
diff --git a/cpu/cc26xx_cc13xx/periph/gpio.c b/cpu/cc26xx_cc13xx/periph/gpio.c
@@ -81,7 +81,7 @@ void gpio_toggle(gpio_t pin)
     GPIO->DOUTTGL = (1 << pin);
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         GPIO->DOUTSET = (1 << pin);
diff --git a/cpu/efm32/periph/gpio.c b/cpu/efm32/periph/gpio.c
@@ -93,7 +93,7 @@ void gpio_toggle(gpio_t pin)
     GPIO_PinOutToggle(_port_num(pin), _pin_num(pin));
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         GPIO_PinOutSet(_port_num(pin), _pin_num(pin));
diff --git a/cpu/esp32/periph/gpio.c b/cpu/esp32/periph/gpio.c
@@ -351,7 +351,7 @@ int gpio_read(gpio_t pin)
     return value;
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     DEBUG("%s gpio=%u val=%d\n", __func__, pin, value);
     assert(pin < GPIO_PIN_NUMOF);
@@ -412,7 +412,7 @@ int gpio_read(gpio_t pin)
     return value;
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     DEBUG("%s gpio=%u val=%d\n", __func__, pin, value);
     assert(pin < GPIO_PIN_NUMOF);
diff --git a/cpu/esp8266/periph/gpio.c b/cpu/esp8266/periph/gpio.c
@@ -249,7 +249,7 @@ int gpio_read (gpio_t pin)
     return (GPIO.IN & BIT(pin)) ? 1 : 0;
 }
 
-void gpio_write (gpio_t pin, int value)
+void gpio_write (gpio_t pin, bool value)
 {
     DEBUG("%s: %d %d\n", __func__, pin, value);
 
diff --git a/cpu/fe310/periph/gpio.c b/cpu/fe310/periph/gpio.c
@@ -111,7 +111,7 @@ void gpio_toggle(gpio_t pin)
                        __ATOMIC_RELAXED);
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         _set_pin_reg(GPIO_OUTPUT_VAL, pin);
diff --git a/cpu/gd32v/periph/gpio.c b/cpu/gd32v/periph/gpio.c
@@ -195,7 +195,7 @@ void gpio_toggle(gpio_t pin)
     }
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         gpio_set(pin);
diff --git a/cpu/kinetis/periph/gpio.c b/cpu/kinetis/periph/gpio.c
@@ -272,7 +272,7 @@ void gpio_toggle(gpio_t pin)
     gpio(pin)->PTOR = (1 << pin_num(pin));
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         gpio(pin)->PSOR = (1 << pin_num(pin));
diff --git a/cpu/lm4f120/periph/gpio.c b/cpu/lm4f120/periph/gpio.c
@@ -157,7 +157,7 @@ void gpio_toggle(gpio_t pin)
     }
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         gpio_set(pin);
diff --git a/cpu/lpc1768/periph/gpio.c b/cpu/lpc1768/periph/gpio.c
@@ -120,7 +120,7 @@ void gpio_toggle(gpio_t pin)
     base->FIOPIN ^= (1 << _pin(pin));
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     LPC_GPIO_TypeDef *base = _base(pin);
 
diff --git a/cpu/lpc23xx/periph/gpio.c b/cpu/lpc23xx/periph/gpio.c
@@ -133,13 +133,13 @@ void gpio_toggle(gpio_t dev)
     }
 }
 
-void gpio_write(gpio_t dev, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
-        gpio_set(dev);
+        gpio_set(pin);
     }
     else {
-        gpio_clear(dev);
+        gpio_clear(pin);
     }
 }
 
diff --git a/cpu/msp430/periph/gpio.c b/cpu/msp430/periph/gpio.c
@@ -138,7 +138,7 @@ void gpio_toggle(gpio_t pin)
     _port(pin)->OD ^= _pin_mask(pin);
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         _port(pin)->OD |= _pin_mask(pin);
diff --git a/cpu/native/periph/gpio_linux.c b/cpu/native/periph/gpio_linux.c
@@ -210,7 +210,7 @@ void gpio_toggle(gpio_t pin)
     _set(pin, !gpio_read(pin));
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     _set(pin, value);
 }
diff --git a/cpu/native/periph/gpio_mock.c b/cpu/native/periph/gpio_mock.c
@@ -92,7 +92,7 @@ __attribute__((weak)) void gpio_toggle(gpio_t pin) {
   }
 }
 
-__attribute__((weak)) void gpio_write(gpio_t pin, int value) {
+__attribute__((weak)) void gpio_write(gpio_t pin, bool value) {
   if (pin) {
     pin->value = value;
   }
diff --git a/cpu/nrf5x_common/periph/gpio.c b/cpu/nrf5x_common/periph/gpio.c
@@ -151,7 +151,7 @@ void gpio_toggle(gpio_t pin)
     port(pin)->OUT ^= (1 << pin_num(pin));
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         port(pin)->OUTSET = (1 << pin_num(pin));
diff --git a/cpu/qn908x/periph/gpio.c b/cpu/qn908x/periph/gpio.c
@@ -232,7 +232,7 @@ void gpio_toggle(gpio_t pin)
     base->OUTENSET = out_clr & dataout;
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         gpio_set(pin);
diff --git a/cpu/rpx0xx/periph/gpio.c b/cpu/rpx0xx/periph/gpio.c
@@ -131,7 +131,7 @@ void gpio_toggle(gpio_t pin)
     SIO->GPIO_OUT_XOR = 1LU << pin;
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         gpio_set(pin);
diff --git a/cpu/sam0_common/periph/gpio.c b/cpu/sam0_common/periph/gpio.c
@@ -230,7 +230,7 @@ void gpio_toggle(gpio_t pin)
     _port_iobus(pin)->OUTTGL.reg = _pin_mask(pin);
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         _port_iobus(pin)->OUTSET.reg = _pin_mask(pin);
diff --git a/cpu/sam3/periph/gpio.c b/cpu/sam3/periph/gpio.c
@@ -225,7 +225,7 @@ void gpio_toggle(gpio_t pin)
     }
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         _port(pin)->PIO_SODR = (1 << _pin_num(pin));
diff --git a/cpu/stm32/periph/gpio_all.c b/cpu/stm32/periph/gpio_all.c
@@ -227,7 +227,7 @@ void gpio_toggle(gpio_t pin)
     }
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         gpio_set(pin);
diff --git a/cpu/stm32/periph/gpio_f1.c b/cpu/stm32/periph/gpio_f1.c
@@ -180,7 +180,7 @@ void gpio_toggle(gpio_t pin)
     }
 }
 
-void gpio_write(gpio_t pin, int value)
+void gpio_write(gpio_t pin, bool value)
 {
     if (value) {
         _port(pin)->BSRR = (1 << _pin_num(pin));
diff --git a/drivers/include/periph/gpio.h b/drivers/include/periph/gpio.h
@@ -76,6 +76,7 @@
 #define PERIPH_GPIO_H
 
 #include <limits.h>
+#include <stdbool.h>
 
 #include "periph_cpu.h"
 #include "periph_conf.h"
@@ -258,7 +259,7 @@ void gpio_toggle(gpio_t pin);
  * @param[in] pin       the pin to set
  * @param[in] value     value to set the pin to, 0 for LOW, HIGH otherwise
  */
-void gpio_write(gpio_t pin, int value);
+void gpio_write(gpio_t pin, bool value);
 
 /**
  * @brief   Test if a GPIO pin is equal to another GPIO pin

Original file line number	Diff line number	Diff line change
`@@ -179,7 +179,7 @@ void gpio_toggle(gpio_t pin)`
`179`	`179`	`}`
`180`	`180`	`}`
`181`	`181`
`182`		`-void gpio_write(gpio_t pin, int value)`
	`182`	`+void gpio_write(gpio_t pin, bool value)`
`183`	`183`	`{`
`184`	`184`	`if (value) {`
`185`	`185`	`gpio_set(pin);`
Original file line number	Diff line number	Diff line change
`@@ -306,7 +306,7 @@ void gpio_toggle(gpio_t pin)`
`306`	`306`	`}`
`307`	`307`	`}`
`308`	`308`
`309`		`-void gpio_write(gpio_t pin, int value)`
	`309`	`+void gpio_write(gpio_t pin, bool value)`
`310`	`310`	`{`
`311`	`311`	`DEBUG("gpio_write pin = 0x%04x, value = 0x%02x \n", pin, value);`
`312`	`312`
Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ void gpio_toggle(gpio_t pin)`
`142`	`142`	`gpio(pin)->DATA ^= _pin_mask(pin);`
`143`	`143`	`}`
`144`	`144`
`145`		`-void gpio_write(gpio_t pin, int value)`
	`145`	`+void gpio_write(gpio_t pin, bool value)`
`146`	`146`	`{`
`147`	`147`	`if (value) {`
`148`	`148`	`gpio(pin)->DATA \|= _pin_mask(pin);`
Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ void gpio_toggle(gpio_t pin)`
`81`	`81`	`GPIO->DOUTTGL = (1 << pin);`
`82`	`82`	`}`
`83`	`83`
`84`		`-void gpio_write(gpio_t pin, int value)`
	`84`	`+void gpio_write(gpio_t pin, bool value)`
`85`	`85`	`{`
`86`	`86`	`if (value) {`
`87`	`87`	`GPIO->DOUTSET = (1 << pin);`
Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ void gpio_toggle(gpio_t pin)`
`93`	`93`	`GPIO_PinOutToggle(_port_num(pin), _pin_num(pin));`
`94`	`94`	`}`
`95`	`95`
`96`		`-void gpio_write(gpio_t pin, int value)`
	`96`	`+void gpio_write(gpio_t pin, bool value)`
`97`	`97`	`{`
`98`	`98`	`if (value) {`
`99`	`99`	`GPIO_PinOutSet(_port_num(pin), _pin_num(pin));`
Original file line number	Diff line number	Diff line change
`@@ -351,7 +351,7 @@ int gpio_read(gpio_t pin)`
`351`	`351`	`return value;`
`352`	`352`	`}`
`353`	`353`
`354`		`-void gpio_write(gpio_t pin, int value)`
	`354`	`+void gpio_write(gpio_t pin, bool value)`
`355`	`355`	`{`
`356`	`356`	`DEBUG("%s gpio=%u val=%d\n", __func__, pin, value);`
`357`	`357`	`assert(pin < GPIO_PIN_NUMOF);`
`@@ -412,7 +412,7 @@ int gpio_read(gpio_t pin)`
`412`	`412`	`return value;`
`413`	`413`	`}`
`414`	`414`
`415`		`-void gpio_write(gpio_t pin, int value)`
	`415`	`+void gpio_write(gpio_t pin, bool value)`
`416`	`416`	`{`
`417`	`417`	`DEBUG("%s gpio=%u val=%d\n", __func__, pin, value);`
`418`	`418`	`assert(pin < GPIO_PIN_NUMOF);`
Original file line number	Diff line number	Diff line change
`@@ -249,7 +249,7 @@ int gpio_read (gpio_t pin)`
`249`	`249`	`return (GPIO.IN & BIT(pin)) ? 1 : 0;`
`250`	`250`	`}`
`251`	`251`
`252`		`-void gpio_write (gpio_t pin, int value)`
	`252`	`+void gpio_write (gpio_t pin, bool value)`
`253`	`253`	`{`
`254`	`254`	`DEBUG("%s: %d %d\n", __func__, pin, value);`
`255`	`255`
Original file line number	Diff line number	Diff line change
`@@ -111,7 +111,7 @@ void gpio_toggle(gpio_t pin)`
`111`	`111`	`__ATOMIC_RELAXED);`
`112`	`112`	`}`
`113`	`113`
`114`		`-void gpio_write(gpio_t pin, int value)`
	`114`	`+void gpio_write(gpio_t pin, bool value)`
`115`	`115`	`{`
`116`	`116`	`if (value) {`
`117`	`117`	`_set_pin_reg(GPIO_OUTPUT_VAL, pin);`
Original file line number	Diff line number	Diff line change
`@@ -195,7 +195,7 @@ void gpio_toggle(gpio_t pin)`
`195`	`195`	`}`
`196`	`196`	`}`
`197`	`197`
`198`		`-void gpio_write(gpio_t pin, int value)`
	`198`	`+void gpio_write(gpio_t pin, bool value)`
`199`	`199`	`{`
`200`	`200`	`if (value) {`
`201`	`201`	`gpio_set(pin);`
Original file line number	Diff line number	Diff line change
`@@ -272,7 +272,7 @@ void gpio_toggle(gpio_t pin)`
`272`	`272`	`gpio(pin)->PTOR = (1 << pin_num(pin));`
`273`	`273`	`}`
`274`	`274`
`275`		`-void gpio_write(gpio_t pin, int value)`
	`275`	`+void gpio_write(gpio_t pin, bool value)`
`276`	`276`	`{`
`277`	`277`	`if (value) {`
`278`	`278`	`gpio(pin)->PSOR = (1 << pin_num(pin));`