[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <175834772899.4354.17970704155310133154@lazor>
Date: Fri, 19 Sep 2025 22:55:28 -0700
From: Stephen Boyd <sboyd@...nel.org>
To: Alex Elder <elder@...cstar.com>, Conor Dooley <conor+dt@...nel.org>, Haylen Chu <heylenay@....org>, Inochi Amaoto <inochiama@...look.com>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Michael Turquette <mturquette@...libre.com>, Rob Herring <robh@...nel.org>, Troy Mitchell <troy.mitchell@...ux.spacemit.com>, Yixun Lan <dlan@...too.org>
Cc: linux-clk@...r.kernel.org, devicetree@...r.kernel.org, linux-riscv@...ts.infradead.org, spacemit@...ts.linux.dev, linux-kernel@...r.kernel.org, Jinmei Wei <weijinmei@...ux.spacemit.com>, Troy Mitchell <troy.mitchell@...ux.spacemit.com>
Subject: Re: [PATCH RESEND v4 3/3] clk: spacemit: fix i2s clock
Quoting Troy Mitchell (2025-09-10 20:34:05)
> Defining i2s_bclk and i2s_sysclk as fixed-rate clocks is insufficient
> for real I2S use cases.
>
> Moreover, the current I2S clock configuration does not work as expected
> due to missing parent clocks.
>
> This patch adds the missing parent clocks, defines i2s_sysclk as
> a DDN clock, and i2s_bclk as a DIV clock.
>
> A special note for i2s_bclk:
>
> From the register definition, the i2s_bclk divider always implies
> an additional 1/2 factor.
>
> The following table shows the correspondence between index
> and frequency division coefficients:
>
> | index | div |
> |-------|-------|
Applied to clk-next
Powered by blists - more mailing lists