[PATCH v2 05/33] ARM: dts: am62l: Fix assigned-clock-parents
Ahmad Fatoum
a.fatoum at pengutronix.de
Mon Jun 16 04:41:17 PDT 2025
On 6/5/25 14:42, Sascha Hauer wrote:
> The AM62l device trees we imported to barebox have problems with the
> clock parents which is described in [1] and [2].
>
> There is no way the Kernel or barebox SCMI clk drivers can work with the
> current way the clk parents are specified in the downstream TF-A. This
> problem is acknowledged by TI and will hopefully be fixed.
>
> This patch fixes the clk parents the way that I hope TI will do it as
> well. Currently this only works with a patched downstream TF-A.
Sounds like worth mentioning that in the barebox K3 documentation?
>
> [1] https://lore.kernel.org/all/aCcSG5ah12N0yOwi@pengutronix.de/
> [2] https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/34834/1/plat/ti/k3/common/drivers/scmi/scmi_clock.c#165
>
> Signed-off-by: Sascha Hauer <s.hauer at pengutronix.de>
> ---
> arch/arm/dts/k3-am62l-main.dtsi | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/dts/k3-am62l-main.dtsi b/arch/arm/dts/k3-am62l-main.dtsi
> index f6fbaaede8c36ac0aab92bef7763202bb0ab30dc..d161df3215c35a2d3de67e8ca021b9a9a89edb80 100644
> --- a/arch/arm/dts/k3-am62l-main.dtsi
> +++ b/arch/arm/dts/k3-am62l-main.dtsi
> @@ -217,7 +217,7 @@ rti0: watchdog at e000000 {
> clocks = <&scmi_clk 273>;
> power-domains = <&scmi_pds 60>;
> assigned-clocks = <&scmi_clk 273>;
> - assigned-clock-parents = <&scmi_clk 1>;
> + assigned-clock-parents = <&scmi_clk 275>;
> };
>
> rti1: watchdog at e010000 {
> @@ -226,7 +226,7 @@ rti1: watchdog at e010000 {
> clocks = <&scmi_clk 279>;
> power-domains = <&scmi_pds 61>;
> assigned-clocks = <&scmi_clk 279>;
> - assigned-clock-parents = <&scmi_clk 1>;
> + assigned-clock-parents = <&scmi_clk 281>;
> };
>
> fss: bus at fc00000 {
> @@ -317,7 +317,7 @@ sdhci0: mmc at fa10000 {
> clocks = <&scmi_clk 122>, <&scmi_clk 123>;
> clock-names = "clk_ahb", "clk_xin";
> assigned-clocks = <&scmi_clk 123>;
> - assigned-clock-parents = <&scmi_clk 0>;
> + assigned-clock-parents = <&scmi_clk 124>;
> bus-width = <8>;
> ti,clkbuf-sel = <0x7>;
> ti,otap-del-sel-legacy = <0x0>;
> @@ -334,7 +334,7 @@ sdhci1: mmc at fa00000 {
> clocks = <&scmi_clk 106>, <&scmi_clk 107>;
> clock-names = "clk_ahb", "clk_xin";
> assigned-clocks = <&scmi_clk 107>;
> - assigned-clock-parents = <&scmi_clk 0>;
> + assigned-clock-parents = <&scmi_clk 108>;
> bus-width = <4>;
> ti,clkbuf-sel = <0x7>;
> ti,otap-del-sel-legacy = <0x0>;
> @@ -351,7 +351,7 @@ sdhci2: mmc at fa20000 {
> clocks = <&scmi_clk 114>, <&scmi_clk 115>;
> clock-names = "clk_ahb", "clk_xin";
> assigned-clocks = <&scmi_clk 115>;
> - assigned-clock-parents = <&scmi_clk 0>;
> + assigned-clock-parents = <&scmi_clk 116>;
> bus-width = <4>;
> ti,clkbuf-sel = <0x7>;
> ti,otap-del-sel-legacy = <0x0>;
> @@ -600,7 +600,7 @@ tscadc0: tscadc at 28001000 {
> power-domains = <&scmi_pds 0>;
> clocks = <&scmi_clk 0>;
> assigned-clocks = <&scmi_clk 0>;
> - assigned-clock-parents = <&scmi_clk 2>;
> + assigned-clock-parents = <&scmi_clk 3>;
> assigned-clock-rates = <60000000>;
> clock-names = "fck";
> status = "disabled";
> @@ -618,7 +618,7 @@ dphy_tx0: phy at 301c0000 {
> clock-names = "psm", "pll_ref";
> power-domains = <&scmi_pds 86>;
> assigned-clocks = <&scmi_clk 341>;
> - assigned-clock-parents = <&scmi_clk 0>;
> + assigned-clock-parents = <&scmi_clk 342>;
> assigned-clock-rates = <25000000>;
> #phy-cells = <0>;
> status = "disabled";
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the barebox
mailing list