clocksource/drivers/timer-ti-dm: Drop unnecessary restore
authorTony Lindgren <tony@atomide.com>
Tue, 18 May 2021 07:53:06 +0000 (10:53 +0300)
committerDaniel Lezcano <daniel.lezcano@linaro.org>
Wed, 16 Jun 2021 15:33:04 +0000 (17:33 +0200)
The device is not losing context on CPU_CLUSTER_PM_ERROR. As we are only
saving and restoring context with cpu_pm, there is no need to restore the
context in case of an error.

Note that the unnecessary restoring of context does not cause issues, it's
just not needed.

Cc: Lokesh Vutla <lokeshvutla@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Link: https://lore.kernel.org/r/20210518075306.35532-1-tony@atomide.com
drivers/clocksource/timer-ti-dm.c

index e5c631f1b5cbe49a9a098f32790ee2a552a3ffc3..3e52c5226c4d9b3e7d2c20d599f97578953f9992 100644 (file)
@@ -128,7 +128,8 @@ static int omap_timer_context_notifier(struct notifier_block *nb,
                        break;
                omap_timer_save_context(timer);
                break;
-       case CPU_CLUSTER_PM_ENTER_FAILED:
+       case CPU_CLUSTER_PM_ENTER_FAILED:       /* No need to restore context */
+               break;
        case CPU_CLUSTER_PM_EXIT:
                if ((timer->capability & OMAP_TIMER_ALWON) ||
                    !atomic_read(&timer->enabled))