linux-kernel - Re: [PATCH] ASoC: codec: es8316: "DAC Soft Ramp Rate" is just a 2 bit control

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: <0fefbfe1-021b-46b2-8945-15956e75e37f@sirena.org.uk>
Date: Mon, 27 Jan 2025 15:54:45 +0000
From: Mark Brown <broonie@...nel.org>
To: Ondřej Jirman <megi@....cz>
Cc: linux-kernel@...r.kernel.org, Liam Girdwood <lgirdwood@...il.com>,
	Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.com>,
	Uwe Kleine-König <u.kleine-koenig@...gutronix.de>,
	Andy Shevchenko <andy.shevchenko@...il.com>,
	Marian Postevca <posteuca@...ex.one>,
	"open list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM..." <linux-sound@...r.kernel.org>
Subject: Re: [PATCH] ASoC: codec: es8316: "DAC Soft Ramp Rate" is just a 2
 bit control

On Mon, Jan 27, 2025 at 04:04:55PM +0100, Ondřej Jirman wrote:
> From: Ondrej Jirman <megi@....cz>
> 
> Max value should be 3, otherwise "DAC Soft Ramp Switch" will be
> overwritten by this control.

We should probably have something to check this during registration,
possibly also have mixer-test check for other controls changing when we
change one (there will be some things that trigger on it due to bodging
things for stereo paths on some older devices but I don't know that
that's a widespread issues).

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)