Re: [PATCH] ASoC: make ops a pointer in 'struct snd_soc_dai'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Mar 2, 2009 at 7:55 PM, Takashi Iwai <tiwai@xxxxxxx> wrote:
> At Mon, 2 Mar 2009 11:25:46 +0800,
> Eric Miao wrote:
>>
>> Considering the fact that most cpu_dai or codec_dai are using a same
>> 'snd_soc_dai_ops' for several similar interfaces, 'ops' would be better
>> made a pointer instead, to make sharing easier and code a bit cleaner.
>>
>> The patch below is rather preliminary since the asoc tree is being
>> actively developed, and this touches almost every piece of code,
>> (and possibly many others in development need to be changed as
>> well). Building of all codecs are OK, yet to every SoC, I didn't test
>> that.
>>
>> Signed-off-by: Eric Miao <eric.miao@xxxxxxxxxxx>
>
> Don't forget to update Documentation/sound/alsa/soc/*, too.
> (I don't mind who does it, though ;)
>

Yes, I took a rough look into those files, and some of which are
already inconsistent with the existing code. E.g. 'dai_ops' looks
non-existent now, merged into '.ops' already. And other stories
I'm afraid I cann't tell much. So

Liam or Mark,

I'd prefer you guys make those descriptions more consistent if
possible?
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux