2020-05-18 13:08:55

by Anson Huang

Subject: [PATCH 1/2] ARM: dts: imx50: Add src node interrupt

Interrupt is a required property according to SRC binding, add
it for SRC node.

Signed-off-by: Anson Huang <[email protected]>
---
arch/arm/boot/dts/imx50.dtsi | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/imx50.dtsi b/arch/arm/boot/dts/imx50.dtsi
index d325658..567dcc5 100644
--- a/arch/arm/boot/dts/imx50.dtsi
+++ b/arch/arm/boot/dts/imx50.dtsi
@@ -336,6 +336,7 @@
src: src@53fd0000 {
compatible = "fsl,imx50-src", "fsl,imx51-src";
reg = <0x53fd0000 0x4000>;
+ interrupts = <75>;
#reset-cells = <1>;
};

--
2.7.4

2020-05-18 13:09:10

by Anson Huang

Subject: [PATCH 2/2] ARM: dts: imx5: make src node name generic

Node name should be generic, use "reset-controller" instead of "src" for
i.MX5 SoCs src nodes.

Signed-off-by: Anson Huang <[email protected]>
---
- If failed to apply this patch series, try them to be based on
https://patchwork.kernel.org/patch/11541935/ series, I should have
sent them in same series, sorry for that.
---
arch/arm/boot/dts/imx50.dtsi | 2 +-
arch/arm/boot/dts/imx51.dtsi | 2 +-
arch/arm/boot/dts/imx53.dtsi | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/boot/dts/imx50.dtsi b/arch/arm/boot/dts/imx50.dtsi
index 567dcc5..30726bf 100644
--- a/arch/arm/boot/dts/imx50.dtsi
+++ b/arch/arm/boot/dts/imx50.dtsi
@@ -333,7 +333,7 @@
status = "disabled";
};

- src: src@53fd0000 {
+ src: reset-controller@53fd0000 {
compatible = "fsl,imx50-src", "fsl,imx51-src";
reg = <0x53fd0000 0x4000>;
interrupts = <75>;
diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi
index 3f1e913..d3583aa 100644
--- a/arch/arm/boot/dts/imx51.dtsi
+++ b/arch/arm/boot/dts/imx51.dtsi
@@ -439,7 +439,7 @@
status = "disabled";
};

- src: src@73fd0000 {
+ src: reset-controller@73fd0000 {
compatible = "fsl,imx51-src";
reg = <0x73fd0000 0x4000>;
interrupts = <75>;
diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi
index 0d06dbd..afa57bf 100644
--- a/arch/arm/boot/dts/imx53.dtsi
+++ b/arch/arm/boot/dts/imx53.dtsi
@@ -588,7 +588,7 @@
status = "disabled";
};

- src: src@53fd0000 {
+ src: reset-controller@53fd0000 {
compatible = "fsl,imx53-src", "fsl,imx51-src";
reg = <0x53fd0000 0x4000>;
interrupts = <75>;
--
2.7.4

2020-05-19 01:57:45

by Aisheng Dong

Subject: RE: [PATCH 1/2] ARM: dts: imx50: Add src node interrupt

> From: Anson Huang <[email protected]>
> Sent: Monday, May 18, 2020 8:54 PM
>
> Interrupt is a required property according to SRC binding, add it for SRC node.
>
> Signed-off-by: Anson Huang <[email protected]>

Reviewed-by: Dong Aisheng <[email protected]>

Regards
Aisheng

2020-05-19 02:00:34

by Aisheng Dong

Subject: RE: [PATCH 2/2] ARM: dts: imx5: make src node name generic

> From: Anson Huang <[email protected]>
> Sent: Monday, May 18, 2020 8:54 PM
> To: [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]
> Cc: dl-linux-imx <[email protected]>
> Subject: [PATCH 2/2] ARM: dts: imx5: make src node name generic
>
> Node name should be generic, use "reset-controller" instead of "src" for
> i.MX5 SoCs src nodes.
>
> Signed-off-by: Anson Huang <[email protected]>

Reviewed-by: Dong Aisheng <[email protected]>

Regards
Aisheng

2020-05-20 02:58:37

by Shawn Guo

Subject: Re: [PATCH 1/2] ARM: dts: imx50: Add src node interrupt

On Mon, May 18, 2020 at 08:54:19PM +0800, Anson Huang wrote:
> Interrupt is a required property according to SRC binding, add
> it for SRC node.
>
> Signed-off-by: Anson Huang <[email protected]>

Applied both, thanks.