Thread (10 messages) 10 messages, 3 authors, 2015-08-20

Re: [PATCH] ALSA: ac97: Switch to dev_pm_ops

flat view

From: Mark Brown <broonie@kernel.org>
Date: 2015-08-20 16:33:27
Also in: alsa-devel

On Fri, Aug 07, 2015 at 09:32:13AM -0700, Dmitry Torokhov wrote:
On Fri, Aug 07, 2015 at 10:13:48AM +0200, Lars-Peter Clausen wrote:
quoted
We know that it is used when CONFIG_PM_SLEEP is defined and we know that it
is unused CONFIG_PM_SLEEP is not defined. Marking the function as
__maybe_unused will cause the compiler to not generate a warning when the
function is really unused. Making this explicit works much better.
It will also drop the code form the final image and having the functions
in provides better compile coverage.
Just discussed this in person with Dmitry: I'll apply the patch just now
for v4.3 and we can incrementally improve the ifdef handling after.

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help