zephyrproject-rtos · albertofloyd · Jan 22, 2019 · May 31, 2019 · Jun 30, 2019 · Aug 26, 2019
diff --git a/CODEOWNERS b/CODEOWNERS
@@ -136,7 +136,9 @@
 /drivers/gpio/                            @mnkp @pabigot
 /drivers/gpio/*ht16k33*                   @henrikbrixandersen
 /drivers/gpio/*lmp90xxx*                  @henrikbrixandersen
-/drivers/gpio/*stm32*                     @rsalveti @idlethread
+/drivers/gpio/*stm32*                     @erwango
+/drivers/gpio/*sx1509b*                   @pabigot
+/drivers/gpio/*litex*                     @mateusz-holenko @kgugala @pgielda
 /drivers/hwinfo/                          @alexanderwachter
 /drivers/i2c/*litex*                      @mateusz-holenko @kgugala @pgielda
 /drivers/i2s/i2s_ll_stm32*                @avisconti
@@ -403,6 +405,7 @@
 /tests/crypto/mbedtls/                    @nashif @ceolin
 /tests/drivers/can/                       @alexanderwachter
 /tests/drivers/flash_simulator/           @nvlsianpu
+/tests/drivers/gpio/                      @mnkp @pabigot
 /tests/drivers/hwinfo/                    @alexanderwachter
 /tests/drivers/spi/                       @tbursztyka
 /tests/drivers/uart/uart_async_api/       @Mierunski

diff --git a/boards/arc/em_starterkit/board.dtsi b/boards/arc/em_starterkit/board.dtsi
@@ -61,37 +61,37 @@
 		compatible = "gpio-keys";
 		button0: button_0 {
 			/* gpio flags need validation */
-			gpios = <&gpio0 0 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 0 GPIO_ACTIVE_LOW>;
 			label = "Push button switch 0";
 		};
 		button1: button_1 {
 			/* gpio flags need validation */
-			gpios = <&gpio0 1 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 1 GPIO_ACTIVE_LOW>;
 			label = "Push button switch 1";
 		};
 		button2: button_2 {
 			/* gpio flags need validation */
-			gpios = <&gpio0 2 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 2 GPIO_ACTIVE_LOW>;
 			label = "Push button switch 2";
 		};
 		switch0: switch_0 {
 			/* gpio flags need validation */
-			gpios = <&gpio2 0 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio2 0 GPIO_ACTIVE_LOW>;
 			label = "DIP SW1 - Switch 1";
 		};
 		switch1: switch_1 {
 			/* gpio flags need validation */
-			gpios = <&gpio2 1 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio2 1 GPIO_ACTIVE_LOW>;
 			label = "DIP SW1 - Switch 2";
 		};
 		switch2: switch_2 {
 			/* gpio flags need validation */
-			gpios = <&gpio2 2 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio2 2 GPIO_ACTIVE_LOW>;
 			label = "DIP SW1 - Switch 3";
 		};
 		switch3: switch_3 {
 			/* gpio flags need validation */
-			gpios = <&gpio2 3 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio2 3 GPIO_ACTIVE_LOW>;
 			label = "DIP SW1 - Switch 4";
 		};
 	};

diff --git a/boards/arc/emsdp/board.dtsi b/boards/arc/emsdp/board.dtsi
@@ -56,22 +56,22 @@
 		compatible = "gpio-keys";
 		switch0: switch_0 {
 			/* gpio flags need validation */
-			gpios = <&gpio0 0 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 0 GPIO_ACTIVE_LOW>;
 			label = "DIP SW1 - Switch 1";
 		};
 		switch1: switch_1 {
 			/* gpio flags need validation */
-			gpios = <&gpio0 1 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 1 GPIO_ACTIVE_LOW>;
 			label = "DIP SW1 - Switch 2";
 		};
 		switch2: switch_2 {
 			/* gpio flags need validation */
-			gpios = <&gpio0 2 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 2 GPIO_ACTIVE_LOW>;
 			label = "DIP SW1 - Switch 3";
 		};
 		switch3: switch_3 {
 			/* gpio flags need validation */
-			gpios = <&gpio0 3 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 3 GPIO_ACTIVE_LOW>;
 			label = "DIP SW1 - Switch 4";
 		};
 	};

