Skip to contentSkip to conversation

ARM: dts: aspeed: Add Balcones system

PATCH RESEND Applied Eddie James 3 participants 14 messages Oct 1, 2025 – Oct 7, 2025

Subsystems (5)
lore
Share & tools
lore

Eddie James

author maintainer
#0 lore raw

v7 cover ARM: dts: aspeed: Add Balcones system

Resending this series to iio and hwmon maintainers too. Andrew J has merged the dts changes with the Aspeed tree already.

The Balcones system is similar to Bonnell but with a POWER11 processor.

Changes since v6:
- Remove un-documented fan properties under max31785 nodes
- Change fan-x to fan@x in max31790 documentation.

Changes since v5:
- Add patch to add address and size cells to max31790 document
- Don't remove address and size cells in max31785 fixes change

Changes since v4:
- Add patch to fix max31785 warnings
- Add patch to fix missing Bonnell documentation

Changes since v3:
- Add max31785 to the max31790 document instead of to trivial-devices
- Fix minor formatting in dps310 document

Changes since v2:
- Fix a couple of incorrect i2c addresses
- Document dps310 and max31785 properly
- Drop the UCD binding documentation update, it's been fixed

Changes since v1:
- Add all the ucd9000 driver supported compatible strings
- Fix node ordering in Balcones device tree
- Improve commit message to explain addition of ibm-power11-dual.dtsi

Eddie James (7):
dt-bindings: arm: aspeed: add IBM Balcones board
dt-bindings: arm: aspeed: add IBM Bonnell board
dt-bindings: iio: Add Infineon DPS310 sensor documentation
dt-bindings: hwmon: Move max31785 compatibles to max31790 document
dt-bindings: hwmon: max31790: Use addressed fan nodes
ARM: dts: aspeed: Add Balcones system
ARM: dts: aspeed: Fix max31785 fan properties

 .../bindings/arm/aspeed/aspeed.yaml           |   2 +
 .../devicetree/bindings/hwmon/max31785.txt    |  22 -
 .../bindings/hwmon/maxim,max31790.yaml        |  22 +-
 .../iio/pressure/infineon,dps310.yaml         |  44 +
 .../devicetree/bindings/trivial-devices.yaml  |   2 -
 MAINTAINERS                                   |   1 +
 arch/arm/boot/dts/aspeed/Makefile             |   1 +
 .../dts/aspeed/aspeed-bmc-ibm-balcones.dts    | 609 ++++++++++++++
 .../dts/aspeed/aspeed-bmc-ibm-bonnell.dts     |   4 -
 .../dts/aspeed/aspeed-bmc-ibm-everest.dts     |   8 -
 .../dts/aspeed/aspeed-bmc-ibm-rainier.dts     |  12 -
 .../boot/dts/aspeed/aspeed-bmc-opp-tacoma.dts |  36 -
 .../arm/boot/dts/aspeed/ibm-power11-dual.dtsi | 779 ++++++++++++++++++
 .../arm/boot/dts/aspeed/ibm-power11-quad.dtsi | 769 +----------------
 14 files changed, 1456 insertions(+), 855 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/hwmon/max31785.txt
 create mode 100644 Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
 create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dts
 create mode 100644 arch/arm/boot/dts/aspeed/ibm-power11-dual.dtsi
Signature · 1 line
2.51.0
author maintainer
#1 lore raw

PATCH RESEND v7 1/7 dt-bindings: arm: aspeed: add IBM Balcones board

Document a new AST2600 BMC board for IBM P11 server.

---
 Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml | 1 +
 1 file changed, 1 insertion(+)
Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml +1 -0
index aedefca7cf4a8..b2fcfcc4a3b9a 100644
@@ -93,6 +93,7 @@ properties:
               - facebook,minerva-cmc
               - facebook,santabarbara-bmc
               - facebook,yosemite4-bmc
+              - ibm,balcones-bmc
               - ibm,blueridge-bmc
               - ibm,everest-bmc
               - ibm,fuji-bmc
