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

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:
 <PH0PR03MB6351036DF0474029CAC68B05F1B42@PH0PR03MB6351.namprd03.prod.outlook.com>
Date: Wed, 9 Apr 2025 01:36:17 +0000
From: "Torreno, Alexis Czezar" <AlexisCzezar.Torreno@...log.com>
To: Mark Brown <broonie@...nel.org>
CC: Liam Girdwood <lgirdwood@...il.com>, Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: RE: [PATCH v5 2/2] regulator: adp5055: Add driver for adp5055



> -----Original Message-----
> From: Mark Brown <broonie@...nel.org>
> Sent: Tuesday, April 8, 2025 7:09 PM
> To: Torreno, Alexis Czezar <AlexisCzezar.Torreno@...log.com>
> Cc: Liam Girdwood <lgirdwood@...il.com>; Rob Herring <robh@...nel.org>;
> Krzysztof Kozlowski <krzk+dt@...nel.org>; Conor Dooley
> <conor+dt@...nel.org>; linux-kernel@...r.kernel.org;
> devicetree@...r.kernel.org
> 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