Re: [PATCH 1/3] spmi: pmic-arb: convert to v2 irq interfaces to support hierarchical IRQ chips

From: Brian Masney
Date: Fri Jan 04 2019 - 20:50:00 EST

On Fri, Jan 04, 2019 at 04:25:23PM -0800, Stephen Boyd wrote:
> > + /*
> > + * Check to see if the hwirq is already associated with another virq on
> > + * this IRQ domain. If so, then disassociate it before associating the
> > + * hwirq with the new virq. IRQs are all initially setup without an IRQ
> > + * hierarchy when this driver is probed and when mfd/qcom-spmi-pmic.c is
> > + * probed. Later in the boot process, an IRQ hierarchy is requested by
> > + * pinctrl-spmi-gpio.c, and the same hwirq is now associated with a new
> > + * virq.
> Is that because we count the irqs in the pmic gpio driver?


The call to devm_of_platform_populate() in drivers/mfd/qcom-spmi-pmic.c
also triggers this condition. I'll experiment with this some more this

> Otherwise patch looks good to me. Thanks for working on this. Are you
> going to convert the ssbi master and gpio chip too?

Yes, I will do that work as well once this patch series is accepted.
I currently don't have the hardware to test it, but I'm willing to pick
up a cheap device on ebay. Any suggestions for something that boots a
mainline kernel with the ssbi? It looks like the Sony Xperia Z phone is
one of the supported devices.

Thanks for your review on the other patches. It all makes sense.