Re: [PATCH 2/3] dt-bindings: hwmon: pmbus: Support for onsemi's FD5121

[Guenter Roeck]

On 6/24/26 15:36, Selvamani Rajagopal wrote:
> > -----Original Message-----
> > From: Conor Dooley <conor@xxxxxxxxxx>
> > Subject: Re: [PATCH 2/3] dt-bindings: hwmon: pmbus: Support for onsemi's FD5121
> >
> >
> > My point is that what's actually being controlled is missing. Maybe it
> > is obvious to you, but it is not to me. Your nodename in your example is
>
>
> You are right. This chip may not be a "controller" in the traditional sense as it doesn't control anything.
> We can change node naming to sensor or regulator so that it aligns with the convention.

One of the problems here is that the chip datasheet is not public,
so we can not verify what this actually is. The only available
public document appears to be the "onsemi FD512x Ax Digital Controller
User Manual" which describes the chip as follows.

"The FD512x Digital Controller is a programmable device designed
for machine vendors to configure their equipment at the factory."

That really does not explain anything at all, and actually looks like
an AI generated summary with the AI not understanding what it is talking
about. According to the onsemi web page, the chip does not exist,
and it appears that it is not available to buy from any distributors
either.

Guenter

> > > +      fd5121@50 {
> > which doesn't comply with node naming requirements and I wanted to come
> > up with a suggestion for what it should be.
> > I am assuming that its power or voltage that you're controlling so
> > either it should be hwmon@ or regulator@.