[alsa-devel] [PATCH 2/4] ASoC: soc-cache: Add support for standard register caching

Timur Tabi timur at freescale.com
Sun Oct 24 15:18:59 CEST 2010


On Sat, Oct 23, 2010 at 1:24 PM, Dimitris Papastamos
<dp at opensource.wolfsonmicro.com> wrote:
> The only problem I see with the above code, is when
> codec_drv->reg_word_size > sizeof (unsigned int) but that can't really
> happen in practice.

I'm going to have to agree with Mark that this code is suspect.  I
understand everything you said, but it makes me nervous.  Unless this
code is in some kind of fast-path, I would prefer to see it rewritten
to avoid any assumption about the sizes of the types involved.

-- 
Timur Tabi
Linux kernel developer at Freescale


More information about the Alsa-devel mailing list