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