Re: [PATCH] pinctrl: sunxi: dt: number the IRQ banks from the first one that has IRQs
From: Pablo Mazzini
Date: Thu Oct 01 2026 - 16:35:46 EST
Hi Andre, Linus,
The B288 only has IRQ banks for PB, PD, PF and PG (banks 0-3). With the
current code it would need a made-up first interrupt; with this patch it
doesn't.
Which way would you prefer for B288?
Thanks,
Pablo
On Thu, Oct 1, 2026 at 8:28 PM Linus Walleij <linusw@xxxxxxxxxx> wrote:
>
> On Thu, Oct 1, 2026 at 2:31 PM Andre Przywara <andre.przywara@xxxxxxx> wrote:
>
> > So we could revert the DT patch, and take Pablo's code patch, with a
> > Fixes: tag, but that would create some broken combinations meanwhile.
>
> I see it like this was a bug in the DTS file, so this if fine as-is.
>
> > And in any case we should add a word to the binding document, to pin
> > down what's expected here.
>
> Can we enforce it with YAML constraints or is it too open-ended?
>
> Yours,
> Linus Walleij