On Tue, Aug 24, 2010 at 03:09:43PM +0200, Uwe Kleine-K??nig wrote:
Signed-off-by: Uwe Kleine-K??nig u.kleine-koenig@pengutronix.de
Pretty much the same comments apply to this as applied to the immediately preceeding patch - doing it at driver level seems like the wrong approacg and please always remember to CC maintainers on patches. Also...
sound/soc/codecs/ad1980.c | 13 +++++++++++++ 1 files changed, 13 insertions(+), 0 deletions(-)
diff --git a/sound/soc/codecs/ad1980.c b/sound/soc/codecs/ad1980.c index 864e65d..a3e78d4 100644 --- a/sound/soc/codecs/ad1980.c +++ b/sound/soc/codecs/ad1980.c @@ -288,6 +288,19 @@ static int ad1980_soc_probe(struct platform_device *pdev) snd_soc_add_controls(codec, ad1980_snd_ac97_controls, ARRAY_SIZE(ad1980_snd_ac97_controls));
+#if AC97_USE_CACHE
- pr_debug("%s: cache verify\n", __func__);
- for (vendor_id2 = 0; vendor_id2 < 0x80; vendor_id2 += 2) {
...this should probably be selected separately and...
u16 *cache = codec->reg_cache;ret = soc_ac97_ops.read(codec->ac97, vendor_id2);if (ret != cache[vendor_id2 >> 1])pr_debug("%s: %02x -> %04x\n",__func__, vendor_id2, ret);
...this seems like the wrong priority if there are any inconsistencies. Note that if the chip includes anything that might reasonably vary (like revision information or status readback) you probably want to handle that too.
- }
- pr_debug("%s: /cache verify\n", __func__);
+#endif
- return 0;
reset_err:
1.7.1
Alsa-devel mailing list Alsa-devel@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel