[PATCH V3 1/5] dt-bindings: mfd: ti,bq25703a: Add TI BQ25703A Charger
Chris Morgan
macroalpha82 at gmail.com
Wed Jun 25 08:29:29 PDT 2025
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
More information about the Linux-rockchip
mailing list