On Thu, Dec 09, 2021 at 08:24:59PM +0300, Sergey Shtylyov wrote: > On 12/9/21 5:59 PM, Andy Shevchenko wrote: > > > platform_get_irq() will print a message when it fails. > > No need to repeat this. > > > > While at it, drop redundant check for 0 as platform_get_irq() spills > > out a big WARN() in such case. > > And? IRQ0 is still returned! :-( It should not be returned in the first place. ... > > - if (!irq) > > - return -EINVAL; > > This is prermature -- let's wait till my patch that stops returning IRQ0 from > platform_get_irq() and friends gets merged.... What patch? Does it fix platform_get_irq_optional()? -- With Best Regards, Andy Shevchenko