Re: [PATCH] net: xilinx: axienet: Propagate errors from optional IRQ lookup

Hi Andrew,

Thank you for your review.

quoted

platform_get_irq_optional() returns a positive IRQ number on success or
a negative error code on failure.

........

quoted

      /* 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