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