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).