From c6cce80ac4b5dc5af86178db9692dc476fdb6378 Mon Sep 17 00:00:00 2001 From: Gerard Marull-Paretas Date: Fri, 30 Jul 2021 11:22:27 +0200 Subject: [PATCH] pm: remove incorrect usages of force suspend Some devices are using PM_DEVICE_STATE_FORCE_SUSPEND as a sort of low power state, something that is not correct. In fact, this state is not an actual state and will be eventually moved, if found necessary, to an action or command. Signed-off-by: Gerard Marull-Paretas --- drivers/interrupt_controller/intc_ioapic.c | 1 - drivers/pwm/pwm_nrfx.c | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/interrupt_controller/intc_ioapic.c b/drivers/interrupt_controller/intc_ioapic.c index b62aea69fec..a35833d87a5 100644 --- a/drivers/interrupt_controller/intc_ioapic.c +++ b/drivers/interrupt_controller/intc_ioapic.c @@ -321,7 +321,6 @@ static int ioapic_device_ctrl(const struct device *dev, ret = ioapic_resume_from_suspend(dev); break; case PM_DEVICE_STATE_SUSPEND: - case PM_DEVICE_STATE_FORCE_SUSPEND: case PM_DEVICE_STATE_OFF: ret = ioapic_suspend(dev); break; diff --git a/drivers/pwm/pwm_nrfx.c b/drivers/pwm/pwm_nrfx.c index 1cbfbfbcc1b..006d5990e21 100644 --- a/drivers/pwm/pwm_nrfx.c +++ b/drivers/pwm/pwm_nrfx.c @@ -303,11 +303,11 @@ static int pwm_nrfx_set_power_state(enum pm_device_state state, break; case PM_DEVICE_STATE_LOW_POWER: case PM_DEVICE_STATE_SUSPEND: - case PM_DEVICE_STATE_FORCE_SUSPEND: case PM_DEVICE_STATE_OFF: pwm_nrfx_uninit(dev); break; default: + __ASSERT_NO_MSG(false); break; }