On Mon, 2021-01-11 at 13:06 -0600, Rob Herring wrote:
> On Fri, Jan 08, 2021 at 03:34:52PM +0200, Matti Vaittinen wrote:
> > Document DT bindings for ROHM BD71815.
> >
> > BD71815 is a single-chip power management IC mainly for battery-
> > powered
> > portable devices. The IC integrates 5 bucks, 7 LDOs, a boost driver
> > for
> > LED, a battery charger with a Coulomb counter, a real-time clock, a
> > 32kHz
> > clock and two general-purpose outputs although only one is
> > documented by
> > the data-sheet.
> >
> > Signed-off-by: Matti Vaittinen <matti.vaitti...@fi.rohmeurope.com>
> > ---
> > .../bindings/mfd/rohm,bd71815-pmic.yaml | 198
> > ++++++++++++++++++
> > 1 file changed, 198 insertions(+)
> > create mode 100644
> > Documentation/devicetree/bindings/mfd/rohm,bd71815-pmic.yaml
> >
> > diff --git a/Documentation/devicetree/bindings/mfd/rohm,bd71815-
> > pmic.yaml b/Documentation/devicetree/bindings/mfd/rohm,bd71815-
> > pmic.yaml
> > new file mode 100644
> > index 000000000000..2206b2008acd
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/mfd/rohm,bd71815-pmic.yaml
> > @@ -0,0 +1,198 @@
> > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/mfd/rohm,bd71815-pmic.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: ROHM BD71815 Power Management Integrated Circuit bindings
> > +
> > +maintainers:
> > + - Matti Vaittinen <matti.vaitti...@fi.rohmeurope.com>
> > +
> > +description: |
> > + BD71815AGW is a single-chip power management ICs for battery-
> > powered
> > + portable devices. It integrates 5 buck converters, 8 LDOs, a
> > boost driver
> > + for LED and a 500 mA single-cell linear charger. Also included
> > is a Coulomb
> > + counter, a real-time clock (RTC), and a 32.768 kHz clock gate
> > and two GPOs.
> > +
> > +properties:
> > + compatible:
> > + const: rohm,bd71815
> > +
> > + reg:
> > + description:
> > + I2C slave address.
> > + maxItems: 1
> > +
> > + interrupts:
> > + maxItems: 1
> > +
> > + gpio-controller: true
>
> Add a blank line here.
>
> > + "#gpio-cells":
> > + const: 2
> > + description: |
> > + The first cell is the pin number and the second cell is used
> > to specify
> > + flags. See ../gpio/gpio.txt for more information.
> > +
> > + clocks:
> > + maxItems: 1
>
> And here.
>
> > + "#clock-cells":
> > + const: 0
>
> And here.
>
> > + clock-output-names:
> > + const: bd71815-32k-out
> > +
> > + rohm,clkout-mode:
> > + description: clk32kout mode. Can be set to "open-drain" or
> > "cmos".
> > + $ref: "/schemas/types.yaml#/definitions/string"
> > + enum:
> > + - open-drain
> > + - cmos
> > +
> > + rohm,charger-sense-resistor-ohms:
> > + minimum: 10000000
> > + maximum: 50000000
> > + description: |
> > + BD71827 and BD71828 have SAR ADC for measuring charging
> > currents.
> > + External sense resistor (RSENSE in data sheet) should be
> > used. If some
> > + other but 30MOhm resistor is used the resistance value
> > should be given
>
> 'something other'
>
> Though this can be expressed as 'default: 30000000'
I guess I'll use both 'something other' and 'default: 30000000' in next
version. 'default: 30000000' is nice for machine parser - but for human
reader the 'something other' is likely to be superior. (No scientific
test conducted to back off this statement though).
Thanks Rob!
Best Regards
Matti Vaittinen