Signature · 1 line
2.51.0
author maintainer
#2 lore raw

PATCH RESEND v7 2/7 dt-bindings: arm: aspeed: add IBM Bonnell board

In reply to Eddie James

Document the existing AST2600 BMC board for IBM P10 server.

---
 Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml | 1 +
 1 file changed, 1 insertion(+)
Documentation/devicetree/bindings/arm/aspeed/aspeed.yaml +1 -0
index b2fcfcc4a3b9a..9730df98b945a 100644
@@ -95,6 +95,7 @@ properties:
               - facebook,yosemite4-bmc
               - ibm,balcones-bmc
               - ibm,blueridge-bmc
+              - ibm,bonnell-bmc
               - ibm,everest-bmc
               - ibm,fuji-bmc
               - ibm,rainier-bmc
Signature · 1 line
2.51.0
author maintainer
#3 lore raw

PATCH RESEND v7 3/7 dt-bindings: iio: Add Infineon DPS310 sensor documentation

In reply to Eddie James

The DPS310 is a barometric pressure and temperature sensor with an I2C interface. Remove it from trivial-devices.yaml and add its own documentation.

---
 .../iio/pressure/infineon,dps310.yaml         | 44 +++++++++++++++++++
 .../devicetree/bindings/trivial-devices.yaml  |  2 -
 MAINTAINERS                                   |  1 +
 3 files changed, 45 insertions(+), 2 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml new file +44 -0
new file mode 100644
index 0000000000000..7c0782e2a821b
@@ -0,0 +1,44 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/pressure/infineon,dps310.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Infineon DPS310 barometric pressure and temperature sensor
+
+maintainers:
+  - Eddie James <[email protected]>
+
+description:
+  The DPS310 is a barometric pressure and temperature sensor with an I2C
+  interface.
+
+properties:
+  compatible:
+    enum:
+      - infineon,dps310
+
+  reg:
+    maxItems: 1
+
+  "#io-channel-cells":
+    const: 0
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pressure-sensor@76 {
+          compatible = "infineon,dps310";
+          reg = <0x76>;
+          #io-channel-cells = <0>;
+        };
+    };
Documentation/devicetree/bindings/trivial-devices.yaml +0 -2
index 7609acaa752d5..a72b7fabc7034 100644
@@ -127,8 +127,6 @@ properties:
           - ibm,cffps2
             # IBM On-Chip Controller hwmon device
           - ibm,p8-occ-hwmon
-            # Infineon barometric pressure and temperature sensor
-          - infineon,dps310
             # Infineon IR36021 digital POL buck controller
           - infineon,ir36021
             # Infineon IRPS5401 Voltage Regulator (PMIC)
MAINTAINERS +1 -0
index 0c8281ea4cc64..92b9854a0e07d 100644
@@ -12191,6 +12191,7 @@ INFINEON DPS310 Driver
 M:	Eddie James <[email protected]>
 L:	[email protected]
 S:	Maintained
+F:	Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
 F:	drivers/iio/pressure/dps310.c
 
 INFINEON PEB2466 ASoC CODEC
Signature · 1 line
2.51.0
#4 lore raw

Re: [PATCH v7 RESEND 3/7] dt-bindings: iio: Add Infineon DPS310 sensor documentation

Quoted 3 lines · On Wed, 1 Oct 2025 09:44:37 -0500 Eddie James <[email protected]> wrote:

The DPS310 is a barometric pressure and temperature sensor with an I2C interface. Remove it from trivial-devices.yaml and add its own documentation.

Hi Eddie,

Why? I guess you need the #io-channel-cells which trivial devices doesn't allow because you have a consumer driver?

Obviously the binding patch shouldn't mention that, but it could call out that there can be such consumers.

I'd also expect to see some supplies even if the driver doesn't yet explicitly handle them.

Jonathan

