[alsa-devel] Is that reasonable to support pinctrl PM in ASoC core?

Mark Brown broonie at kernel.org
Thu Oct 24 16:15:24 CEST 2013

On Thu, Oct 24, 2013 at 03:19:03PM +0200, Linus Walleij wrote:

> This makes a lot of sense, the only problem I have traditionally had
> with this is that there exist cases where you want to put pinctrl
> handles to sleep or idle aside from runtime PM.

Yeah, that was my concern.

> - I2C or SPI hosts with no devices connected to them should be
>   nailed to sleep. (It's not possible to detect an unused I2C or SPI
>   host today, but a feature we should add.)

The SPI case should be mostly handled already if this is integrated via
runtime PM - we have core support for idling controllers whenever there
is no transfer active.  That should go off for devices that aren't
connected to anything.

I should probably go and implement an I2C version of that.

> Maybe the right way to do these is also to use runtime_pm*,
> but the thing is that today these things are done also when
> CONFIG_PM_RUNTIME is not set. Which is something you
> might sometimes want.

> Or not? Should we say that unless you have
> CONFIG_PM_RUNTIME you don't deserve saving these
> microamps anyway?

Personally I tend to think that people who care probably ought to be
using runtime PM but that's not a sufficiently general view to insist

As we discussed in person earlier how about we merge this on a subsystem
basis and then once we've established what the pattern really is we
think about putting it into the core.  Nicolin, do you want to write a
patch for this?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20131024/9568474d/attachment.sig>

More information about the Alsa-devel mailing list