diff --git a/boards/arm/96b_argonkey/96b_argonkey.dts b/boards/arm/96b_argonkey/96b_argonkey.dts
@@ -21,11 +21,11 @@
 	leds {
 		compatible = "gpio-leds";
 		green_led_0: led_0 {
-			gpios = <&gpioc 13 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpioc 13 GPIO_ACTIVE_HIGH>;
 			label = "USR0 LED";
 		};
 		green_led_1: led_1 {
-			gpios = <&gpiob 2 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiob 2 GPIO_ACTIVE_HIGH>;
 			label = "USR1 LED";
 		};
 	};
@@ -34,7 +34,7 @@
 		compatible = "gpio-keys";
 		user_button: button {
 			label = "User";
-			gpios = <&gpioa 2 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpioa 2 GPIO_ACTIVE_LOW>;
 		};
 	};
 
@@ -62,7 +62,7 @@
 		compatible = "st,lsm6dsl";
 		reg = <1>;
 		spi-max-frequency = <1000000>;
-		irq-gpios = <&gpiob 1 0>;
+		irq-gpios = <&gpiob 1 GPIO_ACTIVE_HIGH>;
 		label = "LSM6DSL_SPI";
 	};
 };
@@ -94,7 +94,7 @@
 		compatible = "st,hts221";
 		reg = <0x5f>;
 		label = "HTS221";
-		drdy-gpios = <&gpioa 2 GPIO_INT_ACTIVE_LOW>;
+		drdy-gpios = <&gpioa 2 GPIO_ACTIVE_HIGH>;
 	};
 
 	lps22hb-press@5d {

diff --git a/boards/arm/96b_avenger96/96b_avenger96.dts b/boards/arm/96b_avenger96/96b_avenger96.dts
@@ -28,15 +28,15 @@
 	leds {
 		compatible = "gpio-leds";
 		green_led_0: led_0 {
-			gpios = <&gpiof 3 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiof 3 GPIO_ACTIVE_HIGH>;
 			label = "USR0 LED";
 		};
 		green_led_1: led_1 {
-			gpios = <&gpiog 0 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiog 0 GPIO_ACTIVE_HIGH>;
 			label = "USR1 LED";
 		};
 		green_led_2: led_2 {
-			gpios = <&gpiog 1 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiog 1 GPIO_ACTIVE_HIGH>;
 			label = "USR2 LED";
 		};
 	};

diff --git a/boards/arm/96b_carbon/96b_carbon.dts b/boards/arm/96b_carbon/96b_carbon.dts
@@ -23,15 +23,15 @@
 	leds {
 		compatible = "gpio-leds";
 		green_led_1: led_1 {
-			gpios = <&gpiod 2 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiod 2 GPIO_ACTIVE_HIGH>;
 			label = "USR1 LED";
 		};
 		green_led_2: led_2 {
-			gpios = <&gpioa 15 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpioa 15 GPIO_ACTIVE_HIGH>;
 			label = "USR2 LED";
 		};
 		bt_blue_led: led_3 {
-			gpios = <&gpioa 15 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpioa 15 GPIO_ACTIVE_HIGH>;
 			label = "BT LED";
 		};
 	};
@@ -40,7 +40,7 @@
 		compatible = "gpio-keys";
 		user_button: button {
 			label = "User";
-			gpios = <&gpiob 2 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpiob 2 GPIO_ACTIVE_LOW>;
 		};
 	};
 
@@ -84,8 +84,8 @@
 	bt-hci@0 {
 		compatible = "zephyr,bt-hci-spi";
 		reg = <0>;
-		reset-gpios = <&gpiob 4 0>;
-		irq-gpios = <&gpiob 1 0>;
+		reset-gpios = <&gpiob 4 (GPIO_ACTIVE_LOW | GPIO_PULL_UP)>;
+		irq-gpios = <&gpiob 1 GPIO_ACTIVE_HIGH>;
 		spi-max-frequency = <2000000>;
 		label = "BT_HCI";
 	};

