Re: [PATCH 2/5] pinctrl: keembay: Propagate errors from optional IRQ lookup
Andy Shevchenko <[email protected]>
| Newsgroups | org.kernel.vger.linux-gpio,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <CAHp75Vcb12Ok=yEF_WqKp=yS_EW9SWS=G2+B7Fa7aLH=u-aR0w@mail.gmail.com> |
On Mon, Aug 10, 2026 at 8:32 AM <[email protected]> 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