drivers/ft5x06: configure interrupt mode in any case

gschorcht · gschorcht · commit 7acf801855ce · 2023-08-27T18:55:28.000+02:00
The interrupt mode has to be configured independent on whether the INT pin is initialized already by the `ft5x06_init` function if parameter `cb` is defined or later, for example by the `touch_dev` implementation.
diff --git a/drivers/ft5x06/ft5x06.c b/drivers/ft5x06/ft5x06.c
@@ -80,11 +80,11 @@ int ft5x06_init(ft5x06_t *dev, const ft5x06_params_t *params, ft5x06_event_cb_t
     }
 
     /* Configure interrupt */
-    if (gpio_is_valid(dev->params->int_pin) && cb) {
+    if (gpio_is_valid(dev->params->int_pin)) {
         DEBUG("[ft5x06] init: configuring touchscreen interrupt\n");
         gpio_init_int(dev->params->int_pin, GPIO_IN, GPIO_RISING, cb, arg);
-        i2c_write_reg(FT5X06_BUS, FT5X06_ADDR, FT5X06_G_MODE_REG, FT5X06_G_MODE_INTERRUPT_TRIGGER & 0x01, 0);
     }
+    i2c_write_reg(FT5X06_BUS, FT5X06_ADDR, FT5X06_G_MODE_REG, FT5X06_G_MODE_INTERRUPT_TRIGGER & 0x01, 0);
 
     i2c_release(FT5X06_BUS);
 

Original file line number	Diff line number	Diff line change
`@@ -80,11 +80,11 @@ int ft5x06_init(ft5x06_t dev, const ft5x06_params_t params, ft5x06_event_cb_t`
`80`	`80`	`}`
`81`	`81`
`82`	`82`	`/* Configure interrupt */`
`83`		`- if (gpio_is_valid(dev->params->int_pin) && cb) {`
	`83`	`+ if (gpio_is_valid(dev->params->int_pin)) {`
`84`	`84`	`DEBUG("[ft5x06] init: configuring touchscreen interrupt\n");`
`85`	`85`	`gpio_init_int(dev->params->int_pin, GPIO_IN, GPIO_RISING, cb, arg);`
`86`		`- i2c_write_reg(FT5X06_BUS, FT5X06_ADDR, FT5X06_G_MODE_REG, FT5X06_G_MODE_INTERRUPT_TRIGGER & 0x01, 0);`
`87`	`86`	`}`
	`87`	`+ i2c_write_reg(FT5X06_BUS, FT5X06_ADDR, FT5X06_G_MODE_REG, FT5X06_G_MODE_INTERRUPT_TRIGGER & 0x01, 0);`
`88`	`88`
`89`	`89`	`i2c_release(FT5X06_BUS);`
`90`	`90`