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

Andy Shevchenko
2025-10-18T19:30:20Z

On Thu, Sep 25, 2025 at 09:54:19AM +0200, Romain Gantois wrote:
> 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;

THen provide a proper specifier instead. Casting in printf() is in 99.9% cases
just a pure mistake.

--
With Best Regards,
Andy Shevchenko