All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arm64: zynqmp: Fix irps5401 device nodes
@ 2020-04-13  8:05 Michal Simek
  2020-04-30  7:36 ` Michal Simek
  0 siblings, 1 reply; 2+ messages in thread
From: Michal Simek @ 2020-04-13  8:05 UTC (permalink / raw)
  To: u-boot

- Do not use irps54012 as device node which is not correct.
- Fix addresses of irps5401/u180 on zcu104 revisions.
- Remove clock-cells property. It is PMIC without any clock output.
- Define irps5401 nodes in zynqmp-e-a2197

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynqmp-e-a2197-00-revA.dts | 16 +++++++++++++---
 arch/arm/dts/zynqmp-zcu104-revA.dts     | 10 ++++------
 arch/arm/dts/zynqmp-zcu104-revC.dts     | 10 ++++------
 arch/arm/dts/zynqmp-zcu111-revA.dts     |  9 +++------
 arch/arm/dts/zynqmp-zcu208-revA.dts     |  6 ++----
 arch/arm/dts/zynqmp-zcu216-revA.dts     |  6 ++----
 6 files changed, 28 insertions(+), 29 deletions(-)

diff --git a/arch/arm/dts/zynqmp-e-a2197-00-revA.dts b/arch/arm/dts/zynqmp-e-a2197-00-revA.dts
index 65cf5914945d..bf982e221830 100644
--- a/arch/arm/dts/zynqmp-e-a2197-00-revA.dts
+++ b/arch/arm/dts/zynqmp-e-a2197-00-revA.dts
@@ -198,9 +198,6 @@
 			#size-cells = <0>;
 			reg = <0>;
 			/* u152 IR35215 0x16/0x46 vcc_soc */
-			/* u160 IRPS5401 0x17/0x47 */
-			/* u167 IRPS5401 0x1c/0x4c */
-			/* u175 IRPS5401 0x1d/0x4d */
 			/* u179 ir38164 0x19/0x49 vcco_500 */
 			/* u181 ir38164 0x1a/0x4a vcco_501 */
 			/* u183 ir38164 0x1b/0x4b vcco_502 */
@@ -209,6 +206,19 @@
 			/* u189 ir38164 0x20/0x50 mgtyavtt */
 			/* u194 ir38164 0x13/0x43 vdd1_1v8_lp4 */
 			/* u195 ir38164 0x14/0x44 vdd2_1v8_lp4 */
+
+			irps5401_47: irps5401 at 47 { /* IRPS5401 - u160 */
+				compatible = "infineon,irps5401";
+				reg = <0x47>; /* pmbus / i2c 0x17 */
+			};
+			irps5401_4c: irps5401 at 4c { /* IRPS5401 - u167 */
+				compatible = "infineon,irps5401";
+				reg = <0x4c>; /* pmbus / i2c 0x1c */
+			};
+			irps5401_4d: irps5401 at 4d { /* IRPS5401 - u175 */
+				compatible = "infineon,irps5401";
+				reg = <0x4d>; /* pmbus / i2c 0x1d */
+			};
 		};
 		i2c at 1 { /* PMBUS1_INA226 */
 			#address-cells = <1>;
diff --git a/arch/arm/dts/zynqmp-zcu104-revA.dts b/arch/arm/dts/zynqmp-zcu104-revA.dts
index 3ceb39dce02b..a4bd6b800a18 100644
--- a/arch/arm/dts/zynqmp-zcu104-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu104-revA.dts
@@ -147,15 +147,13 @@
 			#address-cells = <1>;
 			#size-cells = <0>;
 			reg = <2>;
-			irps5401_43: irps54012 at 43 { /* IRPS5401 - u175 */
-				#clock-cells = <0>;
+			irps5401_43: irps5401 at 43 { /* IRPS5401 - u175 */
 				compatible = "infineon,irps5401";
-				reg = <0x43>;
+				reg = <0x43>; /* pmbus / i2c 0x13 */
 			};
-			irps5401_4d: irps54012 at 4d { /* IRPS5401 - u180 */
-				#clock-cells = <0>;
+			irps5401_44: irps5401 at 44 { /* IRPS5401 - u180 */
 				compatible = "infineon,irps5401";
-				reg = <0x4d>;
+				reg = <0x44>; /* pmbus / i2c 0x14 */
 			};
 		};
 
diff --git a/arch/arm/dts/zynqmp-zcu104-revC.dts b/arch/arm/dts/zynqmp-zcu104-revC.dts
index 7dad4523deb0..d4b3769a27cc 100644
--- a/arch/arm/dts/zynqmp-zcu104-revC.dts
+++ b/arch/arm/dts/zynqmp-zcu104-revC.dts
@@ -172,15 +172,13 @@
 			#address-cells = <1>;
 			#size-cells = <0>;
 			reg = <2>;
-			irps5401_43: irps54012 at 43 { /* IRPS5401 - u175 */
-				#clock-cells = <0>;
+			irps5401_43: irps5401 at 43 { /* IRPS5401 - u175 */
 				compatible = "infineon,irps5401";
-				reg = <0x43>;
+				reg = <0x43>; /* pmbus / i2c 0x13 */
 			};
-			irps5401_4d: irps54012 at 4d { /* IRPS5401 - u180 */
-				#clock-cells = <0>;
+			irps5401_44: irps5401 at 44 { /* IRPS5401 - u180 */
 				compatible = "infineon,irps5401";
-				reg = <0x4d>;
+				reg = <0x44>; /* pmbus / i2c 0x14 */
 			};
 		};
 
diff --git a/arch/arm/dts/zynqmp-zcu111-revA.dts b/arch/arm/dts/zynqmp-zcu111-revA.dts
index d16bf8ac7ac7..63e285fe9a9a 100644
--- a/arch/arm/dts/zynqmp-zcu111-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu111-revA.dts
@@ -327,18 +327,15 @@
 			#address-cells = <1>;
 			#size-cells = <0>;
 			reg = <2>;
-			irps5401_43: irps54012 at 43 { /* IRPS5401 - u53 check these */
-				#clock-cells = <0>;
+			irps5401_43: irps5401 at 43 { /* IRPS5401 - u53 check these */
 				compatible = "infineon,irps5401";
 				reg = <0x43>;
 			};
-			irps5401_44: irps54012 at 44 { /* IRPS5401 - u55 */
-				#clock-cells = <0>;
+			irps5401_44: irps5401 at 44 { /* IRPS5401 - u55 */
 				compatible = "infineon,irps5401";
 				reg = <0x44>;
 			};
-			irps5401_45: irps54012 at 45 { /* IRPS5401 - u57 */
-				#clock-cells = <0>;
+			irps5401_45: irps5401 at 45 { /* IRPS5401 - u57 */
 				compatible = "infineon,irps5401";
 				reg = <0x45>;
 			};
diff --git a/arch/arm/dts/zynqmp-zcu208-revA.dts b/arch/arm/dts/zynqmp-zcu208-revA.dts
index 75ecd7a5c203..118a2de96b3c 100644
--- a/arch/arm/dts/zynqmp-zcu208-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu208-revA.dts
@@ -351,13 +351,11 @@
 			/* u112 - ir38164 0x13/0x43 */
 			/* u123 - ir38164 0x1c/0x4c */
 
-			irps5401_44: irps54012 at 44 { /* IRPS5401 - u53 */
-				#clock-cells = <0>;
+			irps5401_44: irps5401 at 44 { /* IRPS5401 - u53 */
 				compatible = "infineon,irps5401";
 				reg = <0x44>; /* i2c addr 0x14 */
 			};
-			irps5401_45: irps54012 at 45 { /* IRPS5401 - u55 */
-				#clock-cells = <0>;
+			irps5401_45: irps5401 at 45 { /* IRPS5401 - u55 */
 				compatible = "infineon,irps5401";
 				reg = <0x45>; /* i2c addr 0x15 */
 			};
diff --git a/arch/arm/dts/zynqmp-zcu216-revA.dts b/arch/arm/dts/zynqmp-zcu216-revA.dts
index f3b5edfeb423..e454bfcba797 100644
--- a/arch/arm/dts/zynqmp-zcu216-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu216-revA.dts
@@ -355,13 +355,11 @@
 			/* u112 - ir38164 0x13/0x43 */
 			/* u123 - ir38164 0x1c/0x4c */
 
-			irps5401_44: irps54012 at 44 { /* IRPS5401 - u53 */
-				#clock-cells = <0>;
+			irps5401_44: irps5401 at 44 { /* IRPS5401 - u53 */
 				compatible = "infineon,irps5401";
 				reg = <0x44>; /* i2c addr 0x14 */
 			};
-			irps5401_45: irps54012 at 45 { /* IRPS5401 - u55 */
-				#clock-cells = <0>;
+			irps5401_45: irps5401 at 45 { /* IRPS5401 - u55 */
 				compatible = "infineon,irps5401";
 				reg = <0x45>; /* i2c addr 0x15 */
 			};
-- 
2.26.0

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [PATCH] arm64: zynqmp: Fix irps5401 device nodes
  2020-04-13  8:05 [PATCH] arm64: zynqmp: Fix irps5401 device nodes Michal Simek
@ 2020-04-30  7:36 ` Michal Simek
  0 siblings, 0 replies; 2+ messages in thread
