Re: [PATCH v6 3/6] clk: spacemit: Add clock support for SpacemiT K1 SoC

From: Alex Elder <elder@riscstar.com>
To: Inochi Amaoto <inochiama@gmail.com>, Yixun Lan <dlan@gentoo.org>
Cc: Haylen Chu <heylenay@4d2.org>,
	Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Haylen Chu <heylenay@outlook.com>,
	Paul Walmsley <paul.walmsley@sifive.com>,
	Palmer Dabbelt <palmer@dabbelt.com>,
	Albert Ou <aou@eecs.berkeley.edu>,
	Alexandre Ghiti <alex@ghiti.fr>,
	linux-riscv@lists.infradead.org, linux-clk@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	spacemit@lists.linux.dev, Inochi Amaoto <inochiama@outlook.com>,
	Chen Wang <unicornxdotw@foxmail.com>,
	Jisheng Zhang <jszhang@kernel.org>,
	Meng Zhang <zhangmeng.kevin@linux.spacemit.com>
Subject: Re: [PATCH v6 3/6] clk: spacemit: Add clock support for SpacemiT K1 SoC
Date: Thu, 10 Apr 2025 07:32:36 -0500	[thread overview]
Message-ID: <0ed3ce4c-9be0-485b-a1ea-6a34931179db@riscstar.com> (raw)
In-Reply-To: <cb13c955-3994-4950-9c28-37703f749b0e@riscstar.com>

On 4/10/25 7:30 AM, Alex Elder wrote:
> 
> I'd like to see that be the only thing there, and have
> various SpacemiT modules define default values that
> depend on ARCH_SPACEMIT (or _K1) in their Kconfig
> file.  Like:
> 
>      config SPACEMIT_K1_CCU
>          tristate "Support for SpacemiT K1 SoC"
>          depends on ARCH_SPACEMIT || COMPILE_TEST
>          default m if ARCH_SPACEMIT
> 
> I *think* Haylen said that's what he's going to do.  You
> could make it "default ARCH_SPACEMIT" too, though that
> builds it in to the kernel.

Nope, I just reread what he said, and I'm mistaken.  Go
with what's preferred by the clock maintainers.

					-Alex

_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv