[PATCH/RFC] arm64: dts: renesas: falcon: Add I2C EEPROM nodes

[PATCH/RFC] arm64: dts: renesas: falcon: Add I2C EEPROM nodes

[Geert Uytterhoeven]

Add device nodes for all I2C EEPROMs on the Falcon board stack.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
Probably we want to move the sub board nodes to separate .dtsi files.

The EEPROM on the CPU board contains some data.
All other EEPROMs are present, but in pristine state (all ones).
---
 .../boot/dts/renesas/r8a779a0-falcon-cpu.dtsi |  7 ++++++
 .../boot/dts/renesas/r8a779a0-falcon.dts      | 23 +++++++++++++++++++
 2 files changed, 30 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a779a0-falcon-cpu.dtsi b/arch/arm64/boot/dts/renesas/r8a779a0-falcon-cpu.dtsi
index fa284a7260d68251..286a5d5b9b08ff88 100644
--- a/arch/arm64/boot/dts/renesas/r8a779a0-falcon-cpu.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a779a0-falcon-cpu.dtsi
@@ -82,6 +82,13 @@ &i2c0 {
 
 	status = "okay";
 	clock-frequency = <400000>;
+
+	eeprom@50 {
+		compatible = "rohm,br24g01", "atmel,24c01";
+		label = "cpu-board";
+		reg = <0x50>;
+		pagesize = <8>;
+	};
 };
 
 &i2c1 {
diff --git a/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts b/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts
index 5617b81dd7dc3e87..daaa712fafc9d3e2 100644
--- a/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts
+++ b/arch/arm64/boot/dts/renesas/r8a779a0-falcon.dts
@@ -21,6 +21,29 @@ chosen {
 		stdout-path = "serial0:115200n8";
 	};
 };
+
+&i2c0 {
+	eeprom@51 {
+		compatible = "rohm,br24g01", "atmel,24c01";
+		label = "breakout-board";
+		reg = <0x51>;
+		pagesize = <8>;
+	};
+
+	eeprom@52 {
+		compatible = "rohm,br24g01", "atmel,24c01";
+		label = "csi-dsi-sub-board-id";
+		reg = <0x52>;
+		pagesize = <8>;
+	};
+
+	eeprom@53 {
+		compatible = "rohm,br24g01", "atmel,24c01";
+		label = "ethernet-sub-board-id";
+		reg = <0x53>;
+		pagesize = <8>;
+	};
+};
 
 &rwdt {
 	timeout-sec = <60>;
-- 
2.25.1

Re: [PATCH/RFC] arm64: dts: renesas: falcon: Add I2C EEPROM nodes

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 484 bytes --]

On Thu, Jan 28, 2021 at 12:05:27PM +0100, Geert Uytterhoeven wrote:
> Add device nodes for all I2C EEPROMs on the Falcon board stack.

Yes, good idea!
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> Probably we want to move the sub board nodes to separate .dtsi files.

I think so. I think we should add them when we have the subboard files.
Or you create them now with only the EEPROMs, if you feel like it :)

The nodes themselves look good.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]