2022-04-22 18:23:26

by Sean Anderson

Subject: [PATCH 5/8] arm64: dts: ls1028a: Update SFP binding to include clock

The clocks property is now mandatory. Add it.

Signed-off-by: Sean Anderson <[email protected]>
---

arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
index 088271d49139..59b289b52a28 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
@@ -224,9 +224,12 @@ rst: syscon@1e60000 {
little-endian;
};

- efuse@1e80000 {
+ sfp: efuse@1e80000 {
compatible = "fsl,ls1028a-sfp";
reg = <0x0 0x1e80000 0x0 0x10000>;
+ clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
+ QORIQ_CLK_PLL_DIV(4)>;
+ clock-names = "sfp";
#address-cells = <1>;
#size-cells = <1>;

--
2.35.1.1320.gc452695387.dirty

2022-04-22 21:07:59

by Michael Walle

Subject: Re: [PATCH 5/8] arm64: dts: ls1028a: Update SFP binding to include clock

Am 2022-04-21 19:56, schrieb Sean Anderson:
> The clocks property is now mandatory. Add it.
>
> Signed-off-by: Sean Anderson <[email protected]>

Reviewed-by: Michael Walle <[email protected]>

> ---
>
> arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> index 088271d49139..59b289b52a28 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> @@ -224,9 +224,12 @@ rst: syscon@1e60000 {
> little-endian;
> };
>
> - efuse@1e80000 {
> + sfp: efuse@1e80000 {
> compatible = "fsl,ls1028a-sfp";
> reg = <0x0 0x1e80000 0x0 0x10000>;
> + clocks = <&clockgen QORIQ_CLK_PLATFORM_PLL
> + QORIQ_CLK_PLL_DIV(4)>;
> + clock-names = "sfp";
> #address-cells = <1>;
> #size-cells = <1>;