Re: [PATCH 0/7][RFC] ASoC: modern style CPU

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

 



On Wed, Apr 17, 2019 at 03:26:48PM +0900, Kuninori Morimoto wrote:

> > +SND_SOC_DAI_LINK_CCP(aif1,
> > +	SND_SOC_DAI_LINK(SDL_CPU("samsung-i2s.0")),
> > +	SND_SOC_DAI_LINK(SDL_CODEC("wm8994-codec", "wm8994-aif1")),
> > +	SND_SOC_DAI_LINK(SDL_PLATFORM("samsung-i2s.0")));

> > is this really the new direction?
> > Even the acronyms are not simple, it took me 15mn to figure out that
> > CCP stood for CPU/Codec/Platform and I couldn't figure out what SDL
> > means.

> The reason of "CCP" (= CPU/CODEC/PLATFORM) was to avoid long naming.
> and SDL is acronyms of "Snd soc Dai Link". but yes, it is un-understandable.
> It should be more understandable (and possibly short) naming.

I think you're right to want a shorter name but Pierre's probably right
that that particular name isn't the best...  I'm thinking it might be
good to look at the fully expanded definitions and then work back from
there, or perhaps just stick with them for a while (since the compact
forms are hopefully not a requirement for legibility, even if they are a
useful improvement).

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

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

  Powered by Linux