Hi Yingjoe, thanks for comments. > > +config SND_SOC_MEDIATEK > > + tristate "SND_SOC_MEDIATEK" > > + depends on ARCH_MEDIATEK > > + help > > + This adds ASoC driver for Mediatek boards > > + that can be used with other codecs. > > + Select Y if you have such device. > > + If unsure select "N". > > + > > config SND_SOC_MT8173 > > tristate "ASoC support for Mediatek MT8173 chip" > > depends on ARCH_MEDIATEK > > + select SND_SOC_MEDIATEK > > help > > This adds ASoC platform driver support for Mediatek MT8173 chip > > that can be used with other codecs. > > It seems both SND_SOC_MT8173 and SND_SOC_MT2701 select SND_SOC_MEDIATEK, > and enabling only SND_SOC_MEDIATEK itself is not really useful. Do we > need to make SND_SOC_MEDIATEK user configurable option?
I'll make the config not user configurable, something like: +config SND_SOC_MEDIATEK + tristate > > If it is, we should at least have a proper config prompt instead of just > "SND_SOC_MEDIATEK". > > > > @@ -49,3 +59,4 @@ config SND_SOC_MT8173_RT5650_RT5676 > > with the RT5650 and RT5676 codecs. > > Select Y if you have such device. > > If unsure select "N". > > + > > nit: This change is not necessary. I'll fix it, thanks. > > Joe.C > >