Quoted 85 lines
---
 .../iio/pressure/infineon,dps310.yaml         | 44 +++++++++++++++++++
 .../devicetree/bindings/trivial-devices.yaml  |  2 -
 MAINTAINERS                                   |  1 +
 3 files changed, 45 insertions(+), 2 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml new file +44 -0
new file mode 100644
index 0000000000000..7c0782e2a821b
@@ -0,0 +1,44 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/pressure/infineon,dps310.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Infineon DPS310 barometric pressure and temperature sensor
+
+maintainers:
+  - Eddie James <[email protected]>
+
+description:
+  The DPS310 is a barometric pressure and temperature sensor with an I2C
+  interface.
+
+properties:
+  compatible:
+    enum:
+      - infineon,dps310
+
+  reg:
+    maxItems: 1
+
+  "#io-channel-cells":
+    const: 0
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pressure-sensor@76 {
+          compatible = "infineon,dps310";
+          reg = <0x76>;
+          #io-channel-cells = <0>;
+        };
+    };
Documentation/devicetree/bindings/trivial-devices.yaml +0 -2
index 7609acaa752d5..a72b7fabc7034 100644
@@ -127,8 +127,6 @@ properties:
           - ibm,cffps2
             # IBM On-Chip Controller hwmon device
           - ibm,p8-occ-hwmon
-            # Infineon barometric pressure and temperature sensor
-          - infineon,dps310
             # Infineon IR36021 digital POL buck controller
           - infineon,ir36021
             # Infineon IRPS5401 Voltage Regulator (PMIC)
MAINTAINERS +1 -0
index 0c8281ea4cc64..92b9854a0e07d 100644
@@ -12191,6 +12191,7 @@ INFINEON DPS310 Driver
 M:	Eddie James <[email protected]>
 L:	[email protected]
 S:	Maintained
+F:	Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
 F:	drivers/iio/pressure/dps310.c
 
 INFINEON PEB2466 ASoC CODEC
author maintainer
#5 lore raw

Re: [PATCH v7 RESEND 3/7] dt-bindings: iio: Add Infineon DPS310 sensor documentation

Quoted 10 lines · Jonathan Cameron
Quoted 3 lines · On Wed, 1 Oct 2025 09:44:37 -0500 Eddie James <[email protected]> wrote:

The DPS310 is a barometric pressure and temperature sensor with an I2C interface. Remove it from trivial-devices.yaml and add its own documentation.

Hi Eddie,

Why? I guess you need the #io-channel-cells which trivial devices doesn't allow because you have a consumer driver?

Correct.

Quoted 3 lines

Obviously the binding patch shouldn't mention that, but it could call out that there can be such consumers.

OK, I can add that.

Quoted 3 lines

I'd also expect to see some supplies even if the driver doesn't yet explicitly handle them.

You mean in the example section? Sure. I'll send a patch separately since the rest is merged

Thanks,

Eddie

Quoted 88 lines

Jonathan

Quoted 84 lines
---
  .../iio/pressure/infineon,dps310.yaml         | 44 +++++++++++++++++++
  .../devicetree/bindings/trivial-devices.yaml  |  2 -
  MAINTAINERS                                   |  1 +
  3 files changed, 45 insertions(+), 2 deletions(-)

create mode 100644 Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml

Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml new file +44 -0
new file mode 100644
index 0000000000000..7c0782e2a821b
@@ -0,0 +1,44 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/pressure/infineon,dps310.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Infineon DPS310 barometric pressure and temperature sensor
+
+maintainers:
+  - Eddie James <[email protected]>
+
+description:
+  The DPS310 is a barometric pressure and temperature sensor with an I2C
+  interface.
+
+properties:
+  compatible:
+    enum:
+      - infineon,dps310
+
+  reg:
+    maxItems: 1
+
+  "#io-channel-cells":
+    const: 0
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pressure-sensor@76 {
+          compatible = "infineon,dps310";
+          reg = <0x76>;
+          #io-channel-cells = <0>;
+        };
+    };
Documentation/devicetree/bindings/trivial-devices.yaml +0 -2
index 7609acaa752d5..a72b7fabc7034 100644
@@ -127,8 +127,6 @@ properties:
            - ibm,cffps2
              # IBM On-Chip Controller hwmon device
            - ibm,p8-occ-hwmon
