From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751380AbdJEIw6 (ORCPT ); Thu, 5 Oct 2017 04:52:58 -0400 Received: from mx2.suse.de ([195.135.220.15]:51375 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751239AbdJEIw5 (ORCPT ); Thu, 5 Oct 2017 04:52:57 -0400 Date: Thu, 5 Oct 2017 10:52:53 +0200 From: Jean Delvare To: Cc: , , , , Subject: Re: [PATCH v2] soc: mediatek: place Kconfig for all SoC drivers under menu Message-ID: <20171005105253.2f8dd7df@endymion> In-Reply-To: <41e81ac2998b171ff4715cb2dc17f15aa10b7518.1507173110.git.sean.wang@mediatek.com> References: <41e81ac2998b171ff4715cb2dc17f15aa10b7518.1507173110.git.sean.wang@mediatek.com> Organization: SUSE Linux X-Mailer: Claws Mail 3.13.2 (GTK+ 2.24.31; x86_64-suse-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Sean, On Thu, 5 Oct 2017 11:17:49 +0800, sean.wang@mediatek.com wrote: > From: Sean Wang > > Add cleanup for placing all Kconfig for all MediaTek SoC drivers under > the independent menu as other SoCs vendor usually did. Since the menu > would be shown depending on "ARCH_MEDIATEK || COMPILE_TEST" selected and > MTK_PMIC_WRAP is still safe compiling with the case of "COMPILE_TEST" > only, the superfluous dependency for those items under the menu also is > also being removed for the sake of simplicity. > > Signed-off-by: Sean Wang > --- > drivers/soc/mediatek/Kconfig | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/drivers/soc/mediatek/Kconfig b/drivers/soc/mediatek/Kconfig > index 609bb34..a7d0667 100644 > --- a/drivers/soc/mediatek/Kconfig > +++ b/drivers/soc/mediatek/Kconfig > @@ -1,9 +1,11 @@ > # > # MediaTek SoC drivers > # > +menu "MediaTek SoC drivers" > + depends on ARCH_MEDIATEK || COMPILE_TEST > + > config MTK_INFRACFG > bool "MediaTek INFRACFG Support" > - depends on ARCH_MEDIATEK || COMPILE_TEST > select REGMAP > help > Say yes here to add support for the MediaTek INFRACFG controller. The > @@ -12,7 +14,6 @@ config MTK_INFRACFG > > config MTK_PMIC_WRAP > tristate "MediaTek PMIC Wrapper Support" > - depends on ARCH_MEDIATEK > depends on RESET_CONTROLLER > select REGMAP > help > @@ -22,7 +23,6 @@ config MTK_PMIC_WRAP > > config MTK_SCPSYS > bool "MediaTek SCPSYS Support" > - depends on ARCH_MEDIATEK || COMPILE_TEST > default ARCH_MEDIATEK > select REGMAP > select MTK_INFRACFG > @@ -30,3 +30,5 @@ config MTK_SCPSYS > help > Say yes here to add support for the MediaTek SCPSYS power domain > driver. > + > +endmenu While trying to test this, I found that I am not able to test-compile these drivers. The problem is in drivers/soc/Makefile: obj-$(CONFIG_ARCH_MEDIATEK) += mediatek/ So while Kconfig lets me select the drivers when COMPILE_TEST is enabled, the build system itself ignores the directory in which these drivers reside and they aren't built. If you really want your drivers to be test-compilable then you must change the above to: obj-y += mediatek/ I'll send a patch. Your patch itself looks good to me. Reviewed-by: Jean Delvare -- Jean Delvare SUSE L3 Support