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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id B8513C36010 for ; Fri, 4 Apr 2025 14:15:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:MIME-Version:References:In-Reply-To:Message-ID:Subject:Cc:To: From:Date:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=DnbUcG+3f9P71zBXaxQwGVI9Ry02rnj7OLnB/jCN4h4=; b=fxRuQGfpk+0JT0leG+DqvXQFGF gu/MDTcvR3UXE8eGzFlgZCarAkn4uXfEwL2lpOvJ2Ij7RSreP2yM9kh3B5o0cl3MXOdIfW353Fnk3 m0PBIPdbYgwDv5Oe93cqU7dTgR6VbR7hNajP4ljs6O5cq//g3xhD6H5THN6nnZojgTfGiM13wTmq4 7jpP7l/VvB/qSiwZ+KHGFR8osscW5cEn1Bud6EfMpjxoQ9HAU19jN7mqYlw7fsCSlovU+gV6ikGrc jh6TMMosqo6ofIV5Cf6kSukuRcVG1INa1ch//i2X7NuHQzjPLRl5QpBfGIMBXmghqSn0KM0shEJlS occM0TpA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.1 #2 (Red Hat Linux)) id 1u0hpG-0000000BxOv-11Ug; Fri, 04 Apr 2025 14:15:18 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98.1 #2 (Red Hat Linux)) id 1u0hnV-0000000BxDp-0Xpn; Fri, 04 Apr 2025 14:13:30 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id BAE311515; Fri, 4 Apr 2025 07:13:27 -0700 (PDT) Received: from donnerap.manchester.arm.com (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 909073F63F; Fri, 4 Apr 2025 07:13:22 -0700 (PDT) Date: Fri, 4 Apr 2025 15:13:20 +0100 From: Andre Przywara To: Krzysztof Kozlowski Cc: Neil Armstrong , Jerome Brunet , Michael Turquette , Stephen Boyd , Kevin Hilman , Martin Blumenstingl , Maxime Coquelin , Alexandre Torgue , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Emilio =?UTF-8?B?TMOzcGV6?= , linux-amlogic@lists.infradead.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-sunxi@lists.linux.dev Subject: Re: [PATCH 4/5] clk: sunxi-ng: Do not enable by default during compile testing Message-ID: <20250404151320.53c4698b@donnerap.manchester.arm.com> In-Reply-To: <20250404-kconfig-defaults-clk-v1-4-4d2df5603332@linaro.org> References: <20250404-kconfig-defaults-clk-v1-0-4d2df5603332@linaro.org> <20250404-kconfig-defaults-clk-v1-4-4d2df5603332@linaro.org> Organization: ARM X-Mailer: Claws Mail 3.18.0 (GTK+ 2.24.32; aarch64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250404_071329_307061_9FB75E0E X-CRM114-Status: GOOD ( 15.81 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, 04 Apr 2025 13:57:00 +0200 Krzysztof Kozlowski wrote: Hi > Enabling the compile test should not cause automatic enabling of all > drivers. Restrict the default to ARCH also for individual drivers, even > though their choice is not visible without selecting parent Kconfig > symbol, because otherwise selecting parent would select the child during > compile testing. so I remember we changed this to "default y", because there were some tricky problems with regards to RISC-V and ARM. See commits: commit 0ff347db4c97cc16b4e428dc1db550ba3628f1e2 Author: Samuel Holland Date: Sat Dec 31 17:14:25 2022 -0600 clk: sunxi-ng: Move SoC driver conditions to dependencies and commit a26dc096f683ca27ac5e68703bfd3098b4212abd Author: Samuel Holland Date: Sat Dec 31 17:14:24 2022 -0600 clk: sunxi-ng: Remove duplicate ARCH_SUNXI dependencies Don't remember what broke, exactly, but just wanted to give a heads up. Cheers, Andre > > Signed-off-by: Krzysztof Kozlowski > --- > drivers/clk/sunxi-ng/Kconfig | 48 ++++++++++++++++++++++---------------------- > 1 file changed, 24 insertions(+), 24 deletions(-) > > diff --git a/drivers/clk/sunxi-ng/Kconfig b/drivers/clk/sunxi-ng/Kconfig > index 5830a9d87bf25d536ac787fe83669c64c8214952..8896fd052ef1784d60d488ab1498737c1405deb2 100644 > --- a/drivers/clk/sunxi-ng/Kconfig > +++ b/drivers/clk/sunxi-ng/Kconfig > @@ -9,123 +9,123 @@ if SUNXI_CCU > > config SUNIV_F1C100S_CCU > tristate "Support for the Allwinner newer F1C100s CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUNIV || COMPILE_TEST > > config SUN20I_D1_CCU > tristate "Support for the Allwinner D1/R528/T113 CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || RISCV || COMPILE_TEST > > config SUN20I_D1_R_CCU > tristate "Support for the Allwinner D1/R528/T113 PRCM CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || RISCV || COMPILE_TEST > > config SUN50I_A64_CCU > tristate "Support for the Allwinner A64 CCU" > - default y > + default ARCH_SUNXI > depends on ARM64 || COMPILE_TEST > > config SUN50I_A100_CCU > tristate "Support for the Allwinner A100 CCU" > - default y > + default ARCH_SUNXI > depends on ARM64 || COMPILE_TEST > > config SUN50I_A100_R_CCU > tristate "Support for the Allwinner A100 PRCM CCU" > - default y > + default ARCH_SUNXI > depends on ARM64 || COMPILE_TEST > > config SUN50I_H6_CCU > tristate "Support for the Allwinner H6 CCU" > - default y > + default ARCH_SUNXI > depends on ARM64 || COMPILE_TEST > > config SUN50I_H616_CCU > tristate "Support for the Allwinner H616 CCU" > - default y > + default ARCH_SUNXI > depends on ARM64 || COMPILE_TEST > > config SUN50I_H6_R_CCU > tristate "Support for the Allwinner H6 and H616 PRCM CCU" > - default y > + default ARCH_SUNXI > depends on ARM64 || COMPILE_TEST > > config SUN55I_A523_CCU > tristate "Support for the Allwinner A523/T527 CCU" > - default y > + default ARCH_SUNXI > depends on ARM64 || COMPILE_TEST > > config SUN55I_A523_R_CCU > tristate "Support for the Allwinner A523/T527 PRCM CCU" > - default y > + default ARCH_SUNXI > depends on ARM64 || COMPILE_TEST > > config SUN4I_A10_CCU > tristate "Support for the Allwinner A10/A20 CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN4I || MACH_SUN7I || COMPILE_TEST > > config SUN5I_CCU > bool "Support for the Allwinner sun5i family CCM" > - default y > + default ARCH_SUNXI > depends on MACH_SUN5I || COMPILE_TEST > depends on SUNXI_CCU=y > > config SUN6I_A31_CCU > tristate "Support for the Allwinner A31/A31s CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN6I || COMPILE_TEST > > config SUN6I_RTC_CCU > tristate "Support for the Allwinner H616/R329 RTC CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || ARM64 || RISCV || COMPILE_TEST > > config SUN8I_A23_CCU > tristate "Support for the Allwinner A23 CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || COMPILE_TEST > > config SUN8I_A33_CCU > tristate "Support for the Allwinner A33 CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || COMPILE_TEST > > config SUN8I_A83T_CCU > tristate "Support for the Allwinner A83T CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || COMPILE_TEST > > config SUN8I_H3_CCU > tristate "Support for the Allwinner H3 CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || ARM64 || COMPILE_TEST > > config SUN8I_V3S_CCU > tristate "Support for the Allwinner V3s CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || COMPILE_TEST > > config SUN8I_DE2_CCU > tristate "Support for the Allwinner SoCs DE2 CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || ARM64 || RISCV || COMPILE_TEST > > config SUN8I_R40_CCU > tristate "Support for the Allwinner R40 CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || COMPILE_TEST > > config SUN9I_A80_CCU > tristate "Support for the Allwinner A80 CCU" > - default y > + default ARCH_SUNXI > depends on MACH_SUN9I || COMPILE_TEST > > config SUN8I_R_CCU > tristate "Support for Allwinner SoCs' PRCM CCUs" > - default y > + default ARCH_SUNXI > depends on MACH_SUN8I || ARM64 || COMPILE_TEST > > endif >