On Fri, Jul 3, 2020 at 5:53 PM Uwe Kleine-König
<u.kleine-koe...@pengutronix.de> wrote:
> On Sun, May 03, 2020 at 12:54:53PM +0200, Miquel Raynal wrote:
...
> > + ret = regmap_read(pca_chip->regmap, reg, &val);
> > + mutex_unlock(&pca_chip->i2c_lock);
> > + if (ret < 0) {
> > + dev_err(dev, "Cannot retrieve PWM intensity (%d)\n", ret);
>
> Please use %pe for error codes.
%pe is mainly for error pointers, with plain integers it will look
awkward a bit.
> > + return 0;
> > + }
--
With Best Regards,
Andy Shevchenko
