[PATCH V3 1/5] dt-bindings: mfd: ti,bq25703a: Add TI BQ25703A Charger

[Chris Morgan]

On Wed, Jun 25, 2025 at 09:48:30AM +0200, Krzysztof Kozlowski wrote:
> On Mon, Jun 23, 2025 at 11:22:19AM -0500, Chris Morgan wrote:
> > From: Chris Morgan <macromorgan at hotmail.com>
> > 
> > Document the Texas instruments BQ25703A series of charger managers/
> > buck/boost regulators.
> > 
> > Signed-off-by: Chris Morgan <macromorgan at hotmail.com>
> > ---
> >  .../devicetree/bindings/mfd/ti,bq25703a.yaml  | 121 ++++++++++++++++++
> >  1 file changed, 121 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/mfd/ti,bq25703a.yaml
> > 
> > diff --git a/Documentation/devicetree/bindings/mfd/ti,bq25703a.yaml b/Documentation/devicetree/bindings/mfd/ti,bq25703a.yaml
> > new file mode 100644
> > index 000000000000..0727f24b8e54
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/mfd/ti,bq25703a.yaml
> > @@ -0,0 +1,121 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/mfd/ti,bq25703a.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: BQ25703A Charger Manager/Buck/Boost Converter
> > +
> > +maintainers:
> > +  - Chris Morgan <macromorgan at hotmail.com>
> > +
> > +allOf:
> > +  - $ref: /schemas/power/supply/power-supply.yaml#
> > +
> > +properties:
> > +  compatible:
> > +    const: ti,bq25703a
> > +
> > +  reg:
> > +    const: 0x6b
> > +
> > +  interrupts:
> > +    maxItems: 1
> > +
> > +  power-supplies: true
> 
> Drop, redundant.

Acknowledged.

> 
> > +
> > +  monitored-battery:
> > +    description:
> > +      The phandle for a simple-battery connected to this gauge.
> 
> This part is redundant. It cannot be anything else, according to
> power-supply.yaml
> 

Acknowleged.

> > +      A minimum of constant-charge-current-max-microamp,
> > +      constant-charge-voltage-max-microvolt, and
> > +      voltage-min-design-microvolt are required.
> 
> This is fine.
> 
> > +
> > +  input-current-limit-microamp:
> > +    description:
> > +      Maximum total input current allowed used for both charging and
> > +      powering the device.
> > +    minimum: 50000
> > +    maximum: 6400000
> > +    default: 3250000
> > +
> > +  regulators:
> > +    type: object
> > +    additionalProperties: false
> > +    description:
> > +      Boost converter regulator output of bq257xx.
> > +
> > +    properties:
> > +      "usb-otg-vbus":
> 
> Drop quotes. Is this the name of regulator called in datasheet? To which
> pin this corresponds to? I cannot find anything on usb-otg or usb_otg.
> 

This corresponds to the VBUS pin. VBUS is an input usually but becomes
an output when OTG output voltage and OTG output current are set; OTG
enable is set, and a GPIO connected to EN_OTG is set high (per section
8.3.3 of the bq25703a technical docs).

> 
> > +        type: object
> > +        $ref: /schemas/regulator/regulator.yaml
> > +
> > +        properties:
> > +          regulator-name: true
> > +          regulator-min-microamp:
> > +            minimum: 0
> > +            maximum: 6350000
> > +          regulator-max-microamp:
> > +            minimum: 0
> > +            maximum: 6350000
> > +          regulator-min-microvolt:
> > +            minimum: 4480000
> > +            maximum: 20800000
> > +          regulator-max-microvolt:
> > +            minimum: 4480000
> > +            maximum: 20800000
> > +          enable-gpios:
> > +            description:
> > +              The BQ25703 may require both a register write and a GPIO
> > +              toggle to enable the boost regulator.
> > +
> > +        additionalProperties: false
> 
> Please place it after $ref.
> 
> > +
> > +        required:
> > +          - regulator-name
> > +          - regulator-min-microamp
> > +          - regulator-max-microamp
> > +          - regulator-min-microvolt
> > +          - regulator-max-microvolt
> > +
> > +unevaluatedProperties: false
> > +
> > +required:
> > +  - compatible
> > +  - reg
> > +  - input-current-limit-microamp
> > +  - monitored-battery
> > +  - power-supplies
> 
> Keep the same order as in properties.
> 
> Best regards,
> Krzysztof
> 

Let me know if you have more questions about the VBUS, otherwise I'll
make the requested changes and resubmit.

Thank you.

-Chris