On Sun, Dec 26, 2021 at 8:41 AM Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > > On Sun, Dec 26, 2021 at 1:45 AM Prabhakar Mahadev Lad > <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote: > > > On Wed, Dec 22, 2021 at 2:41 PM Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote: > > ... > > > > > + if (np) { > > > > > > Same comments as per your other patches, i.e. > > > Why is this check here? > > > > > Because the interrupt resource has range of interrupts in one IRQ resource [0]. Let me know if there is any other alternative way to avoid such case. > > Shouldn't be fixed in platform_get_irq_optional() to return IRQ by > index for all cases? > Sorry, I don't get you here. Wasn't your earlier comment for np check? Cheers, Prabhakar