[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <175615931445.19771.17415403956670708422.b4-ty@gentoo.org>
Date: Tue, 26 Aug 2025 06:05:28 +0800
From: Yixun Lan <dlan@...nel.org>
To: Michael Turquette <mturquette@...libre.com>,
Stephen Boyd <sboyd@...nel.org>,
Alex Elder <elder@...cstar.com>,
Haylen Chu <heylenay@....org>,
Inochi Amaoto <inochiama@...look.com>,
Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Troy Mitchell <troy.mitchell@...ux.spacemit.com>
Cc: Yixun Lan <dlan@...too.org>,
linux-clk@...r.kernel.org,
linux-riscv@...ts.infradead.org,
spacemit@...ts.linux.dev,
linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org,
Yao Zi <ziyao@...root.org>
Subject: Re: [PATCH v5 0/2] clk: spacemit: fix sspax_clk
From: Yixun Lan <dlan@...too.org>
On Mon, 11 Aug 2025 21:40:32 +0800, Troy Mitchell wrote:
> In the SpacemiT public document, when the FNCLKSEL field of
> the APBC_SSPAX_CLK_RST register is 7 (3'b111),
> which is a reserved value. And BIT3 of the same register is
> a reserved bit.
>
> But the documentation is incorrect. In reality, the value 7 (3'b111)
> of FNCLKSEL is not reserved. Instead, it indicates that i2s_bclk is
> selected as the parent clock. Similarly, bit 3 is not reserved either.
> When FNCLKSEL is set to 7, bit 3 determines whether i2s_bclk is actually
> enabled as the parent clock.
>
> [...]
Thanks, Applied to SpacemiT's SoC tree at "k1/clk-for-6.18" branch
[1/2] dt-bindings: clock: spacemit: CLK_SSPA_I2S_BCLK for SSPA
https://github.com/spacemit-com/linux/commit/7d50d9bf1cd00d6bab0abf3b01d5d261aa6a2b04
[2/2] clk: spacemit: fix sspax_clk
https://github.com/spacemit-com/linux/commit/c536e00d58dc8db3e1e430a8e6830bc22d2a38e3
Best regards,
--
Yixun Lan
Powered by blists - more mailing lists