Hi,
Series of patches to cleanup the dts for k2g to sync the dts
with yaml fixups (information provided per patch)
Basic tests performed on k2g-evm. Series has been based on v5.13-rc1 -
will be good to get it in the 5.13 window if possible.
Nishanth Menon (4):
ARM: dts: keystone: k2g: Rename message-manager node
ARM: dts: keystone: k2g: Rename the TI-SCI node
ARM: dts: keystone: k2g: Rename the TI-SCI clocks node name
ARM: dts: keystone: k2g-evm: Move audio oscillator assigned clock to
mcasp
arch/arm/boot/dts/keystone-k2g-evm.dts | 11 +++--------
arch/arm/boot/dts/keystone-k2g.dtsi | 6 +++---
2 files changed, 6 insertions(+), 11 deletions(-)
--
2.31.0
We currently use clocks as the node name for the node representing
TI-SCI clock nodes. This is better renamed to being clock-controller
as that is a better representative of the system controller function
as a clock controller for the SoC.
Signed-off-by: Nishanth Menon <[email protected]>
---
yaml fixup series:
https://lore.kernel.org/linux-arm-kernel/[email protected]
Specific yaml:
https://lore.kernel.org/linux-arm-kernel/[email protected]/
arch/arm/boot/dts/keystone-k2g.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/keystone-k2g.dtsi b/arch/arm/boot/dts/keystone-k2g.dtsi
index d7fc6057ca41..37198294f4b2 100644
--- a/arch/arm/boot/dts/keystone-k2g.dtsi
+++ b/arch/arm/boot/dts/keystone-k2g.dtsi
@@ -272,7 +272,7 @@ k2g_pds: power-controller {
#power-domain-cells = <1>;
};
- k2g_clks: clocks {
+ k2g_clks: clock-controller {
compatible = "ti,k2g-sci-clk";
#clock-cells = <2>;
};
--
2.31.0
Setting assigned clock with a dependency on itself for k2g_clks
creates a circular dependency. Instead, Lets model the audio clock
as a assigned-clock configuration for mcasp and set it up as the
first clock assigned-clock to be satisfied. Following the standard
convention, we use null entries to indicate entries that doesn't need
to be programmed.
References:
[1] schematics: https://www.ti.com/lit/df/sprr302/sprr302.pdf
[2] TRM: https://www.ti.com/lit/ug/spruhy8i/spruhy8i.pdf
Signed-off-by: Nishanth Menon <[email protected]>
---
yaml fixup series:
https://lore.kernel.org/linux-arm-kernel/[email protected]
Specific yaml:
https://lore.kernel.org/linux-arm-kernel/[email protected]/
arch/arm/boot/dts/keystone-k2g-evm.dts | 11 +++--------
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/arch/arm/boot/dts/keystone-k2g-evm.dts b/arch/arm/boot/dts/keystone-k2g-evm.dts
index 14e26a4fd62a..d800f26b6275 100644
--- a/arch/arm/boot/dts/keystone-k2g-evm.dts
+++ b/arch/arm/boot/dts/keystone-k2g-evm.dts
@@ -544,20 +544,15 @@ dpi_out: endpoint {
};
};
-&k2g_clks {
- /* on the board 22.5792MHz is connected to AUDOSC_IN */
- assigned-clocks = <&k2g_clks 0x4c 2>;
- assigned-clock-rates = <22579200>;
-};
-
&mcasp2 {
#sound-dai-cells = <0>;
pinctrl-names = "default";
pinctrl-0 = <&mcasp2_pins>;
- assigned-clocks = <&k2g_clks 0x6 1>;
- assigned-clock-parents = <&k2g_clks 0x6 2>;
+ assigned-clocks = <&k2g_clks 0x4c 2>, <&k2g_clks 0x6 1>;
+ assigned-clock-parents = <0>, <&k2g_clks 0x6 2>;
+ assigned-clock-rates = <22579200>, <0>;
status = "okay";
--
2.31.0
On 10/05/2021 17:43, Nishanth Menon wrote:
> Hi,
>
> Series of patches to cleanup the dts for k2g to sync the dts
> with yaml fixups (information provided per patch)
>
>
> Basic tests performed on k2g-evm. Series has been based on v5.13-rc1 -
> will be good to get it in the 5.13 window if possible.
>
> Nishanth Menon (4):
> ARM: dts: keystone: k2g: Rename message-manager node
> ARM: dts: keystone: k2g: Rename the TI-SCI node
> ARM: dts: keystone: k2g: Rename the TI-SCI clocks node name
> ARM: dts: keystone: k2g-evm: Move audio oscillator assigned clock to
> mcasp
>
> arch/arm/boot/dts/keystone-k2g-evm.dts | 11 +++--------
> arch/arm/boot/dts/keystone-k2g.dtsi | 6 +++---
> 2 files changed, 6 insertions(+), 11 deletions(-)
>
For the whole series:
Reviewed-by: Tero Kristo <[email protected]>