diff --git a/boards/arm/96b_meerkat96/96b_meerkat96.dts b/boards/arm/96b_meerkat96/96b_meerkat96.dts
@@ -21,19 +21,19 @@
 	leds {
 		compatible = "gpio-leds";
 		green_led_0: led_0 {
-			gpios = <&gpio1 4 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio1 4 GPIO_ACTIVE_LOW>;
 			label = "User LED1";
 		};
 		green_led_1: led_1 {
-			gpios = <&gpio1 5 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio1 5 GPIO_ACTIVE_LOW>;
 			label = "User LED2";
 		};
 		green_led_2: led_2 {
-			gpios = <&gpio1 6 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio1 6 GPIO_ACTIVE_LOW>;
 			label = "User LED3";
 		};
 		green_led_3: led_3 {
-			gpios = <&gpio1 7 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio1 7 GPIO_ACTIVE_LOW>;
 			label = "User LED4";
 		};
 	};

diff --git a/boards/arm/96b_neonkey/96b_neonkey.dts b/boards/arm/96b_neonkey/96b_neonkey.dts
@@ -21,19 +21,19 @@
 	leds {
 		compatible = "gpio-leds";
 		green_led_0: led_0 {
-			gpios = <&gpiob 12 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiob 12 GPIO_ACTIVE_HIGH>;
 			label = "USR0 LED";
 		};
 		green_led_1: led_1 {
-			gpios = <&gpiob 13 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiob 13 GPIO_ACTIVE_HIGH>;
 			label = "USR1 LED";
 		};
 		green_led_2: led_2 {
-			gpios = <&gpiob 14 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiob 14 GPIO_ACTIVE_HIGH>;
 			label = "USR2 LED";
 		};
 		green_led_3: led_3 {
-			gpios = <&gpiob 15 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiob 15 GPIO_ACTIVE_HIGH>;
 			label = "USR3 LED";
 		};
 	};
@@ -42,7 +42,7 @@
 		compatible = "gpio-keys";
 		user_button: button_0 {
 			label = "User";
-			gpios = <&gpiob 2 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpiob 2 GPIO_ACTIVE_LOW>;
 		};
 	};
 

diff --git a/boards/arm/96b_stm32_sensor_mez/96b_stm32_sensor_mez.dts b/boards/arm/96b_stm32_sensor_mez/96b_stm32_sensor_mez.dts
@@ -21,15 +21,15 @@
 	leds {
 		compatible = "gpio-leds";
 		red_led_0: led_0 {
-			gpios = <&gpiod 10 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiod 10 GPIO_ACTIVE_HIGH>;
 			label = "User LD0";
 		};
 		green_led_1: led_1 {
-			gpios = <&gpiod 11 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiod 11 GPIO_ACTIVE_HIGH>;
 			label = "User LD1";
 		};
 		blue_led_2: led_2 {
-			gpios = <&gpiod 12 GPIO_INT_ACTIVE_HIGH>;
+			gpios = <&gpiod 12 GPIO_ACTIVE_HIGH>;
 			label = "User LD2";
 		};
 	};
@@ -38,7 +38,7 @@
 		compatible = "gpio-keys";
 		user_button: button {
 			label = "Key";
-			gpios = <&gpiod 13 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpiod 13 GPIO_ACTIVE_LOW>;
 		};
 	};
 

diff --git a/boards/arm/96b_wistrio/96b_wistrio.dts b/boards/arm/96b_wistrio/96b_wistrio.dts
@@ -22,11 +22,11 @@
 	leds {
 		compatible = "gpio-leds";
 		green_led_0: led_0 {
-			gpios = <&gpioa 12 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpioa 12 GPIO_ACTIVE_LOW>;
 			label = "User LD1";
 		};
 		blue_led_0: led_1 {
-			gpios = <&gpiob 4 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpiob 4 GPIO_ACTIVE_LOW>;
 			label = "User LD2";
 		};
 	};

diff --git a/boards/arm/actinius_icarus/actinius_icarus_common.dts b/boards/arm/actinius_icarus/actinius_icarus_common.dts
@@ -19,17 +19,17 @@
 		compatible = "gpio-leds";
 
 		red_led: led_0 {
-			gpios = <&gpio0 10 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 10 GPIO_ACTIVE_LOW>;
 			label = "Red LED";
 		};
 
 		green_led: led_1 {
-			gpios = <&gpio0 11 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 11 GPIO_ACTIVE_LOW>;
 			label = "Green LED";
 		};
 
 		blue_led: led_2 {
-			gpios = <&gpio0 12 GPIO_INT_ACTIVE_LOW>;
+			gpios = <&gpio0 12 GPIO_ACTIVE_LOW>;
 			label = "Blue LED";
 		};
 	};
