Messages in this thread | | | Subject | Re: [PATCH 3/6] gpio: omap: Simplify with dev_err_probe() | From | Grygorii Strashko <> | Date | Fri, 28 Aug 2020 11:01:00 +0300 |
| |
On 27/08/2020 23:08, Krzysztof Kozlowski wrote: > Common pattern of handling deferred probe can be simplified with > dev_err_probe(). Less code and also it prints the error value. > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> > --- > drivers/gpio/gpio-omap.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/gpio/gpio-omap.c b/drivers/gpio/gpio-omap.c > index 7fbe0c9e1fc1..2dc12f4addbd 100644 > --- a/drivers/gpio/gpio-omap.c > +++ b/drivers/gpio/gpio-omap.c > @@ -1394,10 +1394,7 @@ static int omap_gpio_probe(struct platform_device *pdev) > if (bank->irq <= 0) { > if (!bank->irq) > bank->irq = -ENXIO; > - if (bank->irq != -EPROBE_DEFER) > - dev_err(dev, > - "can't get irq resource ret=%d\n", bank->irq); > - return bank->irq; > + return dev_err_probe(dev, bank->irq, "can't get irq resource\n"); > } > > bank->chip.parent = dev; >
Acked-by: Grygorii Strashko <grygorii.strashko@ti.com>
-- Best regards, grygorii
| |