-            # Infineon barometric pressure and temperature sensor
-          - infineon,dps310
              # Infineon IR36021 digital POL buck controller
            - infineon,ir36021
              # Infineon IRPS5401 Voltage Regulator (PMIC)
MAINTAINERS +1 -0
index 0c8281ea4cc64..92b9854a0e07d 100644
@@ -12191,6 +12191,7 @@ INFINEON DPS310 Driver
  M:	Eddie James <[email protected]>
  L:	[email protected]
  S:	Maintained
+F:	Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
  F:	drivers/iio/pressure/dps310.c
 
  INFINEON PEB2466 ASoC CODEC
#6 lore raw

Re: [PATCH v7 RESEND 3/7] dt-bindings: iio: Add Infineon DPS310 sensor documentation

Quoted 25 lines · On Mon, 6 Oct 2025 08:44:55 -0500 Eddie James <[email protected]> wrote:
Quoted 10 lines · Jonathan Cameron
Quoted 3 lines · On Wed, 1 Oct 2025 09:44:37 -0500 Eddie James <[email protected]> wrote:

The DPS310 is a barometric pressure and temperature sensor with an I2C interface. Remove it from trivial-devices.yaml and add its own documentation.

Hi Eddie,

Why? I guess you need the #io-channel-cells which trivial devices doesn't allow because you have a consumer driver?

Correct.

Quoted 3 lines

Obviously the binding patch shouldn't mention that, but it could call out that there can be such consumers.

OK, I can add that.

Quoted 1 line
Quoted 6 lines
Quoted 2 lines

I'd also expect to see some supplies even if the driver doesn't yet explicitly handle them.

You mean in the example section? Sure. I'll send a patch separately since the rest is merged

Supplies need listing as actual properties (+ in the example) From a binding point of view they are 'required' if the power needs to be turned on for the chip to work. On actual DT files they might not be there if a fixed supply is in use and someone didn't other providing it. In those cases the regulator code provides a fake regulator to make everything work.

Quoted 95 lines

Thanks,

Eddie

Quoted 88 lines

Jonathan

Quoted 84 lines
---
  .../iio/pressure/infineon,dps310.yaml         | 44 +++++++++++++++++++
  .../devicetree/bindings/trivial-devices.yaml  |  2 -
  MAINTAINERS                                   |  1 +
  3 files changed, 45 insertions(+), 2 deletions(-)

create mode 100644 Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml

Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml new file +44 -0
new file mode 100644
index 0000000000000..7c0782e2a821b
@@ -0,0 +1,44 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/iio/pressure/infineon,dps310.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Infineon DPS310 barometric pressure and temperature sensor
+
+maintainers:
+  - Eddie James <[email protected]>
+
+description:
+  The DPS310 is a barometric pressure and temperature sensor with an I2C
+  interface.
+
+properties:
+  compatible:
+    enum:
+      - infineon,dps310
+
+  reg:
+    maxItems: 1
+
+  "#io-channel-cells":
+    const: 0
+
+required:
+  - compatible
+  - reg
+
+additionalProperties: false
+
+examples:
+  - |
+    i2c {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        pressure-sensor@76 {
+          compatible = "infineon,dps310";
+          reg = <0x76>;
+          #io-channel-cells = <0>;
+        };
+    };
Documentation/devicetree/bindings/trivial-devices.yaml +0 -2
index 7609acaa752d5..a72b7fabc7034 100644
@@ -127,8 +127,6 @@ properties:
            - ibm,cffps2
              # IBM On-Chip Controller hwmon device
            - ibm,p8-occ-hwmon
