From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751181AbeAVJvi (ORCPT ); Mon, 22 Jan 2018 04:51:38 -0500 Received: from mga06.intel.com ([134.134.136.31]:41492 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750955AbeAVJvh (ORCPT ); Mon, 22 Jan 2018 04:51:37 -0500 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,396,1511856000"; d="scan'208";a="23629547" Message-ID: <1516614692.7000.1155.camel@linux.intel.com> Subject: Re: [PATCH] ASoC: intel: clean up CONFIG_SND_SST_IPC handling From: Andy Shevchenko To: Arnd Bergmann , Mark Brown Cc: Liam Girdwood , Jaroslav Kysela , Takashi Iwai , Pierre-Louis Bossart , Vinod Koul , Harsha Priya N , Naveen M , Daniel Drake , linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org Date: Mon, 22 Jan 2018 11:51:32 +0200 In-Reply-To: <20180121221505.196163-1-arnd@arndb.de> References: <20180121221505.196163-1-arnd@arndb.de> Organization: Intel Finland Oy Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.26.3-1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 2018-01-21 at 23:14 +0100, Arnd Bergmann wrote: > In a configuration with SND_SST_ATOM_HIFI2_PLATFORM_PCI=y and > SND_SST_ATOM_HIFI2_PLATFORM=m, we get this module link failure: > > ERROR: "sst_context_init" [sound/soc/intel/atom/sst/snd-intel-sst- > acpi.ko] undefined! > ERROR: "sst_context_cleanup" [sound/soc/intel/atom/sst/snd-intel-sst- > acpi.ko] undefined! > ERROR: "sst_alloc_drv_context" [sound/soc/intel/atom/sst/snd-intel- > sst-acpi.ko] undefined! > ERROR: "intel_sst_pm" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] > undefined! > ERROR: "sst_configure_runtime_pm" [sound/soc/intel/atom/sst/snd-intel- > sst-acpi.ko] undefined! > > The problem is that the sound/soc/intel/atom/ directory only gets > entered by Kbuild when SND_SST_ATOM_HIFI2_PLATFORM is set, so we > only build modules (obj-m) under here but not built-in files (obj-y) > including the snd-intel-sst-core that we clearly want need here. > > Before commit 4772c16ede52 ("ASoC: Intel: Kconfig: Simplify-clarify > ACPI/PCI > dependencies"), this could not happen as all files in > sound/soc/intel/atom/ > depended on CONFIG_SND_SST_ATOM_HIFI2_PLATFORM anyway. > > Slightly later, commit 05f4434bc130 ("ASoC: Intel: remove > mfld_machine") > was added, which then removed the Merrifield machine code that > happened > to be the only user of SND_SST_ATOM_HIFI2_PLATFORM_PCI. With that > gone, That's what I afraid of. Intel Merrifield *should* be there. What Vinod did, AFAIU, is removal of Intel Medfield support, which is fine with me. So, before this can go, we need to get confirmation from Vinod and Pierre, that Merrifield still stays there. > it appears that we can completely remove that symbol as well, along > with > the SND_SST_IPC_PCI code and everything associated with it that is now > unused. > > For clarify, this also removes the SND_SST_IPC_ACPI symbol that is now > synonymous with SND_SST_IPC, and links both into a single loadable > module. > I checked that this patch leads to a clean compilation, and that the > code I'm removing is only used for the now-obsolete Merrifield PCI ID. NACK for this part. > If I got something else wrong, or you want a different fix, please > just send a replacement patch and treat this as a bug report. -- Andy Shevchenko Intel Finland Oy