From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Subject: Re: [RFC PATCH 4/8] ASoC: Intel: move all ACPI match tables to common module Date: Tue, 26 Sep 2017 09:53:00 +0530 Message-ID: <20170926042300.GL30097@localhost> References: <20170908205702.1985-1-pierre-louis.bossart@linux.intel.com> <20170908205702.1985-5-pierre-louis.bossart@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by alsa0.perex.cz (Postfix) with ESMTP id 4EC51267056 for ; Tue, 26 Sep 2017 06:19:06 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20170908205702.1985-5-pierre-louis.bossart@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Pierre-Louis Bossart Cc: tiwai@suse.de, liam.r.girdwood@linux.intel.com, alsa-devel@alsa-project.org, broonie@kernel.org List-Id: alsa-devel@alsa-project.org On Fri, Sep 08, 2017 at 03:56:58PM -0500, Pierre-Louis Bossart wrote: > First step of cleaning, move all tables to soc-acpi-intel-match module > > Signed-off-by: Pierre-Louis Bossart > --- > include/sound/soc-acpi-intel-match.h | 32 +++ > sound/soc/intel/Kconfig | 24 +- > sound/soc/intel/Makefile | 2 +- > sound/soc/intel/atom/sst/sst_acpi.c | 311 ++----------------------- > sound/soc/intel/common/Makefile | 2 + > sound/soc/intel/common/soc-acpi-intel-match.c | 323 ++++++++++++++++++++++++++ > sound/soc/intel/common/sst-acpi.c | 44 +--- > 7 files changed, 395 insertions(+), 343 deletions(-) > create mode 100644 include/sound/soc-acpi-intel-match.h > create mode 100644 sound/soc/intel/common/soc-acpi-intel-match.c > > diff --git a/include/sound/soc-acpi-intel-match.h b/include/sound/soc-acpi-intel-match.h > new file mode 100644 > index 0000000..1a9191c > --- /dev/null > +++ b/include/sound/soc-acpi-intel-match.h > @@ -0,0 +1,32 @@ > + > +/* > + * Copyright (C) 2017, Intel Corporation. All rights reserved. > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms of the GNU General Public License version > + * 2 as published by the Free Software Foundation. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + */ > + > +#ifndef __LINUX_SND_SOC_ACPI_INTEL_MATCH_H > +#define __LINUX_SND_SOC_ACPI_INTEL_MATCH_H do we need LINUX on this one, it already too long :) > + > +#include > +#include > + > +/* > + * these tables are not constants, some fields can be used for > + * pdata or machine ops > + */ > +extern struct snd_soc_acpi_mach snd_soc_acpi_intel_haswell_machines[]; > +extern struct snd_soc_acpi_mach snd_soc_acpi_intel_broadwell_machines[]; > +extern struct snd_soc_acpi_mach snd_soc_acpi_intel_baytrail_legacy_machines[]; > +extern struct snd_soc_acpi_mach snd_soc_acpi_intel_baytrail_machines[]; > +extern struct snd_soc_acpi_mach snd_soc_acpi_intel_cherrytrail_machines[]; so the header is just for externs, not a pretty thing, can we avoid these somehow. Do they need to be in common file, why not keep then in respective byt/hsw file. > +config SND_SOC_INTEL_COMMON > + tristate > + > config SND_SOC_INTEL_SST > tristate > + select SND_SOC_INTEL_COMMON > select SND_SOC_INTEL_SST_ACPI if ACPI > - select SND_SOC_INTEL_SST_MATCH if ACPI > + select SND_SOC_ACPI_INTEL_MATCH if ACPI > > -config SND_SOC_INTEL_SST_MATCH > +config SND_SOC_ACPI_INTEL_MATCH > tristate > select SND_SOC_ACPI if ACPI > > @@ -145,7 +149,7 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH > select SND_SOC_RT5640 > select SND_SST_ATOM_HIFI2_PLATFORM > select SND_SST_IPC_ACPI > - select SND_SOC_INTEL_SST_MATCH if ACPI > + select SND_SOC_ACPI_INTEL_MATCH if ACPI why do you need this change, SND_SOC_INTEL_SST selects SND_SOC_ACPI_INTEL_MATCH, so we should select top symbol which is SND_SOC_INTEL_SST -- ~Vinod