Re: [PATCH v3 05/20] dt-bindings: mfd: samsung,s2mpg10: Link s2mpg10-pmic to its regulators
From: André Draszik
Date: Sat Nov 08 2025 - 08:48:16 EST
Thanks Krzysztof for your review!
On Tue, 2025-11-04 at 09:28 +0100, Krzysztof Kozlowski wrote:
> On Mon, Nov 03, 2025 at 07:14:44PM +0000, André Draszik wrote:
> > required:
> > - compatible
> > - interrupts
> > - regulators
> >
> > additionalProperties: false
> > +
> > +allOf:
> > + - if:
> > + properties:
> > + compatible:
> > + contains:
> > + const: samsung,s2mpg10-pmic
> > + then:
> > + properties:
>
> This is not correct now. You do not have other variants here and ref
> should be directly in top level "regulators" part of schema.
The commit message explains why, and with your comment on patch 6 I have changed
this now as you suggest.
Thank you,
Andre