Re: [PATCH v14 3/8] genirq: Add mechanism to multiplex a single HW IPI

Anup Patel
2022-12-02T05:04:19Z

On Fri, Dec 2, 2022 at 7:39 AM Thomas Gleixner <tglx@linutronix.de> wrote:
>
> Anup!
>
> On Thu, Dec 01 2022 at 23:30, Anup Patel wrote:
> > On Thu, Dec 1, 2022 at 10:50 PM Thomas Gleixner <tglx@linutronix.de> wrote:
> >> > irqchip driver and it is shared by various RISC-V irqchip drivers.
> >>
> >> Sure, but now we have two copies of this. One in the Apple AIC and one
> >> here. The obvious thing to do is:
> >>
> >> 1) Provide generic infrastructure
> >>
> >> 2) Convert AIC to use it
> >
> > Mark Z already has a converted version of AIC driver.
> > https://git.kernel.org/pub/scm/linux/kernel/git/maz/arm-platforms.git/log/?h=irq/ipi-mux
>
> You are submitting generic infrastructure which is supposed to go
> through the relevant maintainer tree and not be funneled through some
> riscv branch.
>
> So obviously this very maintainer asks the predictable question and also
> wants to see the Acked/Tested-by of the folks who maintain apple-AIC.

Not CCing apple-AIC maintainers was purely accidental.

Actually, I had started off ipi-mux with the intent to share across
RISC-V irqchip drivers. Only in recent patch revisions (based on
suggestion from Marc Z), I tried to converge and make it useful
for apple-AIC driver as well. Unfortunately, I don't have apple HW
for testing this so I was hoping someone will help.

Thanks to Marc Z, we now have a patch for apple-AIC driver as well.

>
> How is that supposed to work without integrating this patch into your
> series and without having the apple-AIC folks on cc?

Okay, I will include the apple-AIC patch from Marc Z in the next revision
and add apple-AIC folks in CC.

>
> You did not start kernel development as of yesterday, right?
>
> Thanks,
>
> tglx

Regards,
Anup