From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 605BFC83000 for ; Tue, 28 Apr 2020 23:00:32 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 135E8206F0 for ; Tue, 28 Apr 2020 23:00:32 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="VG1GDLg/" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 135E8206F0 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type: In-Reply-To:MIME-Version:Date:Message-ID:References:To:From:Subject:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=FAY+XfuPT86bLWMWBgAp6CMxBnsKXjy48bQHU6w5S/k=; b=VG1GDLg/U7Qz4ULbBIHgmKEvn /h6zsO06kGVM/lQ1IP6qEoxsih6dGCsbvbN4UTiE6cgRr3n+Ok80yqilueX9tbSEZq26v3mxux182 vDap7aTHV5H/aqnWHBpHCLpAI+LzcU6v8eyjjZ6sN7YTRo4f0kXfMzPSAmUWoxW+Ot6zyLnlAdoKy Aklmkw70x8I4JFQxq1xc3QcWH53ZOCYNXiW7WW/fxpzzldF1TeeB2wGeO19F+H9ZYhNp6w68oL+fk SSYl/gY70sqO+M18Wsbia451SL7yzdEX0SdKbhbGzZgCaqnulyZbMCUBA/Y/8Sj6CljnbH38HshFw 6kmBDWI9Q==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jTZDH-00069O-M6; Tue, 28 Apr 2020 23:00:27 +0000 Received: from mga18.intel.com ([134.134.136.126]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jTZDE-00068F-Lo for linux-arm-kernel@lists.infradead.org; Tue, 28 Apr 2020 23:00:26 +0000 IronPort-SDR: 32fo3hrh7dFeM0L+A5vfIrmbLZJYxhuVrrtDyCrABzIhkJoVqXIndwiZIz4u1Ni7Xr5AmjjQ/E HpjQVZitpGwA== X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Apr 2020 16:00:23 -0700 IronPort-SDR: XdQu1T4O+w7tdYK2qGoB/pje7lUhIhCK+zaZQPZtijPvA6alH1AnAEYXmwSywMks5ISfrR8SCX JAGfDdavyxtg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,328,1583222400"; d="scan'208,223";a="247840359" Received: from mdioury-mobl.amr.corp.intel.com (HELO [10.251.153.102]) ([10.251.153.102]) by fmsmga007.fm.intel.com with ESMTP; 28 Apr 2020 16:00:20 -0700 Subject: Re: [PATCH] ASoC: SOF: sort out Kconfig, again From: Pierre-Louis Bossart To: Arnd Bergmann References: <20200428212752.2901778-1-arnd@arndb.de> <6b39fbba-c65d-2c02-14bf-11c2d00547af@linux.intel.com> Message-ID: Date: Tue, 28 Apr 2020 18:00:19 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/mixed; boundary="------------83E0C404AAA0BEDB5B37198B" Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200428_160024_764610_25B8F070 X-CRM114-Status: GOOD ( 25.62 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Stephen Rothwell , ALSA Development Mailing List , Kai Vehmanen , Daniel Baluta , Sascha Hauer , Takashi Iwai , YueHaibing , Liam Girdwood , Krzysztof Kozlowski , "linux-kernel@vger.kernel.org" , NXP Linux Team , Mark Brown , Ranjani Sridharan , Pengutronix Kernel Team , Shawn Guo , Fabio Estevam , Linux ARM , sound-open-firmware@alsa-project.org Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org This is a multi-part message in MIME format. --------------83E0C404AAA0BEDB5B37198B Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit >>> Thanks Arnd, do you mind sharing your config? >> >> https://pastebin.com/HRX5xi3R > > will give it a try, thanks! > >>> We noticed last week that >>> there's a depend/select confusion might be simpler to fix, see >>> https://github.com/thesofproject/linux/pull/2047/commits >>> >>> If I look at the first line I see a IMX_DSP=n which looks exactly like >>> what we wanted to fix. >> >> Yes, I think that fix addresses the build warning as well, but looking >> more closely I don't think it's what you want: If you do this on >> a config that has the IMX_DSP disabled, it would appear to the >> user that you have enabled the drivers, but the actual code is still >> disabled. > > Are you sure? we added a select IMX_DSP, so not sure how it can be > disabled? I just tested Arnd's config with the patch we came up with for SOF (attached) and it makes the unmet dependency go away and builds fine. the problem is really using select IMX_DSP if it can be disabled by something else. My proposal looks simpler but I will agree it's not necessarily super elegant to move the dependency on IMX_BOX into SOF, so no sustained objection from me on Arnd's proposal. Daniel, this is your part of SOF, please chime in. Thanks -Pierre --------------83E0C404AAA0BEDB5B37198B Content-Type: text/x-patch; charset=UTF-8; name="0001-ASoC-SOF-imx-fix-depends-select-IMX_DSP-confusion.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename*0="0001-ASoC-SOF-imx-fix-depends-select-IMX_DSP-confusion.patch" >From 208e61ae18d3a4aa93ffa73db01c4e3c24a4979f Mon Sep 17 00:00:00 2001 From: Pierre-Louis Bossart Date: Wed, 22 Apr 2020 06:21:56 -0500 Subject: [PATCH] ASoC: SOF: imx: fix depends/select IMX_DSP confusion The two IMX targets don't use depends/select in a consistent way and there's a potential for an unmet dependency. Move the dependency check to a higher level and select IMX_DSP to avoid builtin/module issues. Fixes: afb93d716533dd ("ASoC: SOF: imx: Add i.MX8M HW support") Signed-off-by: Pierre-Louis Bossart --- sound/soc/sof/imx/Kconfig | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/soc/sof/imx/Kconfig b/sound/soc/sof/imx/Kconfig index f76660e91382..a11ecc1d56db 100644 --- a/sound/soc/sof/imx/Kconfig +++ b/sound/soc/sof/imx/Kconfig @@ -21,7 +21,7 @@ config SND_SOC_SOF_IMX_OF config SND_SOC_SOF_IMX8_SUPPORT bool "SOF support for i.MX8" - depends on IMX_SCU + depends on IMX_SCU && IMX_MBOX help This adds support for Sound Open Firmware for NXP i.MX8 platforms Say Y if you have such a device. @@ -29,7 +29,6 @@ config SND_SOC_SOF_IMX8_SUPPORT config SND_SOC_SOF_IMX8 tristate - depends on IMX_SCU select IMX_DSP help This option is not user-selectable but automagically handled by @@ -37,6 +36,7 @@ config SND_SOC_SOF_IMX8 config SND_SOC_SOF_IMX8M_SUPPORT bool "SOF support for i.MX8M" + depends on IMX_MBOX help This adds support for Sound Open Firmware for NXP i.MX8M platforms Say Y if you have such a device. @@ -44,7 +44,7 @@ config SND_SOC_SOF_IMX8M_SUPPORT config SND_SOC_SOF_IMX8M tristate - depends on IMX_DSP + select IMX_DSP help This option is not user-selectable but automagically handled by 'select' statements at a higher level -- 2.20.1 --------------83E0C404AAA0BEDB5B37198B Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --------------83E0C404AAA0BEDB5B37198B--