Re: [PATCH 1/2] dt-bindings: trivial-devices: add lt3074

[Krzysztof Kozlowski <krzk@kernel.org>]

On 06/02/2025 10:05, Encarnacion, Cedric justine wrote:
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzk@kernel.org>
>> Sent: Monday, January 27, 2025 3:52 PM
>> To: Encarnacion, Cedric justine <Cedricjustine.Encarnacion@analog.com>
>> Cc: Rob Herring <robh@kernel.org>; Krzysztof Kozlowski <krzk+dt@kernel.org>;
>> Conor Dooley <conor+dt@kernel.org>; Jean Delvare <jdelvare@suse.com>;
>> Guenter Roeck <linux@roeck-us.net>; Jonathan Corbet <corbet@lwn.net>;
>> Delphine CC Chiu <Delphine_CC_Chiu@wiwynn.com>;
>> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-
>> hwmon@vger.kernel.org; linux-doc@vger.kernel.org; linux-i2c@vger.kernel.org
>> Subject: Re: [PATCH 1/2] dt-bindings: trivial-devices: add lt3074
>>
>> [External]
>>
>> On Fri, Jan 24, 2025 at 11:39:06PM +0800, Cedric Encarnacion wrote:
>>> Add Analog Devices LT3074 Ultralow Noise, High PSRR Dropout Linear
>>> Regulator.
>>
>> Regulator? Then why is it trivial? No enable-gpios? No I2C interface?
>>
>> Best regards,
>> Krzysztof
> 
> I based this driver primarily on the LT7182S dual-channel silent switcher
> regulator. 


I do not understand why the base of driver matters here? If it was based
on LTFOOBARGROUNDHOG1 would it change something? I know neither LT7182S
nor the LTFOOBARGROUNDHOG1. And driver matters here even less - bindings
are about hardware, not given implementation in OS.

> The LT3074 is a single-channel device with basic enable and
> power-good GPIOs, fewer registers, and fewer functionalities than the

So it has GPIOs? And by GPIOs you mean what the SoC/CPU, thus the
operating system, sees as GPIOs or something else?

> LT7182S. Like other PMBus drivers, its GPIOs are not exposed.
> 
> Here are other PMBus regulators/power modules found in trivial-devices
> I also used as reference:
> 
> - infineon,irps5401
> - delta,q54sj108a2


I don't know these devices so still no clue. Please rather explain in
the terms of the hardware, e.g. what this device has or has not. See
also regulator bindings.

Best regards,
Krzysztof