From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f49.google.com (mail-wr1-f49.google.com [209.85.221.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 1E25C3FE369 for ; Thu, 9 Jul 2026 08:53:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783587200; cv=none; b=PStPVeAAoQwc5w5pERHTWkpxIhR8JvhC/r2gJQs9rJi2RTApgrXPvPsDeiJPF3BFZhy6s4nm3TwBMAQONC9K8GYVd8P6bV9TpTBmCyY83myaqh0RJtqUKscPQA/LxgmeLAK0GTcDPXTLMk7CSGz4VR1/2/CH470Q/CG69zk05Ag= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783587200; c=relaxed/simple; bh=gJH9guXcAfNJ6SsL+eO4i7xE/NDXFjDkDL6cAwmmE40=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=cb4qR/gGhXoNsz/swklGj/iLKwiDWOdIlA4cCspmEitHNhjNWjpZD56SD9x1aLyk2vTARBwkY6x8s0qjMIKmoInfsbMtNMGhEe1LEVw86OPxSHVHNk054HID7mUz3+hTy4BWxBikUPgcIUoa1l5RRuOlUpwUX9KUEzlgUtGbaKE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=jksxRTYV; arc=none smtp.client-ip=209.85.221.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="jksxRTYV" Received: by mail-wr1-f49.google.com with SMTP id ffacd0b85a97d-47de008b020so328129f8f.1 for ; Thu, 09 Jul 2026 01:53:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783587197; x=1784191997; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :content-type:mime-version:references:message-id:subject:cc:to:from :date:from:to:cc:subject:date:message-id:reply-to:content-type; bh=W3WCN+xm9Ku357SP4qkc2nLAhmJTaNSJBwAXmI/hZS4=; b=jksxRTYVH0bOAxQiQT5u/DZMQ87Z5qlltniQc8dywmvycj0i62hJ82UpiEgeDT5lGF fjJdfEoRk6maqbwwKW+gitrKyTMdZcsMT4k1KouHq3RMglWYp7QmJti6wyqQTjboYLTN uz6Ol20xtiYyiA49JlWk220H5bL/spsLxhpkHG7+sSD6X0xTV+HdemhFyq3QRaKgwBWB Dr9b5PVzl+ZQCPzw1OR8pl1xfm25sqEUZVCzDHik84Q38dMYZe6naoR/k0v1erEHHW+z A6ZcQ819qVKNcva8PgADxK5dwwlwDMRXeVW3fhmZs9Nd2x/l/W7wGkTEwdBillnXvNGI 7uqQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783587197; x=1784191997; h=in-reply-to:content-transfer-encoding:content-disposition :content-type:mime-version:references:message-id:subject:cc:to:from :date:x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to:content-type; bh=W3WCN+xm9Ku357SP4qkc2nLAhmJTaNSJBwAXmI/hZS4=; b=INHc7PqieI/VcZgv5aR4LEahD3nkhmjV7xL/I/d924PSk1qvSqJh9F3mE0rFHQ+SNG uOYrY/Ypheo/J7H7Vg1X/EUZbtIIdir0uZedBq8lHgxC2X8Ds4ahDiaGohnPAua8KIrQ 2HD6dYbn9LCWz0KAcGZOpIMhI8mYZ3b+UxqcudVmJybtNE/xr+kWWBAhcSvnEcebnQ0c IFGkbP9OWtvlI15UanZe5+JyuTWhyzRlIHT5sS7QVwo/gX1Wq58Yd+ldnfVBN0OArNqO iuSZIT6VmBdcJnbX0bL7ohv9avQEd6eusbEhtHkybFdHiLOzk99CbyHrMP5dhCy7+eEC P96A== X-Forwarded-Encrypted: i=1; AHgh+RqQYABnKb/vjEQTkkdIJjBs9ugDOLekyaY6uCIhDEdqdkKYRqgbjtR2xny5s65L4dc7EfLlvdgltdeU2h8=@vger.kernel.org X-Gm-Message-State: AOJu0Ywf0xHsp2ePw7avTtnJU2wgwpbxYh+ir3t1Q3Ot4nc37u5D95LO 6n0TyjVelqbOkBZ1/MNr2NXmUnk7F+O2cxtSwy/WAh44LK9aKNIZoPoF X-Gm-Gg: AfdE7ckvIzS4rMPNxYX9dDZsu5Q4L7Q8N7kL/TGmUWGsZYd6D9eGE826P36TEUAuHCB EGZSETzci+CEmb+/1vrsUfl62LDr2ds2pGr0Yxd5E4TJ9lDClttpZ3TFZLR0XwRsHG8hJAAbVe9 WlsWJmqwA9DrtDDgBICvJ0q06aaeJBhKd8tOH6LUinK1tOgkABYXoVUhsP8eHR+MJQmBCPKHGXV 8NU2RuUtGV0umQG2gKdbjpPpLEe4OC9aeUFs7bIwI5loP2JamKyyhT3MOHutjSjq7SBNaPDiRZ4 fX6InNDIFiCeNYb9bQQds1mb7idZuMEDFysF4sBlV3vNb/jVz2ekY1Q2+naWYJVKAFEy5RTY+yT EMP/7M186F4oDC9d7+7EjPDu1pkLzvUPREgPCJ6l1jk/p6kY278ZOei+PKfUkeJ9z+W3doDenQu qN8t02 X-Received: by 2002:a05:6000:2509:b0:475:da0e:744d with SMTP id ffacd0b85a97d-47df754dbc2mr2043290f8f.8.1783587197256; Thu, 09 Jul 2026 01:53:17 -0700 (PDT) Received: from nsa ([148.63.225.166]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-47aa039ae44sm47621073f8f.23.2026.07.09.01.53.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Jul 2026 01:53:16 -0700 (PDT) Date: Thu, 9 Jul 2026 09:54:22 +0100 From: Nuno =?utf-8?B?U8Oh?= To: Fred Chen Cc: "Torreno, Alexis Czezar" , Guenter Roeck , Krzysztof Kozlowski , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jonathan Corbet , Shuah Khan , Jonathan Cameron , Wensheng Wang , Frank Li , Brian Chiang , Cosmo Chou , Dixit Parmar , Eddie James , Antoni Pokusinski , Thorsten Blum , Ashish Yadav , Syed Arif , ChiShih Tsai , Abdurrahman Hussain , "Paller, Kim Seer" , Colin Huang , Yuxi Wang , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-hwmon@vger.kernel.org" , "linux-doc@vger.kernel.org" Subject: Re: [PATCH 2/2] hwmon: (pmbus) Add driver for Analog Devices MAX20912 and MAX20916 Message-ID: References: <20260707122701.751878-1-fredchen.openbmc@gmail.com> <20260707122701.751878-3-fredchen.openbmc@gmail.com> <20260708-true-carp-of-champagne-a0dcca@quoll> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Thu, Jul 09, 2026 at 03:23:55PM +0800, Fred Chen wrote: > Torreno, Alexis Czezar 於 2026年7月9日週四 上午8:59寫道： > > > > > On 7/8/26 04:32, Nuno Sá wrote: > > > > On Wed, Jul 08, 2026 at 12:50:25PM +0200, Krzysztof Kozlowski wrote: > > > >> On Wed, Jul 08, 2026 at 10:19:56AM +0100, Nuno Sá wrote: > > > >>> On Tue, Jul 07, 2026 at 06:52:48AM -0700, Guenter Roeck wrote: > > > >>>> On 7/7/26 05:26, Fred Chen wrote: > > > >>>>> Add support for the Analog Devices MAX20912 and MAX20916 > > > >>>>> dual-output multiphase voltage regulators with PMBus interfaces. > > > >>>>> > > > >>>>> Signed-off-by: Fred Chen > > > >>>> > > > >>>> Please provide evidence that those chips actually exist. > > > >>>> Internet search comes up blank. I'll need confirmation from someone > > > >>>> at Analog. > > > >>> > > > >>> Hi Guenter, > > > >>> > > > >>> Well, in fact I'm in the middle of preparing a series that adds > > > >>> support > > > >>> for: > > > >>> > > > >>> "max20826" > > > >>> "max20855b" > > > >>> "max20908" > > > >>> "max20912" > > > >>> "max20916" > > > >>> > > > >>> All the above parts have the datasheet under NDA. But before we had > > > >>> a one page "datasheet" in analog.com but I guess that is gone! For > > > >>> context I mainly did the base (core) driver for max20826 and then > > > >>> Alexis added the other ones. > > > >>> > > > >>> Not sure how to proceed... I can wait and then work on top of what > > > >>> Fred has but this patch is very minimal when compared with what we have. > > > >>> Like: > > > >>> > > > >>> * No regulator support; > > > >>> * No direct mode. The chip has two ways to access registers (paging > > > >>> and > > > >>> * direct mode). > > > >>> * No way to count how many phases we have or if RAIL_B (func[1]) is > > > >>> being used at all. > > > >>> > > > >>> > > > >>> Some other things more intriguing is that these chips, as far as I'm > > > >>> aware (at least for max20826), always have bit 2 set in > > > >>> ON_OFF_CONFIG so a gpio vout. Also we needed some special handling > > > >>> to read phase current which I'm not seeing in the driver. So I would > > > >>> like to understand how the chip was tested? > > > >>> > > > >>> Anyways, if Fred is ok with it I can just finish what I'm doing and > > > >>> send the patches. It would make sense to have something more > > > >>> complete on submission but I don't want to just "steal" the work already > > > done. > > > >> > > > >> Binding should be in such case posted complete, so probably not a > > > >> trivial device. > > > > > > > > Yes, in my series bindings are not in trivial as we support the enable > > > > gpios. > > > > > > > > > > Same question I asked before: What is your use case ? > > > > > > > Hi Guenter, > > > > Unlike my MAX20830 patches, for some reason these chips (not only Nuno's max20826) > > hardwired the bit 2 in ON_OFF_CONFIG to '1'. This makes the use of gpio to enable the > > device a requirement. (as of the latest info given to us) > > > > Regards, > > Alexis > > Hi Nuno, > > Based on the MAX20912/16 specs on my hand, these chips do not support > PMBUS_PHASE (0x04). Furthermore, the spec only indicates support for VID mode > and does not provide m/b/r. Therefore, some of the features you mentioned might > be specific to the MAX20826 series. I see, phases are not supported using standard PMBUS. > > Regarding enabling VOUT via GPIO, our platform handles this via the CPLD as > part of the hardware power sequencing. Managing this pin through the driver is > not a requirement for our system. But we cannot assume all systems will behave like the above. But now i do wonder about controlling the GPIOs in the driver. In your system you clearly did not need to do it. In mine (testing with a rpi) I had to use a GPIO (well I could have used hogs or pinctrl). But if you control the pin you do gain the ability to turn off the regulator. If you don't it's always on (which might be indeed the bulk of the real usecases for these systems). > > As for the testing, I have verified that the telemetry readings for both Rail A > and Rail B behave as expected. The driver reports valid data after system > power-on, and the readings drop to 0 properly after power-off. > > It's possible my specs are outdated or that I'm missing some use cases. If > you plan to submit your more complete driver covering the MAX20912/16 > soon, I'm more than willing to hold off on sending v2 patch. > Sure, soon enough :) - Nuno Sá