From: Michal Simek @ 2020-04-30  7:36 UTC (permalink / raw)
  To: u-boot

po 13. 4. 2020 v 10:05 odes?latel Michal Simek <michal.simek@xilinx.com> napsal:
>
> - Do not use irps54012 as device node which is not correct.
> - Fix addresses of irps5401/u180 on zcu104 revisions.
> - Remove clock-cells property. It is PMIC without any clock output.
> - Define irps5401 nodes in zynqmp-e-a2197
>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
>  arch/arm/dts/zynqmp-e-a2197-00-revA.dts | 16 +++++++++++++---
>  arch/arm/dts/zynqmp-zcu104-revA.dts     | 10 ++++------
>  arch/arm/dts/zynqmp-zcu104-revC.dts     | 10 ++++------
>  arch/arm/dts/zynqmp-zcu111-revA.dts     |  9 +++------
>  arch/arm/dts/zynqmp-zcu208-revA.dts     |  6 ++----
>  arch/arm/dts/zynqmp-zcu216-revA.dts     |  6 ++----
>  6 files changed, 28 insertions(+), 29 deletions(-)
>
> diff --git a/arch/arm/dts/zynqmp-e-a2197-00-revA.dts b/arch/arm/dts/zynqmp-e-a2197-00-revA.dts
> index 65cf5914945d..bf982e221830 100644
> --- a/arch/arm/dts/zynqmp-e-a2197-00-revA.dts
> +++ b/arch/arm/dts/zynqmp-e-a2197-00-revA.dts
> @@ -198,9 +198,6 @@
>                         #size-cells = <0>;
>                         reg = <0>;
>                         /* u152 IR35215 0x16/0x46 vcc_soc */
> -                       /* u160 IRPS5401 0x17/0x47 */
> -                       /* u167 IRPS5401 0x1c/0x4c */
> -                       /* u175 IRPS5401 0x1d/0x4d */
>                         /* u179 ir38164 0x19/0x49 vcco_500 */
>                         /* u181 ir38164 0x1a/0x4a vcco_501 */
>                         /* u183 ir38164 0x1b/0x4b vcco_502 */
> @@ -209,6 +206,19 @@
>                         /* u189 ir38164 0x20/0x50 mgtyavtt */
>                         /* u194 ir38164 0x13/0x43 vdd1_1v8_lp4 */
>                         /* u195 ir38164 0x14/0x44 vdd2_1v8_lp4 */
> +
> +                       irps5401_47: irps5401 at 47 { /* IRPS5401 - u160 */
> +                               compatible = "infineon,irps5401";
> +                               reg = <0x47>; /* pmbus / i2c 0x17 */
> +                       };
> +                       irps5401_4c: irps5401 at 4c { /* IRPS5401 - u167 */
> +                               compatible = "infineon,irps5401";
> +                               reg = <0x4c>; /* pmbus / i2c 0x1c */
> +                       };
> +                       irps5401_4d: irps5401 at 4d { /* IRPS5401 - u175 */
> +                               compatible = "infineon,irps5401";
> +                               reg = <0x4d>; /* pmbus / i2c 0x1d */
> +                       };
>                 };
>                 i2c at 1 { /* PMBUS1_INA226 */
>                         #address-cells = <1>;
> diff --git a/arch/arm/dts/zynqmp-zcu104-revA.dts b/arch/arm/dts/zynqmp-zcu104-revA.dts
> index 3ceb39dce02b..a4bd6b800a18 100644
> --- a/arch/arm/dts/zynqmp-zcu104-revA.dts
> +++ b/arch/arm/dts/zynqmp-zcu104-revA.dts
> @@ -147,15 +147,13 @@
>                         #address-cells = <1>;
>                         #size-cells = <0>;
>                         reg = <2>;
> -                       irps5401_43: irps54012 at 43 { /* IRPS5401 - u175 */
> -                               #clock-cells = <0>;
> +                       irps5401_43: irps5401 at 43 { /* IRPS5401 - u175 */
>                                 compatible = "infineon,irps5401";
> -                               reg = <0x43>;
> +                               reg = <0x43>; /* pmbus / i2c 0x13 */
>                         };
> -                       irps5401_4d: irps54012 at 4d { /* IRPS5401 - u180 */
> -                               #clock-cells = <0>;
> +                       irps5401_44: irps5401 at 44 { /* IRPS5401 - u180 */
>                                 compatible = "infineon,irps5401";
> -                               reg = <0x4d>;
> +                               reg = <0x44>; /* pmbus / i2c 0x14 */
>                         };
>                 };
>
> diff --git a/arch/arm/dts/zynqmp-zcu104-revC.dts b/arch/arm/dts/zynqmp-zcu104-revC.dts
> index 7dad4523deb0..d4b3769a27cc 100644
> --- a/arch/arm/dts/zynqmp-zcu104-revC.dts
> +++ b/arch/arm/dts/zynqmp-zcu104-revC.dts
> @@ -172,15 +172,13 @@
>                         #address-cells = <1>;
>                         #size-cells = <0>;
>                         reg = <2>;
> -                       irps5401_43: irps54012 at 43 { /* IRPS5401 - u175 */
> -                               #clock-cells = <0>;
> +                       irps5401_43: irps5401 at 43 { /* IRPS5401 - u175 */
>                                 compatible = "infineon,irps5401";
> -                               reg = <0x43>;
> +                               reg = <0x43>; /* pmbus / i2c 0x13 */
>                         };
> -                       irps5401_4d: irps54012 at 4d { /* IRPS5401 - u180 */
> -                               #clock-cells = <0>;
> +                       irps5401_44: irps5401 at 44 { /* IRPS5401 - u180 */
>                                 compatible = "infineon,irps5401";
> -                               reg = <0x4d>;
> +                               reg = <0x44>; /* pmbus / i2c 0x14 */
>                         };
>                 };
>
> diff --git a/arch/arm/dts/zynqmp-zcu111-revA.dts b/arch/arm/dts/zynqmp-zcu111-revA.dts
> index d16bf8ac7ac7..63e285fe9a9a 100644
> --- a/arch/arm/dts/zynqmp-zcu111-revA.dts
> +++ b/arch/arm/dts/zynqmp-zcu111-revA.dts
> @@ -327,18 +327,15 @@
>                         #address-cells = <1>;
>                         #size-cells = <0>;
>                         reg = <2>;
> -                       irps5401_43: irps54012 at 43 { /* IRPS5401 - u53 check these */
> -                               #clock-cells = <0>;
> +                       irps5401_43: irps5401 at 43 { /* IRPS5401 - u53 check these */
>                                 compatible = "infineon,irps5401";
>                                 reg = <0x43>;
>                         };
> -                       irps5401_44: irps54012 at 44 { /* IRPS5401 - u55 */
> -                               #clock-cells = <0>;
> +                       irps5401_44: irps5401 at 44 { /* IRPS5401 - u55 */
>                                 compatible = "infineon,irps5401";
>                                 reg = <0x44>;
>                         };
> -                       irps5401_45: irps54012 at 45 { /* IRPS5401 - u57 */
> -                               #clock-cells = <0>;
> +                       irps5401_45: irps5401 at 45 { /* IRPS5401 - u57 */
>                                 compatible = "infineon,irps5401";
>                                 reg = <0x45>;
>                         };
> diff --git a/arch/arm/dts/zynqmp-zcu208-revA.dts b/arch/arm/dts/zynqmp-zcu208-revA.dts
> index 75ecd7a5c203..118a2de96b3c 100644
> --- a/arch/arm/dts/zynqmp-zcu208-revA.dts
> +++ b/arch/arm/dts/zynqmp-zcu208-revA.dts
> @@ -351,13 +351,11 @@
>                         /* u112 - ir38164 0x13/0x43 */
>                         /* u123 - ir38164 0x1c/0x4c */
>
> -                       irps5401_44: irps54012 at 44 { /* IRPS5401 - u53 */
> -                               #clock-cells = <0>;
> +                       irps5401_44: irps5401 at 44 { /* IRPS5401 - u53 */
>                                 compatible = "infineon,irps5401";
>                                 reg = <0x44>; /* i2c addr 0x14 */
>                         };
> -                       irps5401_45: irps54012 at 45 { /* IRPS5401 - u55 */
> -                               #clock-cells = <0>;
> +                       irps5401_45: irps5401 at 45 { /* IRPS5401 - u55 */
>                                 compatible = "infineon,irps5401";
>                                 reg = <0x45>; /* i2c addr 0x15 */
>                         };
> diff --git a/arch/arm/dts/zynqmp-zcu216-revA.dts b/arch/arm/dts/zynqmp-zcu216-revA.dts
> index f3b5edfeb423..e454bfcba797 100644
> --- a/arch/arm/dts/zynqmp-zcu216-revA.dts
> +++ b/arch/arm/dts/zynqmp-zcu216-revA.dts
> @@ -355,13 +355,11 @@
>                         /* u112 - ir38164 0x13/0x43 */
>                         /* u123 - ir38164 0x1c/0x4c */
>
> -                       irps5401_44: irps54012 at 44 { /* IRPS5401 - u53 */
> -                               #clock-cells = <0>;
> +                       irps5401_44: irps5401 at 44 { /* IRPS5401 - u53 */
>                                 compatible = "infineon,irps5401";
>                                 reg = <0x44>; /* i2c addr 0x14 */
>                         };
> -                       irps5401_45: irps54012 at 45 { /* IRPS5401 - u55 */
> -                               #clock-cells = <0>;
> +                       irps5401_45: irps5401 at 45 { /* IRPS5401 - u55 */
>                                 compatible = "infineon,irps5401";
>                                 reg = <0x45>; /* i2c addr 0x15 */
>                         };
> --
> 2.26.0
>

Applied.
M

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-04-30  7:36 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-13  8:05 [PATCH] arm64: zynqmp: Fix irps5401 device nodes Michal Simek
2020-04-30  7:36 ` Michal Simek

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.