Re: [PATCH 05/15] dt_bindings: mfd: Add ROHM BD71815 PMIC

[Vaittinen, Matti]

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.vaittinen@xxxxxxxxxxxxxxxxx>
> > ---
> >  .../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.vaittinen@xxxxxxxxxxxxxxxxx>
> > +
> > +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