-            # Infineon barometric pressure and temperature sensor
-          - infineon,dps310
              # Infineon IR36021 digital POL buck controller
            - infineon,ir36021
              # Infineon IRPS5401 Voltage Regulator (PMIC)
MAINTAINERS +1 -0
index 0c8281ea4cc64..92b9854a0e07d 100644
@@ -12191,6 +12191,7 @@ INFINEON DPS310 Driver
  M:	Eddie James <[email protected]>
  L:	[email protected]
  S:	Maintained
+F:	Documentation/devicetree/bindings/iio/pressure/infineon,dps310.yaml
  F:	drivers/iio/pressure/dps310.c
 
  INFINEON PEB2466 ASoC CODEC  
author maintainer
#7 lore raw

PATCH RESEND v7 4/7 dt-bindings: hwmon: Move max31785 compatibles to max31790 document

In reply to Eddie James

The binding for the max31785 is the same as the max31790, so just add some compatible strings for the max31785 chip.

---
 .../devicetree/bindings/hwmon/max31785.txt    | 22 -------------------
 .../bindings/hwmon/maxim,max31790.yaml        |  6 ++++-
 2 files changed, 5 insertions(+), 23 deletions(-)
 delete mode 100644 Documentation/devicetree/bindings/hwmon/max31785.txt
Documentation/devicetree/bindings/hwmon/max31785.txt deleted +0 -22
deleted file mode 100644
index 106e08c56aaa9..0000000000000
@@ -1,22 +0,0 @@
-Bindings for the Maxim MAX31785 Intelligent Fan Controller
-==========================================================
-
-Reference:
-
-https://datasheets.maximintegrated.com/en/ds/MAX31785.pdf
-
-The Maxim MAX31785 is a PMBus device providing closed-loop, multi-channel fan
-management with temperature and remote voltage sensing. Various fan control
-features are provided, including PWM frequency control, temperature hysteresis,
-dual tachometer measurements, and fan health monitoring.
-
-Required properties:
-- compatible     : One of "maxim,max31785" or "maxim,max31785a"
-- reg            : I2C address, one of 0x52, 0x53, 0x54, 0x55.
-
-Example:
-
-        fans@52 {
-                compatible = "maxim,max31785";
-                reg = <0x52>;
-        };
Documentation/devicetree/bindings/hwmon/maxim,max31790.yaml +5 -1
index b1ff496f87f95..b2dc813b1ab44 100644
@@ -20,7 +20,11 @@ description: >
 
 properties:
   compatible:
-    const: maxim,max31790
+    enum:
+      - maxim,max31785
+      - maxim,max31785a
+      - maxim,max31785b
+      - maxim,max31790
 
   reg:
     maxItems: 1
Signature · 1 line
2.51.0
maintainer
#8 lore raw

Re: [PATCH v7 RESEND 4/7] dt-bindings: hwmon: Move max31785 compatibles to max31790 document

Quoted 5 lines · Eddie James

The binding for the max31785 is the same as the max31790, so just add some compatible strings for the max31785 chip.

I'll pick this and the next patch of the series after the commit window closes.

Guenter

author maintainer
#10 lore raw

PATCH RESEND v7 5/7 dt-bindings: hwmon: max31790: Use addressed fan nodes

In reply to Eddie James

Since fan properties can include reg, the fan controller should be able to specify address-cells and size-cells properties and use an addressed fan child node.

---
 .../bindings/hwmon/maxim,max31790.yaml           | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)
Documentation/devicetree/bindings/hwmon/maxim,max31790.yaml +13 -3
index b2dc813b1ab44..558cbd251b0fd 100644
@@ -35,11 +35,17 @@ properties:
   resets:
     maxItems: 1
 
+  '#address-cells':
+    const: 1
+
+  '#size-cells':
+    const: 0
+
   "#pwm-cells":
     const: 1
 
 patternProperties:
-  "^fan-[0-9]+$":
+  "^fan@[0-9]+$":
     $ref: fan-common.yaml#
     unevaluatedProperties: false
 
