Re: [PATCH v2 1/2] dt-bindings: hwmon: pmbus: add max20830

[Guenter Roeck]

On 4/16/26 18:04, Torreno, Alexis Czezar wrote:
> > > > On the previous version, you got an LLM comment about not having the
> > > > interrupts property amongst other things.
> > > > I think the other things got implemented, but I didn't see any reply
> > > > to the bot about that?
>
> I wasn't sure if it was that type of bot. I'll try replying on the other patch review.
> I just added a note in the cover letter change log about the lacking smbalert.
>
> > > > I think the answer is that it shouldn't because the pin it
> > > > referenced doesn't exist, but when looking at the schematic I have
> > > > to wonder if
> > >
> > > I had to look this up in the datasheet. A SMBus chip with no alert pin
> > > is a bit odd, but you are correct.
> > >
> > > > there should be an interrupts property for dealing with "pgood"?
> > > FWIW, I have never seen that. Normally such pins are used to take
> > > devices out of reset.
> >
> > It's an output on this device seemingly. I don't care if the driver ignores it, but
> > for completeness (and we like completeness with
> > bindings) I think it should be documented as an interrupt or gpio.
>
> Alright, I'll add it as an interrupt: optional power-good signal

Uuh, that really doesn't make any sense. Please at least make it a gpio pin,
matching pwr-good-gpios of ti,tps65185.yaml.

Guenter