Messages in this thread |  | | Date | Sun, 11 Dec 2022 17:34:15 +0100 | Subject | Re: [PATCH] net: ipa: Remove redundant dev_err() | From | Heiner Kallweit <> |
| |
On 11.12.2022 15:47, Kang Minchul wrote: > Function dev_err() is redundant because platform_get_irq_byname() > already prints an error. > > Signed-off-by: Kang Minchul <tegongkang@gmail.com> > --- > drivers/net/ipa/gsi.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/net/ipa/gsi.c b/drivers/net/ipa/gsi.c > index 55226b264e3c..585cfd3f9ec0 100644 > --- a/drivers/net/ipa/gsi.c > +++ b/drivers/net/ipa/gsi.c > @@ -1967,11 +1967,8 @@ int gsi_init(struct gsi *gsi, struct platform_device *pdev, bool prefetch, > > /* Get the GSI IRQ and request for it to wake the system */ > ret = platform_get_irq_byname(pdev, "gsi"); > - if (ret <= 0) { > - dev_err(gsi->dev, > - "DT error %d getting \"gsi\" IRQ property\n", ret); > + if (ret <= 0)
According to the function description it can't return 0. You can further simplify the code. And you patch should be annotated net-next.
> return ret ? : -EINVAL; > - } > irq = ret; > > ret = request_irq(irq, gsi_isr, 0, "gsi", gsi);
|  |