michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: sensor: bmi323 interrupt fix

Browse source 
Interupts should be enabled after int line configuration in bmi.
When the device goes to a suspended state interrupts must be disabled.

Signed-off-by: Wojciech Slenska <wsl@trackunit.com>
This commit is contained in:
Wojciech Slenska 2023-09-14 14:15:44 +02:00 • committed by Maureen Helm
commit fdc1cbf833
1 changed files with 16 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

17
drivers/sensor/bmi323/bmi323.c
View file

@ -1142,7 +1142,7 @@ static int bosch_bmi323_init_irq(const struct device *dev)
return ret;
}
return gpio_pin_interrupt_configure_dt(&config->int_gpio, GPIO_INT_EDGE_TO_ACTIVE);
return gpio_pin_interrupt_configure_dt(&config->int_gpio, GPIO_INT_DISABLE);
}
static int bosch_bmi323_init_int1(const struct device *dev)
@ -1172,6 +1172,7 @@ static void bosch_bmi323_irq_callback_handler(struct k_work *item)
static int bosch_bmi323_pm_resume(const struct device *dev)
{
const struct bosch_bmi323_config *config = (const struct bosch_bmi323_config *)dev->config;
int ret;
ret = bosch_bmi323_bus_init(dev);
@ -1218,6 +1219,12 @@ static int bosch_bmi323_pm_resume(const struct device *dev)
if (ret < 0) {
LOG_WRN("Failed to enable INT1");
return ret;
}
ret = gpio_pin_interrupt_configure_dt(&config->int_gpio, GPIO_INT_EDGE_TO_ACTIVE);
if (ret < 0) {
LOG_WRN("Failed to configure int");
}
return ret;
@ -1226,6 +1233,14 @@ static int bosch_bmi323_pm_resume(const struct device *dev)
#ifdef CONFIG_PM_DEVICE
static int bosch_bmi323_pm_suspend(const struct device *dev)
{
const struct bosch_bmi323_config *config = (const struct bosch_bmi323_config *)dev->config;
int ret;
ret = gpio_pin_interrupt_configure_dt(&config->int_gpio, GPIO_INT_DISABLE);
if (ret < 0) {
LOG_WRN("Failed to disable int");
}
/* Soft reset device to put it into suspend */
return bosch_bmi323_soft_reset(dev);
}