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

[Mark Brown]

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...

Attachment: signature.asc
Description: PGP signature