Re: [PATCH v2 3/5] pwm: dwc: simplify error handling

From: Andy Shevchenko
Date: Thu Feb 08 2024 - 12:22:36 EST


On Thu, Feb 08, 2024 at 12:35:27PM +0530, Raag Jadav wrote:
> Simplify error handling in ->probe() function using dev_err_probe() helper.

..

> + return dev_err_probe(dev, ret, "Failed to enable device (%pe)\n", ERR_PTR(ret));

Have you checked the output?
Note, it will duplicate error codes which we don't want.

..

> + return dev_err_probe(dev, ret, "Failed to iomap PCI BAR (%pe)\n", ERR_PTR(ret));

Ditto.

--
With Best Regards,
Andy Shevchenko