@@ -57,7 +57,7 @@
 		compatible = "gpio-keys";
 
 		button0: button_0 {
-			gpios = <&gpio0 5 (GPIO_PUD_PULL_UP | GPIO_INT_ACTIVE_LOW)>;
+			gpios = <&gpio0 5 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
 			label = "Push Button 1";
 		};
 	};
@@ -123,7 +123,8 @@
 	lis2dh12-accel@19 {
 		compatible = "st,lis2dh";
 		reg = <0x19>;
-		irq-gpios = <&gpio0 28 0>, <&gpio0 29 0>;
+		irq-gpios = <&gpio0 28 GPIO_ACTIVE_HIGH>,
+			  <&gpio0 29 GPIO_ACTIVE_HIGH>;
 		label = "LIS2DH12-ACCEL";
 	};
 };

diff --git a/boards/arm/actinius_icarus/board.c b/boards/arm/actinius_icarus/board.c
@@ -22,12 +22,11 @@ static void select_sim(void)
 		return;
 	}
 
-	gpio_pin_configure(port, SIM_SELECT_PIN, GPIO_DIR_OUT);
 	#ifdef CONFIG_BOARD_SELECT_SIM_EXTERNAL
-		gpio_pin_write(port, SIM_SELECT_PIN, 0);
+		gpio_pin_configure(port, SIM_SELECT_PIN, GPIO_OUTPUT_LOW);
 		LOG_INF("External SIM is selected");
 	#else
-		gpio_pin_write(port, SIM_SELECT_PIN, 1);
+		gpio_pin_configure(port, SIM_SELECT_PIN, GPIO_OUTPUT_HIGH);
 		LOG_INF("eSIM is selected");
 	#endif
 }

diff --git a/boards/arm/atsamd20_xpro/atsamd20_xpro.dts b/boards/arm/atsamd20_xpro/atsamd20_xpro.dts
@@ -26,15 +26,15 @@
 	leds {
 		compatible = "gpio-leds";
 		yellow_led: led_0 {
-			gpios = <&porta 14 0>;
+			gpios = <&porta 14 GPIO_ACTIVE_LOW>;
 			label = "LED0";
 		};
 	};
 
 	buttons {
 		compatible = "gpio-keys";
 		user_button: button_0 {
-			gpios = <&porta 15 (GPIO_PUD_PULL_UP | GPIO_INT_ACTIVE_LOW)>;
+			gpios = <&porta 15 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
 			label = "SW0";
 		};
 	};

diff --git a/boards/arm/atsamd21_xpro/atsamd21_xpro.dts b/boards/arm/atsamd21_xpro/atsamd21_xpro.dts
@@ -27,15 +27,15 @@
 	leds {
 		compatible = "gpio-leds";
 		led0: led_0 {
-			gpios = <&portb 30 0>;
+			gpios = <&portb 30 GPIO_ACTIVE_LOW>;
 			label = "Yellow LED";
 		};
 	};
 
 	buttons {
 		compatible = "gpio-keys";
 		user_button: button_0 {
-			gpios = <&porta 15 (GPIO_PUD_PULL_UP | GPIO_INT_ACTIVE_LOW)>;
+			gpios = <&porta 15 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
 			label = "SW0";
 		};
 	};

diff --git a/boards/arm/atsamr21_xpro/atsamr21_xpro.dts b/boards/arm/atsamr21_xpro/atsamr21_xpro.dts
@@ -27,15 +27,15 @@
 	leds {
 		compatible = "gpio-leds";
 		led0: led_0 {
-			gpios = <&porta 19 0>;
+			gpios = <&porta 19 GPIO_ACTIVE_LOW>;
 			label = "Yellow LED";
 		};
 	};
 
 	buttons {
 		compatible = "gpio-keys";
 		user_button: button_0 {
-			gpios = <&porta 28 (GPIO_PUD_PULL_UP | GPIO_INT_ACTIVE_LOW)>;
+			gpios = <&porta 28 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
 			label = "SW0";
 		};
 	};