Re: [PATCH v2 1/2] dt-bindings: hwmon/pmbus: Add Infineon XDP720/XDP730

[Conor Dooley <conor@kernel.org>]

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

On Fri, May 29, 2026 at 11:12:17AM +0530, ASHISH YADAV wrote:
> From: Ashish Yadav <ashish.yadav@infineon.com>
> 
> Add documentation for the device tree binding of the XDP720/XDP730
> eFuse Controllers.
> 
> Signed-off-by: Ashish Yadav <ashish.yadav@infineon.com>
> ---
>  .../bindings/hwmon/pmbus/infineon,xdp720.yaml | 59 --------------
>  .../bindings/hwmon/pmbus/infineon,xdp730.yaml | 80 +++++++++++++++++++

This rename just bloats out the diff, no thanks. Please do not do this,
certainly not without a justification.
pw-bot: changes-requested

Thanks,
Conor.

>  2 files changed, 80 insertions(+), 59 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/hwmon/pmbus/infineon,xdp720.yaml
>  create mode 100644 Documentation/devicetree/bindings/hwmon/pmbus/infineon,xdp730.yaml
> 
> diff --git a/Documentation/devicetree/bindings/hwmon/pmbus/infineon,xdp720.yaml b/Documentation/devicetree/bindings/hwmon/pmbus/infineon,xdp720.yaml
> deleted file mode 100644
> index 72bc3a5e7139..000000000000
> --- a/Documentation/devicetree/bindings/hwmon/pmbus/infineon,xdp720.yaml
> +++ /dev/null
> @@ -1,59 +0,0 @@
> -# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> -%YAML 1.2
> ----
> -
> -$id: http://devicetree.org/schemas/hwmon/pmbus/infineon,xdp720.yaml#
> -$schema: http://devicetree.org/meta-schemas/core.yaml#
> -
> -title: Infineon XDP720 Digital eFuse Controller
> -
> -maintainers:
> -  - Ashish Yadav <ashish.yadav@infineon.com>
> -
> -description: |
> -  The XDP720 is an eFuse with integrated current sensor and digital
> -  controller. It provides accurate system telemetry (V, I, P, T) and
> -  reports analog current at the IMON pin for post-processing.
> -
> -  Datasheet:
> -     https://www.infineon.com/assets/row/public/documents/24/49/infineon-xdp720-001-datasheet-en.pdf
> -
> -properties:
> -  compatible:
> -    enum:
> -      - infineon,xdp720
> -
> -  reg:
> -    maxItems: 1
> -
> -  infineon,rimon-micro-ohms:
> -    description:
> -      The value of the RIMON resistor, in micro ohms, required to enable
> -      the system overcurrent protection.
> -
> -  vdd-vin-supply:
> -    description:
> -      Supply for the VDD_VIN pin (pin 9), the IC controller power supply.
> -      Typically connected to the input bus (VIN) through a 100 ohm / 100 nF
> -      RC filter.
> -
> -required:
> -  - compatible
> -  - reg
> -  - vdd-vin-supply
> -
> -additionalProperties: false
> -
> -examples:
> -  - |
> -    i2c {
> -        #address-cells = <1>;
> -        #size-cells = <0>;
> -
> -        hwmon@11 {
> -            compatible = "infineon,xdp720";
> -            reg = <0x11>;
> -            vdd-vin-supply = <&vdd_vin>;
> -            infineon,rimon-micro-ohms = <1098000000>;  /* 1.098k ohm */
> -        };
> -    };
> diff --git a/Documentation/devicetree/bindings/hwmon/pmbus/infineon,xdp730.yaml b/Documentation/devicetree/bindings/hwmon/pmbus/infineon,xdp730.yaml
> new file mode 100644
> index 000000000000..fdbc577bccdd
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/hwmon/pmbus/infineon,xdp730.yaml
> @@ -0,0 +1,80 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +
> +$id: http://devicetree.org/schemas/hwmon/pmbus/infineon,xdp730.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Infineon XDP720 / XDP730 Digital eFuse Controllers
> +
> +maintainers:
> +  - Ashish Yadav <ashish.yadav@infineon.com>
> +
> +description: |
> +  The XDP720 and XDP730 are PMBus-compliant digital eFuse controllers
> +  with an integrated current sensor.  They provide accurate system
> +  telemetry (V, I, P, T) and report analog current at the IMON pin for
> +  post-processing.
> +
> +  Both parts share the same PMBus register map and direct-format
> +  coefficients; they differ in the GIMON gain step exposed via the
> +  TELEMETRY_AVG register (bit 10) and in the VDD_VIN pin number
> +  (XDP720: pin 9, XDP730: pin 20).
> +
> +  Datasheets:
> +    - XDP720: https://www.infineon.com/assets/row/public/documents/24/49/infineon-xdp720-001-datasheet-en.pdf
> +    - XDP730: https://www.infineon.com/assets/row/public/documents/24/49/infineon-xdp730-001-datasheet-en.pdf
> +
> +properties:
> +  compatible:
> +    enum:
> +      - infineon,xdp720
> +      - infineon,xdp730
> +
> +  reg:
> +    maxItems: 1
> +
> +  infineon,rimon-micro-ohms:
> +    description:
> +      The value of the RIMON resistor, in micro ohms, required to enable
> +      the system overcurrent protection.
> +
> +  vdd-vin-supply:
> +    description:
> +      Supply for the VDD_VIN pin (XDP720 pin 9, XDP730 pin 20), the IC
> +      controller power supply.  Typically connected to the input bus
> +      (VIN) through a 100 ohm / 100 nF RC filter.
> +
> +required:
> +  - compatible
> +  - reg
> +  - vdd-vin-supply
> +
> +additionalProperties: false
> +
> +examples:
> +  - |
> +    i2c {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        efuse@11 {
> +            compatible = "infineon,xdp720";
> +            reg = <0x11>;
> +            vdd-vin-supply = <&vdd_vin>;
> +            infineon,rimon-micro-ohms = <1098000000>;  /* 1.098k ohm */
> +        };
> +    };
> +
> +  - |
> +    i2c {
> +        #address-cells = <1>;
> +        #size-cells = <0>;
> +
> +        efuse@12 {
> +            compatible = "infineon,xdp730";
> +            reg = <0x12>;
> +            vdd-vin-supply = <&vdd_vin>;
> +            infineon,rimon-micro-ohms = <1098000000>;  /* 1.098k ohm */
> +        };
> +    };
> -- 
> 2.39.5
> 

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