Re: [PATCH v3 01/16] dt-bindings: regulator: ROHM BD72720

[Matti Vaittinen]

On 05/11/2025 11:00, Rob Herring (Arm) wrote:
> On Wed, 05 Nov 2025 09:35:59 +0200, Matti Vaittinen wrote:
> > From: Matti Vaittinen <mazziesaccount@xxxxxxxxx>
> >
> > The ROHM BD72720 is a new PMIC with 10 BUCk and 11 LDO regulators.
> >
> > The BD72720 is designed to support using the BUCK10 as a supply for
> > the LDOs 1 to 4. When the BUCK10 is used for this, it can be set to a
> > LDON_HEAD mode. In this mode, the BUCK10 voltage can't be controlled by
> > software, but the voltage is adjusted by PMIC to match the LDO1 .. LDO4
> > voltages with a given offset. Offset can be 50mV .. 300mV and is
> > changeable at 50mV steps.
> >
> > Add 'ldon-head-microvolt' property to denote a board which is designed
> > to utilize the LDON_HEAD mode.
> >
> > All other properties are already existing.
> >
> > Add dt-binding doc for ROHM BD72720 regulators to make it usable.
> >
> > Signed-off-by: Matti Vaittinen <mazziesaccount@xxxxxxxxx>
> >
> > ---
> > Revision history:
> >   v2 => v3:
> >   - drop unnecessary descriptions
> >   - use microvolts for the 'ldon-head' dt-property
> >
> >   RFCv1 => v2:
> >   - No changes
> > ---
> >   .../regulator/rohm,bd72720-regulator.yaml     | 149 ++++++++++++++++++
> >   1 file changed, 149 insertions(+)
> >   create mode 100644 Documentation/devicetree/bindings/regulator/rohm,bd72720-regulator.yaml
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/regulator/rohm,bd72720-regulator.yaml: patternProperties:^buck[1-10]$:properties:rohm,ldon-head-microvolt: '$ref' should not be valid under {'const': '$ref'}
> 	hint: Standard unit suffix properties don't need a type $ref
> 	from schema $id: http://devicetree.org/meta-schemas/core.yaml

Nice! I'm not sure if anyone has said it but these bots are helpful :) I 
forgot the type to ldon-head when switching from -millivolt to 
-microvolt. I'll address this for the next version.

> doc reference errors (make refcheckdocs):

Thanks for including the make -command to the mail! I didn't even know 
about 'refcheckdocs' target.

> Warning: Documentation/devicetree/bindings/regulator/rohm,bd72720-regulator.yaml references a file that doesn't exist: Documentation/devicetree/bindings/mfd/rohm,bd72720-pmic.yaml
> Documentation/devicetree/bindings/regulator/rohm,bd72720-regulator.yaml: Documentation/devicetree/bindings/mfd/rohm,bd72720-pmic.yaml

Hmm. I suppose this is because the MFD binding is added only later in 
the series(?) I suppose we can't help it because the MFD binding 
references the regulator binding as well. So, this is kind of a chicken 
and egg problem?

Yours,
	-- Matti

--
Matti Vaittinen
Linux kernel developer at ROHM Semiconductors
Oulu Finland

~~ When things go utterly wrong vim users can always type :help! ~~