Hi Rob, sorry for the delay.
On Fri, Apr 21, 2017 at 09:22:02PM +0200, Jens Rottmann wrote:
- Remove Eukrea naming, i.MX is from Freescale, TLV320AIC23 is from TI, driver was written by Eukrea, but it's DT capable, so it's not exclusive:
- Kconfig option title
- 'model' option
- driver 'compatible' string
[...] CONFIG variable and files have not been renamed, though, so no need to change old defconfigs.
On 04/28/2017 19:11, Rob Herring answered:
Perhaps change the filename. [...]
I wanted to avoid the churn, but ok. In that case:
Do you want me to change all indentifiers inside driver, too? eukrea_tlv320_probe() --> imx_tlv320aic23_probe()
Do you want me to also rename the CONFIG var? CONFIG_SND_SOC_EUKREA_TLV320 --> CONFIG_SND_SOC_IMX_TLV320 I'd adapt in-tree defconfigs, but would unavoidably break out-of-tree defconfigs...
- fsl,mux-int-port : The internal port of the i.MX audio muxer (AUDMUX).
- mux-int-port : The internal port of the i.MX audio muxer (AUDMUX).
- fsl,mux-int-port : Dito, deprecated.
- fsl,mux-ext-port : The external port of the i.MX audio muxer.
- mux-ext-port : The external port of the i.MX audio muxer.
- fsl,mux-ext-port : Dito, deprecated.
Is this used elsewhere? This is FSL specific, so you should keep the prefix.
.../sound/soc/fsl$ egrep 'property.*mux-(int|ext)-port' * eukrea-tlv320.c: ret = of_property_read_u32(np, "fsl,mux-int-port", &int_port); eukrea-tlv320.c: ret = of_property_read_u32(np, "fsl,mux-ext-port", &ext_port); fsl-asoc-card.c: ret = of_property_read_u32(np, "mux-int-port", &int_port); fsl-asoc-card.c: ret = of_property_read_u32(np, "mux-ext-port", &ext_port); imx-es8328.c: ret = of_property_read_u32(np, "mux-int-port", &int_port); imx-es8328.c: ret = of_property_read_u32(np, "mux-ext-port", &ext_port); imx-sgtl5000.c: ret = of_property_read_u32(np, "mux-int-port", &int_port); imx-sgtl5000.c: ret = of_property_read_u32(np, "mux-ext-port", &ext_port); imx-wm8962.c: ret = of_property_read_u32(np, "mux-int-port", &int_port); imx-wm8962.c: ret = of_property_read_u32(np, "mux-ext-port", &ext_port);
All drivers use mux-int|ext-port without prefix, tlv320 is the only odd driver out, that's why I did this streamlining patch in the 1st place.
I would rather not have to change all other drivers to match the tlv320 one.
Thanks, Jens