Re: [PATCH v2 1/2] regulator: dt-bindings: add hi3798cv200 regulator

[David Yang]

On Tue, Oct 13, 2026 at 7:16 AM Mark Brown <broonie@xxxxxxxxxx> wrote:
>
> On Mon, Oct 12, 2026 at 11:11:33AM +0200, Krzysztof Kozlowski wrote:
> > On Sat, Oct 10, 2026 at 05:43:39AM +0800, David Yang wrote:
>
> > > +description: |
> > > +  HiSilicon STB SoCs (Hi3798cv200/mv100/mv200) provide PWM outputs in the PMOC
> > > +  block which drive an external converter.
>
> > SoCs rarely provide supplies for themselves (they are rather called
> > power domains in such case) and above wording suggests this is actually
> > something external. It also speaks about PWM but binding has no PWM.
>
> I think this is talking about something like pwm-regulator.c.
>
> > > +        compatible = "hisilicon,hi3798cv200-regulator";
> > > +        reg = <0x8a23018 0x4>;
>
> > And one word is rarely a device, which adds to the confusion.
>
> It is possible for a simple analog block...

Hi, I'm planning to spin up with another version. What should I do
with the binding next?