[alsa-devel] [PATCH 0/7][RFC] ASoC: modern style CPU

Mark Brown broonie at kernel.org
Wed Apr 17 19:28:07 CEST 2019


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).
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20190417/65935961/attachment.sig>


More information about the Alsa-devel mailing list