ARM: dts: aspeed: yosemite4: Remove Floating Falls board nodes
Subsystems (2)
PATCH v2 1/1 ARM: dts: aspeed: yosemite4: Remove Floating Falls board nodes
From: Sean He <[email protected]>
The Floating Falls (FF) board connects to the Sentinel Dome through a connector and carries a PCA9537 GPIO expander at address 0x49 and a FRU EEPROM at address 0x50 on each slot I2C bus. No system currently in use has an FF board, and its connector on the Sentinel Dome is not populated, so these addresses never acknowledge: the pca953x driver fails to probe the eight PCA9537 nodes on every boot and the at24 driver does not bind to the eight EEPROM nodes. Remove the nodes so that the devicetree matches the actual hardware.
---
The changelog was drafted with an AI assistant (Claude Code) and
reviewed by the submitter.
---
Changes in v2:
- Reworded the commit message: the FF board is a separate board, no
system in use has it, and its connector is not populated
- Dropped the verification paragraph
- Link to v1: https://lore.kernel.org/r/[email protected]
--- .../dts/aspeed/aspeed-bmc-facebook-yosemite4.dts | 96 ---------------------- 1 file changed, 96 deletions(-)
arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-yosemite4.dts +0 -96
@@ -604,18 +604,6 @@ imux16: i2c@0 { reg = <0>; #address-cells = <1>; #size-cells = <0>; - gpio@49 { - compatible = "nxp,pca9537"; - reg = <0x49>; - gpio-controller; - #gpio-cells = <2>; - }; - - eeprom@50 { - compatible = "atmel,24c128"; - reg = <0x50>; - }; - eeprom@51 { compatible = "atmel,24c128"; reg = <0x51>;
@@ -631,18 +619,6 @@ imux17: i2c@1 { reg = <1>; #address-cells = <1>; #size-cells = <0>; - gpio@49 { - compatible = "nxp,pca9537"; - reg = <0x49>; - gpio-controller; - #gpio-cells = <2>; - }; - - eeprom@50 { - compatible = "atmel,24c128"; - reg = <0x50>; - }; - eeprom@51 { compatible = "atmel,24c128"; reg = <0x51>;
@@ -658,18 +634,6 @@ imux18: i2c@2 { reg = <2>; #address-cells = <1>; #size-cells = <0>; - gpio@49 { - compatible = "nxp,pca9537"; - reg = <0x49>; - gpio-controller; - #gpio-cells = <2>; - }; - - eeprom@50 { - compatible = "atmel,24c128"; - reg = <0x50>; - }; - eeprom@51 { compatible = "atmel,24c128"; reg = <0x51>;
@@ -685,18 +649,6 @@ imux19: i2c@3 { reg = <3>; #address-cells = <1>; #size-cells = <0>; - gpio@49 { - compatible = "nxp,pca9537"; - reg = <0x49>; - gpio-controller; - #gpio-cells = <2>; - }; - - eeprom@50 { - compatible = "atmel,24c128"; - reg = <0x50>; - }; - eeprom@51 { compatible = "atmel,24c128"; reg = <0x51>;
@@ -726,18 +678,6 @@ imux20: i2c@0 { reg = <0>; #address-cells = <1>; #size-cells = <0>; - gpio@49 { - compatible = "nxp,pca9537"; - reg = <0x49>; - gpio-controller; - #gpio-cells = <2>; - }; - - eeprom@50 { - compatible = "atmel,24c128"; - reg = <0x50>; - }; - eeprom@51 { compatible = "atmel,24c128"; reg = <0x51>;
@@ -753,18 +693,6 @@ imux21: i2c@1 { reg = <1>; #address-cells = <1>; #size-cells = <0>; - gpio@49 { - compatible = "nxp,pca9537"; - reg = <0x49>; - gpio-controller; - #gpio-cells = <2>; - }; - - eeprom@50 { - compatible = "atmel,24c128"; - reg = <0x50>; - }; - eeprom@51 { compatible = "atmel,24c128"; reg = <0x51>;
@@ -780,18 +708,6 @@ imux22: i2c@2 { reg = <2>; #address-cells = <1>; #size-cells = <0>; - gpio@49 { - compatible = "nxp,pca9537"; - reg = <0x49>; - gpio-controller; - #gpio-cells = <2>; - }; - - eeprom@50 { - compatible = "atmel,24c128"; - reg = <0x50>; - }; - eeprom@51 { compatible = "atmel,24c128"; reg = <0x51>;
@@ -807,18 +723,6 @@ imux23: i2c@3 { reg = <3>; #address-cells = <1>; #size-cells = <0>; - gpio@49 { - compatible = "nxp,pca9537"; - reg = <0x49>; - gpio-controller; - #gpio-cells = <2>; - }; - - eeprom@50 { - compatible = "atmel,24c128"; - reg = <0x50>; - }; - eeprom@51 { compatible = "atmel,24c128"; reg = <0x51>;
---
base-commit: 6628e7d4c6b25c8d18f1345e3a20d21829aae7e4
change-id: 20261103-yv4-remove-ff-nodes-9e20104d2362
Best regards,
Signature · 1 line
Eric Yang <[email protected]>