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

From: Mark Brown
Date: Wed May 08 2013 - 10:50:20 EST


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.

Attachment: signature.asc
Description: Digital signature