]> git.baikalelectronics.ru Git - kernel.git/commit
ASoC: fsl_easrc: mark PM functions __maybe_unused
authorArnd Bergmann <arnd@arndb.de>
Tue, 28 Apr 2020 21:28:08 +0000 (23:28 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 29 Apr 2020 16:21:32 +0000 (17:21 +0100)
commitd8d74d0202916d32c442b1a1aa0d7587401bca83
tree6ae962d9f22ccb461d890268b290df61ea4bdf8e
parentb90f277dd1f9b29d3a507403115a7a2ca6f2a7e6
ASoC: fsl_easrc: mark PM functions __maybe_unused

ifdefs are hard, and in this driver the suspend/resume functions are
the only callers of some other helpers that trigger a harmless warning
when CONFIG_PM is disabled:

sound/soc/fsl/fsl_easrc.c:1807:12: warning: 'fsl_easrc_get_firmware' defined but not used [-Wunused-function]
 1807 | static int fsl_easrc_get_firmware(struct fsl_asrc *easrc)
      |            ^~~~~~~~~~~~~~~~~~~~~~
sound/soc/fsl/fsl_easrc.c:303:12: warning: 'fsl_easrc_resampler_config' defined but not used [-Wunused-function]
  303 | static int fsl_easrc_resampler_config(struct fsl_asrc *easrc)
      |            ^~~~~~~~~~~~~~~~~~~~~~~~~~

Remove the #ifdef and just mark the callers as __maybe_unused to
suppress the warnings altogether.

Fixes: bf14c853dfac ("ASoC: fsl_easrc: Add EASRC ASoC CPU DAI drivers")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Nicolin Chen <nicoleotsuka@gmail.com>
Link: https://lore.kernel.org/r/20200428212847.2926376-1-arnd@arndb.de
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/fsl/fsl_easrc.c