On Sat, 17 Dec 2016 01:25:27 +0100,
Jérémy Lefaure wrote:
>
> When CONFIG_PM_SLEEP is disabled, SIMPLE_DEV_PM_OPS does not use
> snd_cs5535audio_resume and snd_cs5535audio_suspend functions:
>
> sound/pci/cs5535audio/cs5535audio_pm.c:77:12: warning: ‘snd_cs5535audio_resume’ defined but not used [-Wunused-function]
> static int snd_cs5535audio_resume(struct device *dev)
> ^~~~~~~~~~~~~~~~~~~~~~
> sound/pci/cs5535audio/cs5535audio_pm.c:58:12: warning: ‘snd_cs5535audio_suspend’ defined but not used [-Wunused-function]
> static int snd_cs5535audio_suspend(struct device *dev)
> ^~~~~~~~~~~~~~~~~~~~~~~
>
> Adding __maybe_unused to the declaration of these functions removes the
> warnings.
>
> Signed-off-by: Jérémy Lefaure <jeremy.lefaure@xxxxxxxxxxxx>
Applied, thanks.
Takashi
> ---
> sound/pci/cs5535audio/cs5535audio_pm.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/sound/pci/cs5535audio/cs5535audio_pm.c b/sound/pci/cs5535audio/cs5535audio_pm.c
> index 06ac5d8da362..82bd10b68a77 100644
> --- a/sound/pci/cs5535audio/cs5535audio_pm.c
> +++ b/sound/pci/cs5535audio/cs5535audio_pm.c
> @@ -55,7 +55,7 @@ static void snd_cs5535audio_stop_hardware(struct cs5535audio *cs5535au)
>
> }
>
> -static int snd_cs5535audio_suspend(struct device *dev)
> +static int __maybe_unused snd_cs5535audio_suspend(struct device *dev)
> {
> struct snd_card *card = dev_get_drvdata(dev);
> struct cs5535audio *cs5535au = card->private_data;
> @@ -74,7 +74,7 @@ static int snd_cs5535audio_suspend(struct device *dev)
> return 0;
> }
>
> -static int snd_cs5535audio_resume(struct device *dev)
> +static int __maybe_unused snd_cs5535audio_resume(struct device *dev)
> {
> struct snd_card *card = dev_get_drvdata(dev);
> struct cs5535audio *cs5535au = card->private_data;
> --
> 2.11.0
>
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
