linux-kernel - Re: [alsa-devel] [PATCH v5 15/18] ASoC: twl6040: Remove pll and headset mode dependency

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20110617132759.GB4951@sirena.org.uk>
Date:	Fri, 17 Jun 2011 14:27:59 +0100
From:	Mark Brown <broonie@...nsource.wolfsonmicro.com>
To:	Peter Ujfalusi <peter.ujfalusi@...com>
Cc:	Liam Girdwood <lrg@...com>, Tony Lindgren <tony@...mide.com>,
	Samuel Ortiz <sameo@...ux.intel.com>,
	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	alsa-devel@...a-project.org, linux-kernel@...r.kernel.org,
	linux-input@...r.kernel.org,
	Misael Lopez Cruz <misael.lopez@...com>,
	linux-omap@...r.kernel.org
Subject: Re: [alsa-devel] [PATCH v5 15/18] ASoC: twl6040: Remove pll and
	headset mode dependency

On Mon, Jun 13, 2011 at 07:37:47PM +0300, Peter Ujfalusi wrote:
> From: Misael Lopez Cruz <misael.lopez@...com>
> 
> Remove dependency between pll (hppll, lppll) and headset power
> mode (low-power, high-performance), as headset power mode can
> be used with any pll.

Acked-by: Mark Brown <broonie@...nsource.wolfsonmicro.com>

but...

> A new control is created to allow headset power mode configuration
> from userspace. Changing headset power mode during earpiece related
> usecases is not allowed as earpiece requires HS DAC in HP mode.

Might be nicer to allow the user to set the control given that you'll
just ignore the configured value anyway if it's not usable right now.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/