Re: [PATCH 0/2] Add bcm2835aux interrupt controller

[Phil Elwell]

On 07/06/2017 21:37, Stefan Wahren wrote:
> Hi Phil,
>
> > Phil Elwell <phil@xxxxxxxxxxxxxxx> hat am 7. Juni 2017 um 13:11 geschrieben:
> >
> >
> > Devices in the AUX block share a common interrupt line, with a register
> > indicating which devices have active IRQs. Expose this as a nested
> > interrupt controller to avoid IRQ sharing problems (easily observed if
> > UART1 and SPI1/2 are enabled simultaneously).
> >
> > The interrupt functionality could arguably be forked off as a separate
> > irqchip driver, but the clock driver has already claimed the AUX_IRQ
> > register so some driver and DT surgery would still be required.
> > Eric Anholt thought that including it here is reasonable, but I'm
> > prepared to split it out if this is considered too hacky.
>
> in order to give the maintainers (e.g. irqchip) a chance, they should be included into CC.

Will do.

> > Phil Elwell (2):
> >   clk: bcm2835: Add AUX interrupt controller
>
> Either way the dt-binding must be updated as a separate patch. So Rob can review it.

Thanks, Stefan - I'll add that to v2.

Phil