> This add ways for the SoC to wake from accelerometer wake events.
>
> In the suspend function we skip disabling the sensor if wakeup-source
> and events are activated.
>
[...]
> EXPORT_SYMBOL(st_lsm6dsx_probe);
> @@ -1372,6 +1376,12 @@ static int __maybe_unused st_lsm6dsx_suspend(struct device *dev)
> if (!hw->iio_devs[i])
> continue;
>
> + if (device_may_wakeup(dev) && (i == ST_LSM6DSX_ID_ACC)) {
unnecessary brackets
> + /* Enable wake from IRQ */
> + enable_irq_wake(hw->irq);
> + continue;
> + }
I think we need to move this after enable_mask check, dont' we?
> +
> sensor = iio_priv(hw->iio_devs[i]);
> if (!(hw->enable_mask & BIT(sensor->id)))
> continue;
> @@ -1404,6 +1414,11 @@ static int __maybe_unused st_lsm6dsx_resume(struct device *dev)
> if (!hw->iio_devs[i])
> continue;
>
> + if (device_may_wakeup(dev) && (i == ST_LSM6DSX_ID_ACC)) {
unnecessary brackets
> + disable_irq_wake(hw->irq);
> + continue;
> + }
> +
> sensor = iio_priv(hw->iio_devs[i]);
> if (!(hw->suspend_mask & BIT(sensor->id)))
> continue;
> --
> 2.22.0
>
Attachment:
signature.asc
Description: PGP signature
