linux-kernel - RE: [PATCH 2/2] hwmon: (pmbus/max3440): add support adpm12160

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] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <PH0PR03MB63517FB4C1D8E8197FBA1316F1D82@PH0PR03MB6351.namprd03.prod.outlook.com>
Date: Thu, 20 Mar 2025 04:40:08 +0000
From: "Torreno, Alexis Czezar" <AlexisCzezar.Torreno@...log.com>
To: Guenter Roeck <linux@...ck-us.net>, Jean Delvare <jdelvare@...e.com>,
        Jonathan Corbet <corbet@....net>,
        Delphine CC Chiu
	<Delphine_CC_Chiu@...ynn.com>
CC: "linux-hwmon@...r.kernel.org" <linux-hwmon@...r.kernel.org>,
        "linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-i2c@...r.kernel.org" <linux-i2c@...r.kernel.org>
Subject: RE: [PATCH 2/2] hwmon: (pmbus/max3440): add support adpm12160

> [External]
> 
> On 3/19/25 20:55, Alexis Czezar Torreno wrote:
> > ASPM12160 is a quarter brick DC/DC Power Module. It is a high
> 
> ASPM12160 or ADPM12160 ?
> 
> > power non-isolated converter capable of delivering a fully regulated
> > 12V, with continuous power level of 1600W with peak power at 2400W for
> > a limited time. Uses PMBus Configuration.
> >
> > Signed-off-by: Alexis Czezar Torreno <alexisczezar.torreno@...log.com>
> 
> I am not convinced that this chip should be supported as part of this driver.
> Unfortunately the datasheet is not public ... or, rather, the chip does not
> officially exist. Sorry, this driver will have to wait until I can have a look into the
> datasheet.
> 
> Guenter

Apology on the typo, it is ADPM12160.
They wanted to upstream this same time as the chip release.
But it makes sense, this upstream should at least be after it releases.

Will attend to the feedback on the other commit (Patch 1/2) for the meantime.

Thank you,
Alexis