clocksource/drivers/ingenic-ost: Define pm functions properly in platform_driver...
authorLukas Bulwahn <lukas.bulwahn@gmail.com>
Wed, 23 Nov 2022 08:31:59 +0000 (09:31 +0100)
committerThomas Gleixner <tglx@linutronix.de>
Thu, 1 Dec 2022 10:56:36 +0000 (11:56 +0100)
commit035629547999d0e9095886f248c2580dc56f36c6
treeddeeefc369bcec9f0d17956c56cd12c45c3f2897
parent3f44f7156f59cae06e9160eafb5d8b2dfd09e639
clocksource/drivers/ingenic-ost: Define pm functions properly in platform_driver struct

Commit ca7b72b5a5f2 ("clocksource: Add driver for the Ingenic JZ47xx OST")
adds the struct platform_driver ingenic_ost_driver, with the definition of
pm functions under the non-existing config PM_SUSPEND, which means the
intended pm functions were never actually included in any build.

As the only callbacks are .suspend_noirq and .resume_noirq, we can assume
that it is intended to be CONFIG_PM_SLEEP.

Since commit 1a3c7bb08826 ("PM: core: Add new *_PM_OPS macros, deprecate
old ones"), the default pattern for platform_driver definitions
conditional for CONFIG_PM_SLEEP is to use pm_sleep_ptr().

As __maybe_unused annotations on the dev_pm_ops structure and its callbacks
are not needed anymore, remove these as well.

Suggested-by: Paul Cercueil <paul@crapouillou.net>
Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Paul Cercueil <paul@crapouillou.net>
Link: https://lore.kernel.org/r/20221123083159.22821-1-lukas.bulwahn@gmail.com
drivers/clocksource/ingenic-ost.c