@@ -60,13 +66,17 @@ examples:
         reg = <0x20>;
         clocks = <&sys_clk>;
         resets = <&reset 0>;
+        #address-cells = <1>;
         #pwm-cells = <1>;
+        #size-cells = <0>;
 
-        fan-0 {
+        fan@0 {
+          reg = <0x0>;
           pwms = <&pwm_provider 1>;
         };
 
-        fan-1 {
+        fan@1 {
+          reg = <0x1>;
           pwms = <&pwm_provider 2>;
         };
       };
Signature · 1 line
2.51.0
author maintainer
#12 lore raw

PATCH RESEND v7 6/7 ARM: dts: aspeed: Add Balcones system

In reply to Eddie James

The Balcones system is similar to Bonnell but with a POWER11 processor. Like POWER10, the POWER11 is a dual-chip module, so a dual chip FSI tree is needed. Therefore, split up the quad chip FSI tree.

---
 arch/arm/boot/dts/aspeed/Makefile             |   1 +
 .../dts/aspeed/aspeed-bmc-ibm-balcones.dts    | 609 ++++++++++++++
 .../arm/boot/dts/aspeed/ibm-power11-dual.dtsi | 779 ++++++++++++++++++
 .../arm/boot/dts/aspeed/ibm-power11-quad.dtsi | 769 +----------------
 4 files changed, 1391 insertions(+), 767 deletions(-)
 create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dts
 create mode 100644 arch/arm/boot/dts/aspeed/ibm-power11-dual.dtsi
arch/arm/boot/dts/aspeed/Makefile +1 -0
index 0f0b5b7076545..b56429777c7ef 100644
@@ -39,6 +39,7 @@ dtb-$(CONFIG_ARCH_ASPEED) += \
 	aspeed-bmc-facebook-yamp.dtb \
 	aspeed-bmc-facebook-yosemitev2.dtb \
 	aspeed-bmc-facebook-yosemite4.dtb \
+	aspeed-bmc-ibm-balcones.dtb \
 	aspeed-bmc-ibm-blueridge.dtb \
 	aspeed-bmc-ibm-bonnell.dtb \
 	aspeed-bmc-ibm-everest.dtb \
arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-balcones.dts new file +609 -0

609 lines not on this page · raw message

arch/arm/boot/dts/aspeed/ibm-power11-dual.dtsi new file +779 -0

779 lines not on this page · raw message

arch/arm/boot/dts/aspeed/ibm-power11-quad.dtsi +2 -767

802 lines not on this page · raw message

Signature · 1 line
2.51.0
author maintainer
#13 lore raw

PATCH RESEND v7 7/7 ARM: dts: aspeed: Fix max31785 fan properties

In reply to Eddie James

Remove non-existant fan properties from max31785 nodes.

---
 .../dts/aspeed/aspeed-bmc-ibm-bonnell.dts     |  4 ---
 .../dts/aspeed/aspeed-bmc-ibm-everest.dts     |  8 -----
 .../dts/aspeed/aspeed-bmc-ibm-rainier.dts     | 12 -------
 .../boot/dts/aspeed/aspeed-bmc-opp-tacoma.dts | 36 -------------------
 4 files changed, 60 deletions(-)
arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-bonnell.dts +0 -4
index 2f5d4075a64ad..a37399ff3cea2 100644
@@ -277,15 +277,11 @@ max31785@52 {
 		#size-cells = <0>;
 
 		fan0: fan@0 {
-			compatible = "pmbus-fan";
 			reg = <0>;
-			tach-pulses = <2>;
 		};
 
 		fan1: fan@1 {
-			compatible = "pmbus-fan";
 			reg = <1>;
-			tach-pulses = <2>;
 		};
 	};
 
arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-everest.dts +0 -8
index 9f144f527f03b..5a0975d52492c 100644
@@ -2066,27 +2066,19 @@ max31785@52 {
 				reg = <0x52>;
 
 				fan@0 {
-					compatible = "pmbus-fan";
 					reg = <0>;
-					tach-pulses = <2>;
 				};
 
 				fan@1 {
-					compatible = "pmbus-fan";
 					reg = <1>;
-					tach-pulses = <2>;
 				};
 
 				fan@2 {
-					compatible = "pmbus-fan";
 					reg = <2>;
-					tach-pulses = <2>;
 				};
 
 				fan@3 {
-					compatible = "pmbus-fan";
 					reg = <3>;
-					tach-pulses = <2>;
 				};
 			};
 
arch/arm/boot/dts/aspeed/aspeed-bmc-ibm-rainier.dts +0 -12
index c5fb5d410001b..e90421bf7e3af 100644
@@ -1080,39 +1080,27 @@ max: max31785@52 {
 		#size-cells = <0>;
 
 		fan0: fan@0 {
-			compatible = "pmbus-fan";
 			reg = <0>;
-			tach-pulses = <2>;
 		};
 
 		fan1: fan@1 {
-			compatible = "pmbus-fan";
 			reg = <1>;
-			tach-pulses = <2>;
 		};
 
 		fan2: fan@2 {
-			compatible = "pmbus-fan";
 			reg = <2>;
-			tach-pulses = <2>;
 		};
 
 		fan3: fan@3 {
-			compatible = "pmbus-fan";
 			reg = <3>;
-			tach-pulses = <2>;
 		};
 
 		fan4: fan@4 {
-			compatible = "pmbus-fan";
 			reg = <4>;
-			tach-pulses = <2>;
 		};
 
 		fan5: fan@5 {
-			compatible = "pmbus-fan";
 			reg = <5>;
-			tach-pulses = <2>;
 		};
 	};
 
arch/arm/boot/dts/aspeed/aspeed-bmc-opp-tacoma.dts +0 -36
index b31eb8e58c6b4..6fe7023599e88 100644
@@ -481,55 +481,19 @@ max31785@52 {
 		#size-cells = <0>;
 
 		fan@0 {
-			compatible = "pmbus-fan";
 			reg = <0>;
-			tach-pulses = <2>;
-			maxim,fan-rotor-input = "tach";
-			maxim,fan-pwm-freq = <25000>;
-			maxim,fan-dual-tach;
-			maxim,fan-no-watchdog;
-			maxim,fan-no-fault-ramp;
-			maxim,fan-ramp = <2>;
-			maxim,fan-fault-pin-mon;
 		};
 
 		fan@1 {
-			compatible = "pmbus-fan";
 			reg = <1>;
-			tach-pulses = <2>;
-			maxim,fan-rotor-input = "tach";
-			maxim,fan-pwm-freq = <25000>;
-			maxim,fan-dual-tach;
-			maxim,fan-no-watchdog;
-			maxim,fan-no-fault-ramp;
-			maxim,fan-ramp = <2>;
-			maxim,fan-fault-pin-mon;
 		};
 
 		fan@2 {
-			compatible = "pmbus-fan";
 			reg = <2>;
-			tach-pulses = <2>;
-			maxim,fan-rotor-input = "tach";
-			maxim,fan-pwm-freq = <25000>;
-			maxim,fan-dual-tach;
-			maxim,fan-no-watchdog;
-			maxim,fan-no-fault-ramp;
-			maxim,fan-ramp = <2>;
-			maxim,fan-fault-pin-mon;
 		};
 
 		fan@3 {
-			compatible = "pmbus-fan";
 			reg = <3>;
-			tach-pulses = <2>;
-			maxim,fan-rotor-input = "tach";
-			maxim,fan-pwm-freq = <25000>;
-			maxim,fan-dual-tach;
-			maxim,fan-no-watchdog;
-			maxim,fan-no-fault-ramp;
-			maxim,fan-ramp = <2>;
-			maxim,fan-fault-pin-mon;
 		};
 	};
 
Signature · 1 line
2.51.0