Re: [PATCH 4/4] regulator: ltm8054: Support output current limit control

Romain Gantois
2025-09-25T07:54:27Z

[-- Attachment #1: Type: text/plain, Size: 661 bytes --]

On Tuesday, 16 September 2025 16:27:25 CEST Romain Gantois wrote:
> On Tuesday, 16 September 2025 15:19:16 CEST Andy Shevchenko wrote:
> > On Tue, Sep 16, 2025 at 12:24:09PM +0200, Romain Gantois wrote:
...
> > > CTL pin voltage */ + vdac_uV = (u64)min_uA * LTM8054_MAX_CTL_V;
> > > + do_div(vdac_uV, priv->max_uA);
> > > +
> > > + dev_dbg(&rdev->dev,
> > > + "Setting current limit to %duA, CTL pin to %duV\n", min_uA,
> > > (int)vdac_uV);
> >
> > Why casting?
>
> This one is indeed unnecessary.

My mistake, this cast is required to avoid a compiler warning;

Thanks,

--
Romain Gantois, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]