RE: [PATCH v5 2/2] regulator: adp5055: Add driver for adp5055

From: Torreno, Alexis Czezar
Date: Tue Apr 08 2025 - 21:36:32 EST




> -----Original Message-----
> From: Mark Brown <broonie@xxxxxxxxxx>
> Sent: Tuesday, April 8, 2025 7:09 PM
> To: Torreno, Alexis Czezar <AlexisCzezar.Torreno@xxxxxxxxxx>
> Cc: Liam Girdwood <lgirdwood@xxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>;
> Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>; Conor Dooley
> <conor+dt@xxxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx;
> devicetree@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH v5 2/2] regulator: adp5055: Add driver for adp5055
>
> [External]
>
> On Tue, Apr 08, 2025 at 12:25:49PM +0800, Alexis Czezar Torreno wrote:
> > Add ADI ADP5055 driver support. The device consists of 3 buck
> > regulators able to connect to high input voltages of up to 18V with no
> > preregulators.
>
> This breaks an x86_64 allmodconfig build with clang-19:
>
> /build/stage/linux/drivers/regulator/adp5055-regulator.c:169:3: error: variable
> 'i' is incremented both in the loop header and in the loop body [-Werror,-Wfor-
> loop-analysis]
> 169 | i++;
> | ^
> /build/stage/linux/drivers/regulator/adp5055-regulator.c:160:34: note:
> incremented here
> 160 | for (i = 0; i < ADP5055_NUM_CH; i++) {
> | ^

Noted on this, will send a quick fix