linux-kernel - Re: [PATCH] riscv: dts: canaan: Add K230 clock provider DTSI

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <935ab772-fc9d-443b-8c8d-f2d17f9259aa@kernel.org>
Date: Wed, 3 Dec 2025 10:50:18 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: q691800735@....com, linux-riscv@...ts.infradead.org,
 devicetree@...r.kernel.org
Cc: conor@...nel.org, robh@...nel.org, krzk+dt@...nel.org, pjw@...nel.org,
 palmer@...belt.com, aou@...s.berkeley.edu, alex@...ti.fr,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH] riscv: dts: canaan: Add K230 clock provider DTSI

On 03/12/2025 10:26, q691800735@....com wrote:
> From: jybruceupup <q691800735@....com>
> 
> Add clock provider DTSI for Kendryte K230 SoC, required for subsystem
> clock management and boot.
> 
> Signed-off-by: jybruceupup <q691800735@....com>
> ---
>  .../boot/dts/canaan/k230_clock_provider.dtsi  | 2724 +++++++++++++++++
>  1 file changed, 2724 insertions(+)
>  create mode 100644 arch/riscv/boot/dts/canaan/k230_clock_provider.dtsi
> 
> diff --git a/arch/riscv/boot/dts/canaan/k230_clock_provider.dtsi b/arch/riscv/boot/dts/canaan/k230_clock_provider.dtsi
> new file mode 100644
> index 000000000..9d193ca22
> --- /dev/null
> +++ b/arch/riscv/boot/dts/canaan/k230_clock_provider.dtsi
> @@ -0,0 +1,2724 @@
> +// SPDX-License-Identifier: GPL-2.0 OR MIT
> +/*
> + * Copyright (c) 2023, Canaan Bright Sight Co., Ltd
> + * Copyright (c) 2025 jybruceupup <q691800735@....com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +&sysctl_boot {
> +/* osc24m clock */
> +	osc24m: osc24m {
> +		#clock-cells		= <0>;	/* only one clock output */
> +		compatible		= "fixed-clock";/* It's clock source, no parent */
> +		clock-frequency		= <24000000>;	/* clock freqence is 24MHz */
> +		clock-accuracy		= <0>;	/* software assume clock no jitter/wander.*/
> +		status				= "okay";
> +		read-only			= <0>;
> +	};
> +
> +	/* pll clock */
> +	pll0: pll0 {


NAK.

Not at all Linux style (actually DTS coding style), not used, no benefit
of any of this.

Best regards,
Krzysztof