lkml.org 
[lkml]   [2016]   [Apr]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [alsa-devel] [RFC PATCH v2 1/5] ASoC: Mediatek: Refine mt8173 driver
From
Date
On Thu, 2016-04-14 at 20:44 +0800, Yingjoe Chen wrote:
> On Thu, 2016-04-14 at 20:14 +0800, Garlic Tseng wrote:
> > Move mt8173 driver to another folder.
> >
> > The software control sequence of mt2701 is very different from that of mt8173,
> > so this patch moves mt8173 code to another folder.
> >
> >
> > Signed-off-by: Garlic Tseng <garlic.tseng@mediatek.com>
> > ---
> > sound/soc/mediatek/Makefile | 12 +++++-------
> > sound/soc/mediatek/mt8173/Makefile | 7 +++++++
> > sound/soc/mediatek/{ => mt8173}/mt8173-max98090.c | 2 +-
> > sound/soc/mediatek/{ => mt8173}/mt8173-rt5650-rt5514.c | 2 +-
> > sound/soc/mediatek/{ => mt8173}/mt8173-rt5650-rt5676.c | 4 ++--
> > sound/soc/mediatek/{ => mt8173}/mt8173-rt5650.c | 2 +-
> > sound/soc/mediatek/{ => mt8173}/mtk-afe-common.h | 0
> > sound/soc/mediatek/{ => mt8173}/mtk-afe-pcm.c | 0
> > 8 files changed, 17 insertions(+), 12 deletions(-)
> > create mode 100644 sound/soc/mediatek/mt8173/Makefile
> > rename sound/soc/mediatek/{ => mt8173}/mt8173-max98090.c (99%)
> > rename sound/soc/mediatek/{ => mt8173}/mt8173-rt5650-rt5514.c (99%)
> > rename sound/soc/mediatek/{ => mt8173}/mt8173-rt5650-rt5676.c (99%)
> > rename sound/soc/mediatek/{ => mt8173}/mt8173-rt5650.c (99%)
> > rename sound/soc/mediatek/{ => mt8173}/mtk-afe-common.h (100%)
> > rename sound/soc/mediatek/{ => mt8173}/mtk-afe-pcm.c (100%)
> >
> > diff --git a/sound/soc/mediatek/Makefile b/sound/soc/mediatek/Makefile
> > index d486860..5048165 100644
> > --- a/sound/soc/mediatek/Makefile
> > +++ b/sound/soc/mediatek/Makefile
> > @@ -1,7 +1,5 @@
> > -# MTK Platform Support
> > -obj-$(CONFIG_SND_SOC_MEDIATEK) += mtk-afe-pcm.o
> > -# Machine support
> > -obj-$(CONFIG_SND_SOC_MT8173_MAX98090) += mt8173-max98090.o
> > -obj-$(CONFIG_SND_SOC_MT8173_RT5650) += mt8173-rt5650.o
> > -obj-$(CONFIG_SND_SOC_MT8173_RT5650_RT5514) += mt8173-rt5650-rt5514.o
> > -obj-$(CONFIG_SND_SOC_MT8173_RT5650_RT5676) += mt8173-rt5650-rt5676.o
> > +# 8173 Machine support
> > +obj-$(CONFIG_SND_SOC_MT8173_MAX98090) += mt8173/
> > +obj-$(CONFIG_SND_SOC_MT8173_RT5650_RT5676) += mt8173/
> > +obj-$(CONFIG_SND_SOC_MT8173_RT5650_RT5514) += mt8173/
> > +obj-$(CONFIG_SND_SOC_MT8173_RT5650) += mt8173/
>
> Since all CONFIG_SND_SOC_MT8173_* depends on CONFIG_SND_SOC_MEDIATEK,
> you could just do:
>
> obj-$(CONFIG_SND_SOC_MEDIATEK) += mt8173/
>
>
> This way we don't need to modify 2 makefiles when adding new ASoC
> driver.

Yes you're right. I'll modify it.

>
>
> > diff --git a/sound/soc/mediatek/mt8173/Makefile b/sound/soc/mediatek/mt8173/Makefile
> > new file mode 100644
> > index 0000000..d486860
> > --- /dev/null
> > +++ b/sound/soc/mediatek/mt8173/Makefile
> > @@ -0,0 +1,7 @@
> > +# MTK Platform Support
> > +obj-$(CONFIG_SND_SOC_MEDIATEK) += mtk-afe-pcm.o
>
> Maybe we should also rename this to mt8173-afe-pcm.o?

Well I agree with you. Maybe I have to check all the files and modify
them with prefix "mt8173" and "mt2701". (or "mtk8173" and "mtk2701"?
I'm not so sure here)

>
> Joe.C

Thanks for your comments.



\
 
 \ /
  Last update: 2016-04-15 04:21    [W:0.041 / U:0.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site