| 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
| ||
|
Date: Sun, 17 Dec 2017 13:23:21 -0600
From: Steven Eckhoff <steven.eckhoff.opensource@...il.com>
To: Charles Keepax <ckeepax@...nsource.cirrus.com>
Cc: Mark Brown <broonie@...nel.org>, linux-kernel@...r.kernel.org,
alsa-devel@...a-project.org, Takashi Iwai <tiwai@...e.com>,
Liam Girdwood <lgirdwood@...il.com>
Subject: Re: [alsa-devel] [PATCH v5] ASoC: TSCS42xx: Add support for Tempo
Semiconductor's TSCS42xx audio CODEC
On Thu, Dec 14, 2017 at 09:32:44AM +0000, Charles Keepax wrote:
> Not sure what you mean here but setting up CODEC registers
> directly from the machine driver is probably not ideal. You
> should probably be looking into regmap_register_patch this lets
> you apply a bunch of register settings to the chip every time you
> do a regmap_cache_sync. This is useful for situations like the
> chip has terrible register defaults you want to correct.
I ended up using your suggestion with regmap_register_patch in v6. I
experimented a bit with handling all the interface configurations and it
ultimately became too complex since we would need to handle cases where
there could be one synchronous interface or two asynchronous interfaces that
only have capture or playback. Typically we have only seen uses where
there is one synchronous interface where adc and dac run at the same
rate and on other parts we have sample rate converters that allow capture
and playback to run at different rates. The default value here was
infact not very sane since it is the least used, so the patch method was
very useful in this case. Thanks for the input.
Powered by blists - more mailing lists