Re: [PATCH 3/6] ASoC: ux500: Drop pinctrl sleep support

[Lee Jones <lee.jones@linaro.org>]

On Wed, 08 May 2013, Mark Brown wrote:

> On Wed, May 08, 2013 at 04:27:34PM +0200, Fabio Baltieri wrote:
> > On Wed, May 08, 2013 at 04:17:23PM +0200, Fabio Baltieri wrote:
> 
> > > Ok, what I'm trying to say is that the codec used in this platform
> > > should be able to handle sleep modes without requiring any
> > > reconfiguration of the digital interface on the SoC side.  In support of
> > > this the fact that the STE fork of the driver does not do that, and the
> > > same goes for all other ASoC drivers currently in mainline.
> 
> > And by the way, if the current code is *really* setting the digital
> > audio bus pins in hi-z mode (without any pull-up/down/keeper) as it
> > claims, this is not just usless, it's plain wrong.  The bus should never
> > be left floating on both sides, right?
> 
> Probably not, no.

Why don't we wait and see what LinusW says?

If anyone would know, it's him.

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog