Received-SPF: pass (google.com: domain of linux-kernel+bounces-214727-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Fri, 14 Jun 2024 11:01:56 +0100
From: Andre Przywara <andre.przywara@arm.com>
To: Lee Jones <lee@kernel.org>, Philippe Simons <simons.philippe@gmail.com>
Cc: Chen-Yu Tsai <wens@csie.org>, Liam Girdwood <lgirdwood@gmail.com>, Mark
 Brown <broonie@kernel.org>, Rob Herring <robh@kernel.org>, Krzysztof
 Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley
 <conor+dt@kernel.org>, devicetree@vger.kernel.org,
 linux-kernel@vger.kernel.org, linux-sunxi@lists.linux.dev, Jernej Skrabec
 <jernej.skrabec@gmail.com>, Samuel Holland <samuel@sholland.org>, Ryan
 Walklin <ryan@testtoast.com>, Chris Morgan <macroalpha82@gmail.com>
Subject: Re: [PATCH v2 2/5] regulator: axp20x: AXP717: fix LDO supply rails
 and off-by-ones
Message-ID: <20240614110156.230f5109@donnerap.manchester.arm.com>
In-Reply-To: <20240614081224.GD2561462@google.com>
References: <20240418000736.24338-1-andre.przywara@arm.com>
	<20240418000736.24338-3-andre.przywara@arm.com>
	<f685b49a-36b1-46c9-a0a3-734bca8bd968@arm.com>
	<20240614081224.GD2561462@google.com>
Organization: ARM
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Fri, 14 Jun 2024 09:12:24 +0100
Lee Jones <lee@kernel.org> wrote:

Hi Lee,

> On Wed, 12 Jun 2024, Andre Przywara wrote:
> 
> > Hi Broonie,
> > 
> > On 18/04/2024 01:07, Andre Przywara wrote:  
> > > The X-Powers AXP717 PMIC has separate input supply pins for each group
> > > of LDOs, so they are not all using the same DCDC1 input, as described
> > > currently.
> > > 
> > > Replace the "supply" member of each LDO description with the respective
> > > group supply name, so that the supply dependencies can be correctly
> > > described in the devicetree.
> > > 
> > > Also fix two off-by-ones in the regulator macros, after some double
> > > checking the numbers against the datasheet. This uncovered a bug in the
> > > datasheet: add a comment to document this.
> > > 
> > > Fixes: d2ac3df75c3a ("regulator: axp20x: add support for the AXP717")
> > > Signed-off-by: Andre Przywara <andre.przywara@arm.com>  
> > 
> > Can you please take (at least) this one patch as a fix for 6.10? Applies
> > cleanly on top of v6.10-rc3. The changes are not super-critical, but worth
> > fixing nevertheless.  
> 
> FWIW, I have no idea what's going on with this set now.
> 
> Once all of the necessary patches have been expedited, please gather
> together what's left and I'll take them via MFD.

Yes, we dropped the ball on this one a bit, apologies.
Broonie just took 2/5, so I will resend 3/5-5/5 as a new series, targeting
6.11.

Cheers,
Andre.