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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <5d2b71a9-a62b-418c-91ae-fa2a195aa27c@kernel.org>
Date: Sun, 9 Feb 2025 13:01:18 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: "Encarnacion, Cedric justine" <Cedricjustine.Encarnacion@...log.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Jean Delvare <jdelvare@...e.com>,
 Guenter Roeck <linux@...ck-us.net>, Jonathan Corbet <corbet@....net>,
 Delphine CC Chiu <Delphine_CC_Chiu@...ynn.com>,
 "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
 "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
 "linux-hwmon@...r.kernel.org" <linux-hwmon@...r.kernel.org>,
 "linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
 "linux-i2c@...r.kernel.org" <linux-i2c@...r.kernel.org>
Subject: Re: [PATCH 1/2] dt-bindings: trivial-devices: add lt3074

On 06/02/2025 10:05, Encarnacion, Cedric justine wrote:
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzk@...nel.org>
>> Sent: Monday, January 27, 2025 3:52 PM
>> To: Encarnacion, Cedric justine <Cedricjustine.Encarnacion@...log.com>
>> Cc: Rob Herring <robh@...nel.org>; Krzysztof Kozlowski <krzk+dt@...nel.org>;
>> Conor Dooley <conor+dt@...nel.org>; Jean Delvare <jdelvare@...e.com>;
>> Guenter Roeck <linux@...ck-us.net>; Jonathan Corbet <corbet@....net>;
>> Delphine CC Chiu <Delphine_CC_Chiu@...ynn.com>;
>> devicetree@...r.kernel.org; linux-kernel@...r.kernel.org; linux-
>> hwmon@...r.kernel.org; linux-doc@...r.kernel.org; linux-i2c@...r.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