driver/sensor: lis2mdl: update to use new GPIO API

Get rid of all the deprecated functions and definitions
replacing them with the new ones.

Signed-off-by: Armando Visconti <[email protected]>

Author: avisconti

boards/arm/bbc_microbit/bbc_microbit.dts

@@ -81,7 +81,7 @@
 		status = "disabled";
 		reg = <0x1e>;
 		label = "LSM303AGR-MAGN";
-		irq-gpios = <&gpio0 27 0>;	/* A3 */
+		irq-gpios = <&gpio0 27 GPIO_ACTIVE_HIGH>;	/* A3 */
 	};
 
 	lsm303agr-accel@19 {

boards/shields/x_nucleo_iks01a2/x_nucleo_iks01a2.overlay

@@ -30,7 +30,7 @@
 		compatible = "st,lis2mdl","st,lsm303agr-magn";
 		reg = <0x1e>;
 		label = "LSM303AGR-MAGN";
-		irq-gpios = <&arduino_header 3 0>;	/* A3 */
+		irq-gpios = <&arduino_header 3 GPIO_ACTIVE_HIGH>;	/* A3 */
 	};
 
 	lsm303agr-accel@19 {

boards/shields/x_nucleo_iks01a3/x_nucleo_iks01a3.overlay

@@ -29,7 +29,7 @@
 	lis2mdl@1e {
 		compatible = "st,lis2mdl";
 		reg = <0x1e>;
-		irq-gpios =  <&arduino_header 2 0>; /* A2 */
+		irq-gpios =  <&arduino_header 2 GPIO_ACTIVE_HIGH>; /* A2 */
 		label = "LIS2MDL";
 	};
 

drivers/sensor/lis2mdl/lis2mdl.c

@@ -259,6 +259,7 @@ static const struct lis2mdl_config lis2mdl_dev_config = {
 #ifdef CONFIG_LIS2MDL_TRIGGER
 	.gpio_name = DT_INST_0_ST_LIS2MDL_IRQ_GPIOS_CONTROLLER,
 	.gpio_pin = DT_INST_0_ST_LIS2MDL_IRQ_GPIOS_PIN,
+	.gpio_flags = DT_INST_0_ST_LIS2MDL_IRQ_GPIOS_FLAGS,
 #endif  /* CONFIG_LIS2MDL_TRIGGER */
 #if defined(DT_ST_LIS2MDL_BUS_SPI)
 	.bus_init = lis2mdl_spi_init,

drivers/sensor/lis2mdl/lis2mdl.h

@@ -33,6 +33,7 @@ struct lis2mdl_config {
 #ifdef CONFIG_LIS2MDL_TRIGGER
 	char *gpio_name;
 	u32_t gpio_pin;
+	u8_t gpio_flags;
 #endif  /* CONFIG_LIS2MDL_TRIGGER */
 #ifdef DT_ST_LIS2MDL_BUS_I2C
 	u16_t i2c_slv_addr;

drivers/sensor/lis2mdl/lis2mdl_trigger.c

@@ -62,7 +62,8 @@ static void lis2mdl_handle_interrupt(void *arg)
 		lis2mdl->handler_drdy(dev, &drdy_trigger);
 	}
 
-	gpio_pin_enable_callback(lis2mdl->gpio, config->gpio_pin);
+	gpio_pin_interrupt_configure(lis2mdl->gpio, config->gpio_pin,
+				     GPIO_INT_EDGE_TO_ACTIVE);
 }
 
 static void lis2mdl_gpio_callback(struct device *dev,
@@ -74,7 +75,7 @@ static void lis2mdl_gpio_callback(struct device *dev,
 
 	ARG_UNUSED(pins);
 
-	gpio_pin_disable_callback(dev, config->gpio_pin);
+	gpio_pin_interrupt_configure(dev, config->gpio_pin, GPIO_INT_DISABLE);
 
 #if defined(CONFIG_LIS2MDL_TRIGGER_OWN_THREAD)
 	k_sem_give(&lis2mdl->gpio_sem);
@@ -134,8 +135,7 @@ int lis2mdl_init_interrupt(struct device *dev)
 #endif
 
 	gpio_pin_configure(lis2mdl->gpio, config->gpio_pin,
-			   GPIO_DIR_IN | GPIO_INT | GPIO_INT_EDGE |
-			   GPIO_INT_ACTIVE_HIGH | GPIO_INT_DEBOUNCE);
+			   GPIO_INPUT | config->gpio_flags);
 
 	gpio_init_callback(&lis2mdl->gpio_cb,
 			   lis2mdl_gpio_callback,
@@ -146,5 +146,6 @@ int lis2mdl_init_interrupt(struct device *dev)
 		return -EIO;
 	}
 
-	return gpio_pin_enable_callback(lis2mdl->gpio, config->gpio_pin);
+	return gpio_pin_interrupt_configure(lis2mdl->gpio, config->gpio_pin,
+					    GPIO_INT_EDGE_TO_ACTIVE);
 }

dts/bindings/sensor/st,lis2mdl-i2c.yaml

@@ -12,3 +12,8 @@ properties:
     irq-gpios:
       type: phandle-array
       required: false
+      description: IRQ pin
+
+        This pin defaults to active high when produced by the sensor.
+        The property value should ensure the flags properly describe
+        the signal that is presented to the driver.

dts/bindings/sensor/st,lis2mdl-spi.yaml

@@ -12,3 +12,8 @@ properties:
     irq-gpios:
       type: phandle-array
       required: false
+      description: IRQ pin
+
+        This pin defaults to active high when produced by the sensor.
+        The property value should ensure the flags properly describe
+        the signal that is presented to the driver.