mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH v3 1/2] dt-bindings: i3c: Add NXP P3H2x4x i3c-hub support
@ 2025-08-08 13:20 Aman Kumar Pandey
  2025-08-08 13:20 ` [PATCH v3 2/2] mfd: i3c: Add driver for NXP P3H2x4x i3c-hub device Aman Kumar Pandey
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Aman Kumar Pandey @ 2025-08-08 13:20 UTC (permalink / raw)
  To: linux-kernel, linux-i3c, alexandre.belloni, krzk+dt, robh,
	conor+dt, devicetree, broonie, lee
  Cc: vikash.bansal, priyanka.jain, shashank.rebbapragada, Frank.Li,
	Aman Kumar Pandey

Add bindings for the NXP P3H2x4x (P3H2440/P3H2441/P3H2840/P3H2841)
multiport I3C hub family. These devices connect to a host via
I3C/I2C/SMBus and allow communication with multiple downstream
peripherals.

Signed-off-by: Aman Kumar Pandey <aman.kumarpandey@nxp.com>
Signed-off-by: Vikash Bansal <vikash.bansal@nxp.com>

---
V1 -> V2: Cleaned up coding style and addressed review comments
V2 -> V3: Added mfd device support for i3c hub and regulator.
---
 .../bindings/mfd/nxp,p3h2840-i3c-hub.yaml     | 320 ++++++++++++++++++
 MAINTAINERS                                   |   8 +
 2 files changed, 328 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/mfd/nxp,p3h2840-i3c-hub.yaml

diff --git a/Documentation/devicetree/bindings/mfd/nxp,p3h2840-i3c-hub.yaml b/Documentation/devicetree/bindings/mfd/nxp,p3h2840-i3c-hub.yaml
new file mode 100644
index 000000000000..490595c94627
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/nxp,p3h2840-i3c-hub.yaml
@@ -0,0 +1,320 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+# Copyright 2025 NXP
+
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mfd/nxp,p3h2840-i3c-hub.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: NXP P3H2X4X I3C HUB
+
+maintainers:
+  - Aman Kumar Pandey <aman.kumarpandey@nxp.com>
+  - Vikash Bansal <vikash.bansal@nxp.com>
+
+description: |
+  P3H2x4x (P3H2440/P3H2441/P3H2840/P3H2841) is a family of multiport I3C
+  hub devices that connect to:-
+  1. A host CPU via I3C/I2C/SMBus bus on upstream side and connect to multiple
+     peripheral devices on the downstream  side.
+  2. Have two Controller Ports which can support either
+     I2C/SMBus or I3C buses and connect to a CPU, BMC or SOC.
+  3. P3H2840/ P3H2841 are 8 port I3C hub with eight I3C/I2C Target Port.
+  4. P3H2440/ P3H2441 are 4 port I3C hub with four I3C/I2C Target Port.
+     Target ports can be configured as I2C/SMBus, I3C or GPIO and connect to
+     peripherals.
+
+allOf:
+  - $ref: i3c.yaml#
+
+properties:
+  compatible:
+   items:
+     - const: nxp,p3h2840
+
+  cp0-ldo-microvolt:
+    description:
+      Selects the voltage for controller Port 0, in microvolts.
+    enum: [1000000, 1100000, 1200000, 1800000]
+    default: 1000000
+
+  cp1-ldo-microvolt:
+    description:
+      Selects the voltage for controller Port 1, in microvolts.
+    enum: [1000000, 1100000, 1200000, 1800000]
+    default: 1000000
+
+  tp0145-ldo-microvolt:
+    description:
+      Selects the voltage for target Port 0/1/4/5, in microvolts.
+    enum: [1000000, 1100000, 1200000, 1800000]
+    default: 1000000
+
+  tp2367-ldo-microvolt:
+    description:
+      Selects the voltage for target Port 2/3/6/7, in microvolts.
+    enum: [1000000, 1100000, 1200000, 1800000]
+    default: 1000000
+
+  tp0145-pullup-ohms:
+    description:
+      Selects the pull up resistance for target Port 0/1/4/5, in ohms.
+    enum: [250, 500, 1000, 2000]
+    default: 500
+
+  tp2367-pullup-ohms:
+    description:
+      Selects the pull up resistance for target Port 2/3/6/7, in ohms.
+    enum: [250, 500, 1000, 2000]
+    default: 500
+
+  cp0-io-strength-ohms:
+    description:
+      Selects the IO drive strength for controller Port 0, in ohms.
+    enum: [20, 30, 40, 50]
+    default: 20
+
+  cp1-io-strength-ohms:
+    description:
+      Selects the IO drive strength for controller Port 1, in ohms.
+    enum: [20, 30, 40, 50]
+    default: 20
+
+  tp0145-io-strength-ohms:
+    description:
+      Selects the IO drive strength for target port 0/1/4/5, in ohms.
+    enum: [20, 30, 40, 50]
+    default: 20
+
+  tp2367-io-strength-ohms:
+    description:
+      Selects the IO drive strength for target port 2/3/6/7, in ohms.
+    enum: [20, 30, 40, 50]
+    default: 20
+
+  cp0-supply:
+    description: Phandle to voltage regulator providing power to controller port 0.
+
+  cp1-supply:
+    description: Phandle to voltage regulator providing power to controller port 1.
+
+  tp0145-supply:
+    description: Phandle to voltage regulator providing power to target port 0/1/4/5.
+
+  tp2367-supply:
+    description: Phandle to voltage regulator providing power to target port 2/3/6/7.
+
+  regulators:
+    type: object
+    additionalProperties: false
+
+    properties:
+      cp0:
+        type: object
+        $ref: /schemas/regulator/regulator.yaml#
+        unevaluatedProperties: false
+
+      cp1:
+        type: object
+        $ref: /schemas/regulator/regulator.yaml#
+        unevaluatedProperties: false
+
+      tp0145:
+        type: object
+        $ref: /schemas/regulator/regulator.yaml#
+        unevaluatedProperties: false
+
+      tp2367:
+        type: object
+        $ref: /schemas/regulator/regulator.yaml#
+        unevaluatedProperties: false
+
+patternProperties:
+  "@[0-7]$":
+    type: object
+    additionalProperties: false
+    description:
+      I3C HUB Target Port child, should be named as target-port@<target-port-id>
+
+    properties:
+      reg:
+        description:
+          The I3C HUB Target Port number.
+        minimum: 0
+        maximum: 0x07
+
+      mode:
+        enum: [i3c, smbus, i2c, gpio]
+        description:
+          I3C HUB Target Port mode setting to control Target Port functionality.
+          As per now it is only supporting SMBus, i2c and i3c( i2c mode will work
+          with hub network).
+
+      pullup-enable:
+        type: boolean
+        description:
+          Enables the on-die pull-up for Target Port.
+
+      ibi-enable:
+        type: boolean
+        description:
+          Enables the IBI for Target Port.
+
+      local-dev:
+        $ref: /schemas/types.yaml#/definitions/uint8-array
+        description:
+          SMBus Target Agent can discard transactions of downstream device and not generate an IBI to upstream I3C Hub
+          Controller Port. Up to 8 device (addresses) can be configured as local Devices.
+
+          This property is optional. If not provided, local device list will empty.
+
+      hub_bridge_en:
+        type: boolean
+        description:
+          Enables the hub network (Controller port -> target port).
+
+    patternProperties:
+      "@[0-9a-f]+$":
+        type: object
+        additionalProperties: false
+        description: |
+          I2C child, should be named: <device-type>@<i2c-address>
+
+        properties:
+          compatible:
+            description:
+              Compatible of the I2C/SMBus downstream device.
+
+          reg:
+            description: |
+              I2C address, Downstream device address which are connected to target port.
+            minimum: 0
+            maximum: 0x7f
+
+        required:
+          - compatible
+          - reg
+
+      "@[0-9a-f]+,[0-9a-f]+$":
+        type: object
+        additionalProperties: false
+        description: |
+          I3C child, should be named: <device-type>@<static-i2c-address>,<i3c-pid>
+
+        properties:
+          reg:
+            items:
+              - items:
+                  - description:
+                      Encodes the static I2C address. Should be 0 if the device does
+                      not have one (0 is not a valid I2C address).
+                    minimum: 0
+                    maximum: 0x7f
+                  - description: |
+                      First half of the Provisioned ID (following the PID
+                      definition provided by the I3C specification).
+
+                      Contains the manufacturer ID left-shifted by 1.
+                  - description: |
+                      Second half of the Provisioned ID (following the PID
+                      definition provided by the I3C specification).
+
+                      Contains the ORing of the part ID left-shifted by 16,
+                      the instance ID left-shifted by 12 and extra information.
+
+          assigned-address:
+            $ref: /schemas/types.yaml#/definitions/uint32
+            minimum: 0x1
+            maximum: 0xff
+            description:
+              Dynamic address to be assigned to this device. In case static address is
+              present (first cell of the reg property != 0), this address is assigned
+              through SETDASA. If static address is not present, this address is assigned
+              through SETNEWDA after assigning a temporary address via ENTDAA.
+
+        required:
+          - reg
+
+unevaluatedProperties: false
+
+examples:
+  - |
+    i3c {
+        #address-cells = <3>;
+        #size-cells = <0>;
+
+        hub@70,236153000c2 {
+            reg = <0x70 0x236 0x3000c2>;
+            #address-cells = <1>;
+            #size-cells = <0>;
+            assigned-address = <0x50>;
+
+            cp0-ldo-microvolt = <1800000>;
+            cp1-ldo-microvolt = <1800000>;
+            tp0145-ldo-microvolt = <1800000>;
+            tp2367-ldo-microvolt = <1800000>;
+            tp0145-pullup-ohm = <1000>;
+            tp2367-pullup-ohm = <1000>;
+            cp0-io-strength-ohm = <50>;
+            cp1-io-strength-ohm = <50>;
+            tp0145-io-strength-ohm = <50>;
+            tp2367-io-strength-ohm = <50>;
+            cp0-supply = <&cp0>;
+            tp0145-supply = <&tp0145>;
+
+            regulators {
+              cp0 {
+                regulator-name = "cp0";
+                regulator-min-microvolt = <1000000>;
+                regulator-max-microvolt = <1800000>;
+              };
+
+              cp1 {
+                regulator-name = "cp1";
+                regulator-min-microvolt = <1000000>;
+                regulator-max-microvolt = <1800000>;
+              };
+
+              tp0145 {
+                regulator-name = "tp0145";
+                regulator-min-microvolt = <1000000>;
+                regulator-max-microvolt = <1800000>;
+              };
+
+              tp2367 {
+                regulator-name = "tp2367";
+                regulator-min-microvolt = <1000000>;
+                regulator-max-microvolt = <1800000>;
+              };
+            };
+
+            target-port@0 {
+                reg = <0x0>;
+                #address-cells = <1>;
+                #size-cells = <0>;
+                mode = "smbus";
+                pullup-enable;
+
+                eeprom@57 {
+                    compatible = "atmel,24c32";
+                    reg = <0x57>;
+                    pagesize = <32>;
+                    wp-gpios = <&gpio2 2 0>;
+                    num-addresses = <8>;
+                };
+            };
+
+            target-port@2 {
+                reg = <0x2>;
+                #address-cells = <1>;
+                #size-cells = <0>;
+                mode = "i3c";
+                pullup-enable;
+
+                sensor@68,39200144004 {
+                  reg = <0x68 0x392 0x144004>;
+                  assigned-address = <0xa>;
+                };
+            };
+        };
+    };
diff --git a/MAINTAINERS b/MAINTAINERS
index 4262e8d833c4..b33e90030188 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15477,6 +15477,14 @@ S:	Maintained
 F:	Documentation/devicetree/bindings/sound/nxp,tfa989x.yaml
 F:	sound/soc/codecs/tfa989x.c
 
+NXP P3H2X4X I3C-HUB DRIVER
+M:	Vikash Bansal <vikash.bansal@nxp.com>
+M:	Aman Kumar Pandey <aman.kumarpandey@nxp.com>
+L:	linux-kernel@vger.kernel.org
+L:	linux-i3c-owner@lists.infradead.org
+S:	Maintained
+F:	Documentation/devicetree/bindings/i3c/p3h2x4x_i3c_hub.yaml
+
 NZXT-KRAKEN2 HARDWARE MONITORING DRIVER
 M:	Jonas Malaco <jonas@protocubo.io>
 L:	linux-hwmon@vger.kernel.org
-- 
2.25.1


^ permalink raw reply	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2025-08-11 12:35 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-08-08 13:20 [PATCH v3 1/2] dt-bindings: i3c: Add NXP P3H2x4x i3c-hub support Aman Kumar Pandey
2025-08-08 13:20 ` [PATCH v3 2/2] mfd: i3c: Add driver for NXP P3H2x4x i3c-hub device Aman Kumar Pandey
2025-08-08 20:38   ` Frank Li
2025-08-11  7:01   ` Krzysztof Kozlowski
2025-08-11  6:56 ` [PATCH v3 1/2] dt-bindings: i3c: Add NXP P3H2x4x i3c-hub support Krzysztof Kozlowski
2025-08-11  7:55 ` Krzysztof Kozlowski
2025-08-11 12:35 ` Rob Herring (Arm)

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®