Re: [PATCH 0/3] hwmon: (pmbus) Use PMBUS_REGULATOR_ONE to declare regulators with single output

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 2251 bytes --]

On Mon, Feb 19, 2024 at 07:13:33AM -0800, Guenter Roeck wrote:
> On 2/19/24 06:48, Conor Dooley wrote:
> > On Sat, Feb 17, 2024 at 10:00:30PM +0000, Conor Dooley wrote:
> > > On Wed, Feb 14, 2024 at 11:43:39AM -0800, Guenter Roeck wrote:
> > > > If a chip only provides a single regulator, it should be named 'vout'
> > > > and not 'vout0'. Declare regulators for chips with a single ouput using
> > > > PMBUS_REGULATOR_ONE() to make that happen.
> > > 
> > > If we're gonna do this, we should go ahead and document each of these
> > > regulators. If Naresh sends a v2 for the tda, I could do the others.
> > 
> > As it turns out, the infineon devices in question are not PMICs, but
> > regulators themselves. I think something like the following is what we
> > need. Perhaps Zev or the 9elements guys can comment.
> > 
> IMO The lm25066 patch should be separate.

It should already be a pair of changes - one for the infineon stuff and
another for the ti devices:

> > commit 9cf2671c0b98c0a0dd599fe9152985d1320bf320
> > Author: Conor Dooley <conor.dooley@microchip.com>
> > Date:   Mon Feb 19 14:24:48 2024 +0000
> > 
> >      dt-bindings: regulator: promote infineon buck converters to their own binding
> >      These devices may need to make use of the common regulator properties,
> >      but these are not permitted while only documented in
> >      trivial-devices.yaml
> >      Signed-off-by: Conor Dooley <conor.dooley@microchip.com>

> > commit 597739840d2bc95ebb884d7f5fc4f3cc206d9af6
> > Author: Conor Dooley <conor.dooley@microchip.com>
> > Date:   Mon Feb 19 13:29:12 2024 +0000
> > 
> >      dt-bindings: hwmon/pmbus: ti,lm25066: document regulators
> >      All devices documented in the lm25066 binding are intended for use with
> >      a regulator, be that for purely monitoring purposes (lm25056) or, for
> >      the other devices, as the controller of that regulator. The binding does
> >      not currently allow regulator child nodes, so add one.
> >      Each of these devices interacts with only a single regulator and
> >      documentation refers to it as "Vout", hence the choice of child node
> >      name.
> >      Signed-off-by: Conor Dooley <conor.dooley@microchip.com>


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]