RE: [PATCH 1/2] dt-bindings: hwmon: add adi,max42500.yaml

[Libetario, Kent]

> -----Original Message-----
> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
> Sent: Friday, December 27, 2024 4:51 PM
> To: Libetario, Kent <Kent.Libetario@xxxxxxxxxx>
> Cc: linux-hwmon@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; Jean Delvare <jdelvare@xxxxxxxx>; Guenter Roeck
> <linux@xxxxxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Krzysztof Kozlowski
> <krzk+dt@xxxxxxxxxx>; Conor Dooley <conor+dt@xxxxxxxxxx>
> Subject: Re: [PATCH 1/2] dt-bindings: hwmon: add adi,max42500.yaml
> 
> [External]
> 
> On Fri, Dec 20, 2024 at 09:20:02AM +0800, Kent Libetario wrote:
> > Add documentation for devicetree bindings for MAX42500
> >
> > Signed-off-by: Kent Libetario <Kent.Libetario@xxxxxxxxxx>
> 
> Why not adding it to trivial devices? Aren't you missing some properties, like
> VDD, EN and RESET?
> 
> Best regards,
> Krzysztof

Yes, I will update the EN properties as output-only in the host to control and
trigger events to the device.

But the VDD and RESET properties are fixed in the device itself and they cannot
be configured by software or device tree. The RESET is commonly input to the NMI
or RESET pin of the host MCU while the VDD depends only in the configuration of
the hardware circuit connected to the system supply.

Regards,
Kent