[PATCH] net: xilinx: axienet: Propagate errors from optional IRQ lookup
Bui Duc Phuc
phucduc.bui at gmail.com
Tue Aug 11 23:14:38 PDT 2026
Hi Andrew,
Thank you for your review.
> > platform_get_irq_optional() returns a positive IRQ number on success or
> > a negative error code on failure.
........
> > /* Check for Ethernet core IRQ (optional) */
> > - if (lp->eth_irq <= 0)
> > + if (lp->eth_irq < 0)
> > dev_info(&pdev->dev, "Ethernet core IRQ not defined\n");
>
> What has this change got to do with what is described in the commit
> message? If you think this is a needed change, please put it in a
> commit of its own, with a good commit message.
>
lp->eth_irq is assigned from platform_get_irq_optional(), which returns a
positive IRQ number on success or a negative error code on failure.
Therefore, a value of 0 is not possible here, so changing <= 0 to < 0
is directly related to the error handling described in the commit message.
Best regards,
Phuc
More information about the linux-arm-kernel
mailing list