ASoC: max98388: fix unused function warnings
authorArnd Bergmann <arnd@arndb.de>
Fri, 16 Jun 2023 09:00:37 +0000 (11:00 +0200)
committerMark Brown <broonie@kernel.org>
Fri, 16 Jun 2023 11:56:06 +0000 (12:56 +0100)
commit0c340ba05fda0fbf5a54207452728911c6388330
tree8f578b5c4c0e40fd68ac674f6274ae7f87ad4042
parentfadccca8f33959857948e279045a3757b5f21d55
ASoC: max98388: fix unused function warnings

The PM functions are never referenced when CONFIG_PM_SLEEP is
disabled:

sound/soc/codecs/max98388.c:854:12: error: unused function 'max98388_suspend' [-Werror,-Wunused-function]
static int max98388_suspend(struct device *dev)
           ^
sound/soc/codecs/max98388.c:864:12: error: unused function 'max98388_resume' [-Werror,-Wunused-function]
static int max98388_resume(struct device *dev)

Fix this by using the modern SYSTEM_SLEEP_PM_OPS() macro in place of
the deprecated SET_SYSTEM_SLEEP_PM_OPS() version, and use pm_sleep_ptr()
to hide the entire structure as well.

On a related note, the of_match_ptr() and ACPI_PTR() macros have the same
problem and would cause the device id table to be unused when the driver
is built-in and the respective subsystems are disabled. This does not
cause warnings unless -Wunused-const-variable is passed to the compiler,
but it's better to just not use the macros at all here.

Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/20230616090156.2347850-1-arnd@kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/max98388.c