Re: [PATCH 4/5] pinctrl: baytrail: Propagate errors from optional IRQ lookup

From: Andy Shevchenko

Date: Mon Aug 10 2026 - 03:01:21 EST


On Mon, Aug 10, 2026 at 8:32 AM <phucduc.bui@xxxxxxxxx> wrote:
>
> From: bui duc phuc <phucduc.bui@xxxxxxxxx>
>
> 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.

On this platform (ACPI-based) there are no other possibilities to get
something meaningful that can be returned (deferred probe). The rest
of the error codes will actually regress since the user won't have a
pin control driver at all, instead of having it IRQ-less (which is
doubtfully useful, though). Have you considered that scenario?

--
With Best Regards,
Andy Shevchenko