2022-09-23 08:06:57

by Peng Fan (OSS)

Subject: [PATCH V4 0/2] add anatop bindings and correct dts node

From: Peng Fan <[email protected]>

V4:
Add R-b from DT maintainer
Include dts change

V3:
https://lore.kernel.org/linux-arm-kernel/20220919055904.1659124-1-peng.fa
[email protected]/t/
Add #clock-cells property

V2:
Drop syscon, use clock-controller
Add fsl vendor prefix
Add interrupt property

Peng Fan (2):
dt-bindings: clock: add i.MX8M Anatop
arm64: dts: imx8m: align anatop with bindings

.../bindings/clock/fsl,imx8m-anatop.yaml | 51 +++++++++++++++++++
arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +-
arch/arm64/boot/dts/freescale/imx8mn.dtsi | 6 +--
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 6 +--
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 5 +-
5 files changed, 63 insertions(+), 10 deletions(-)
create mode 100644 Documentation/devicetree/bindings/clock/fsl,imx8m-anatop.yaml

--
2.37.1

2022-09-23 08:25:02

by Peng Fan (OSS)

Subject: [PATCH V4 2/2] arm64: dts: imx8m: align anatop with bindings

From: Peng Fan <[email protected]>

The CCM ANALOG module is used for generate PLLs, align the node
with DT bindings

Signed-off-by: Peng Fan <[email protected]>
---
arch/arm64/boot/dts/freescale/imx8mm.dtsi | 5 +++--
arch/arm64/boot/dts/freescale/imx8mn.dtsi | 6 +++---
arch/arm64/boot/dts/freescale/imx8mp.dtsi | 6 +++---
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 5 +++--
4 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm.dtsi b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
index afb90f59c83c..ea5feb04a0b7 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm.dtsi
@@ -573,9 +573,10 @@ fec_mac_address: mac-address@90 {
};
};

- anatop: anatop@30360000 {
- compatible = "fsl,imx8mm-anatop", "syscon";
+ anatop: clock-controller@30360000 {
+ compatible = "fsl,imx8mm-anatop";
reg = <0x30360000 0x10000>;
+ #clock-cells = <1>;
};

snvs: snvs@30370000 {
diff --git a/arch/arm64/boot/dts/freescale/imx8mn.dtsi b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
index cb2836bfbd95..fc86e7337313 100644
--- a/arch/arm64/boot/dts/freescale/imx8mn.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mn.dtsi
@@ -576,10 +576,10 @@ fec_mac_address: mac-address@90 {
};
};

- anatop: anatop@30360000 {
- compatible = "fsl,imx8mn-anatop", "fsl,imx8mm-anatop",
- "syscon";
+ anatop: clock-controller@30360000 {
+ compatible = "fsl,imx8mn-anatop", "fsl,imx8mm-anatop";
reg = <0x30360000 0x10000>;
+ #clock-cells = <1>;
};

snvs: snvs@30370000 {
diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
index 53493dc7d976..130dec9b23fc 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
@@ -441,10 +441,10 @@ eth_mac2: mac-address@96 {
};
};

- anatop: anatop@30360000 {
- compatible = "fsl,imx8mp-anatop", "fsl,imx8mm-anatop",
- "syscon";
+ anatop: clock-controller@30360000 {
+ compatible = "fsl,imx8mp-anatop", "fsl,imx8mm-anatop";
reg = <0x30360000 0x10000>;
+ #clock-cells = <1>;
};

snvs: snvs@30370000 {
diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
index 19eaa523564d..b14dbf2ffb9a 100644
--- a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mq.dtsi
@@ -605,10 +605,11 @@ fec_mac_address: mac-address@90 {
};
};

- anatop: syscon@30360000 {
- compatible = "fsl,imx8mq-anatop", "syscon";
+ anatop: clock-controller@30360000 {
+ compatible = "fsl,imx8mq-anatop";
reg = <0x30360000 0x10000>;
interrupts = <GIC_SPI 49 IRQ_TYPE_LEVEL_HIGH>;
+ #clock-cells = <1>;
};

snvs: snvs@30370000 {
--
2.37.1

2022-10-24 01:19:34

by Shawn Guo

Subject: Re: [PATCH V4 0/2] add anatop bindings and correct dts node

On Fri, Sep 23, 2022 at 03:49:41PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <[email protected]>
>
> V4:
> Add R-b from DT maintainer
> Include dts change
>
> V3:
> https://lore.kernel.org/linux-arm-kernel/20220919055904.1659124-1-peng.fa
> [email protected]/t/
> Add #clock-cells property
>
> V2:
> Drop syscon, use clock-controller
> Add fsl vendor prefix
> Add interrupt property
>
> Peng Fan (2):
> dt-bindings: clock: add i.MX8M Anatop
> arm64: dts: imx8m: align anatop with bindings

Applied both, thanks!