[PATCH] soundwire: intel: fix CONFIG_PM and CONFIG_PM_SLEEP confusion
Pierre-Louis Bossart
pierre-louis.bossart at linux.intel.com
Fri Aug 21 17:23:53 CEST 2020
>> When CONFIG_PM_SLEEP is not defined, GCC throws compilation warnings:
>>
>> drivers/soundwire/intel.c:1816:12: warning: ‘intel_resume’ defined but
>> not used [-Wunused-function]
>> 1816 | static int intel_resume(struct device *dev)
>> | ^~~~~~~~~~~~
>>
>> drivers/soundwire/intel.c:1697:12: warning: ‘intel_suspend’ defined
>> but not used [-Wunused-function]
>> 1697 | static int intel_suspend(struct device *dev)
>>
>> Fix by adding the missing CONFIG_PM_SLEEP.
>
> Can you rather use __maybe for for these rather than wrapping in another
> ifdef, that is the recommended way to do this
No objections, that would work as well.
More information about the Alsa-devel
mailing list