Re: [PATCH 2/5] pinctrl: keembay: Propagate errors from optional IRQ lookup

From: Andy Shevchenko

Date: Mon Aug 10 2026 - 02:54:22 EST


On Mon, Aug 10, 2026 at 8:32 AM <phucduc.bui@xxxxxxxxx> wrote:
>
> platform_get_irq_optional() returns a positive IRQ number on success or
> a negative error code on failure. For an optional IRQ, -ENXIO indicates
> that no optional IRQ is available, while other errors should be propagated.
>
> Propagate all error codes returned by platform_get_irq_optional() other
> than -ENXIO.

While this looks okay, have you considered actually having the
platform_get_irq_optional() to be optional in this sense?

--
With Best Regards,
Andy Shevchenko