Hi Xu, Thank you for the patch. > -----Original Message----- > From: Xu Wang <vulab@xxxxxxxxxxx> > Sent: 05 November 2021 01:55 > To: Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>; jic23@xxxxxxxxxx; lars@xxxxxxxxxx > Cc: linux-iio@xxxxxxxxxxxxxxx; linux-renesas-soc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: [PATCH] iio: adc: rzg2l_adc: Remove unnecessary print function dev_err() > > The print function dev_err() is redundant because > platform_get_irq() already prints an error. > > Signed-off-by: Xu Wang <vulab@xxxxxxxxxxx> > --- > drivers/iio/adc/rzg2l_adc.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> Cheers, Prabhakar > diff --git a/drivers/iio/adc/rzg2l_adc.c b/drivers/iio/adc/rzg2l_adc.c index > 32fbf57c362f..9d5be52bd948 100644 > --- a/drivers/iio/adc/rzg2l_adc.c > +++ b/drivers/iio/adc/rzg2l_adc.c > @@ -506,10 +506,8 @@ static int rzg2l_adc_probe(struct platform_device *pdev) > } > > irq = platform_get_irq(pdev, 0); > - if (irq < 0) { > - dev_err(dev, "no irq resource\n"); > + if (irq < 0) > return irq; > - } > > ret = devm_request_irq(dev, irq, rzg2l_adc_isr, > 0, dev_name(dev), adc); > -- > 2.25.1