Re: [PATCH 4/5] devfreq: Use %pe to print error pointers symbolically

From: Sumeet Pawnikar

Date: Sat Sep 12 2026 - 15:11:05 EST


On Sun, Sep 6, 2026 at 12:00 PM Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote:
>
> On 06/09/2026 08:17, Sumeet Pawnikar wrote:
> > ret = -EPROBE_DEFER;
> > - dev_err(dev, "failed to add devfreq device: %ld\n",
> > - PTR_ERR(drv->devfreq));
> > + dev_err(dev, "failed to add devfreq device: %pe\n",
> > + drv->devfreq);
>
> Instead you should use dev_err_probe and drop the last argument.
>
Sure. I'll do this suggested usage of dev_err_probe in different patch
series for *_probe() functions update.

Planning to post V2 for this original series to use %pe to print the
error pointers symbolically, reverting the change for dev_err under
*_porbe() function.

Thanks,
Sumeet.

>
> Best regards,
> Krzysztof