Hi, On 02/04/2020 13:33, Tang Bin wrote: > In order to simply code,because platform_get_irq() already has > dev_err() message. > > Signed-off-by: Tang Bin <tangbin@xxxxxxxxxxxxxxxxxxxx> > --- > drivers/crypto/amlogic/amlogic-gxl-core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/crypto/amlogic/amlogic-gxl-core.c b/drivers/crypto/amlogic/amlogic-gxl-core.c > index 9d4ead2f7..37901bd81 100644 > --- a/drivers/crypto/amlogic/amlogic-gxl-core.c > +++ b/drivers/crypto/amlogic/amlogic-gxl-core.c > @@ -252,7 +252,7 @@ static int meson_crypto_probe(struct platform_device *pdev) > > mc->irqs = devm_kcalloc(mc->dev, MAXFLOW, sizeof(int), GFP_KERNEL); > for (i = 0; i < MAXFLOW; i++) { > - mc->irqs[i] = platform_get_irq(pdev, i); > + mc->irqs[i] = platform_get_irq_optional(pdev, i); > if (mc->irqs[i] < 0) { > dev_err(mc->dev, "Cannot get IRQ for flow %d\n", i); > return mc->irqs[i]; > NACK, the irq is not optional, I don't see why platform_get_irq_optional() should be used here and how it could "simply code". Neil