[PATCH] define binding for fsl mpic interrupt controllers
Yoder Stuart-B08248
B08248 at freescale.com
Wed Jan 19 06:47:25 EST 2011
> -----Original Message-----
> From: Meador Inge [mailto:meador_inge at mentor.com]
> Sent: Monday, January 17, 2011 7:21 PM
> To: Yoder Stuart-B08248
> Cc: linuxppc-dev at lists.ozlabs.org; devicetree-discuss at lists.ozlabs.org;
> Blanchard, Hollis; Meador Inge
> Subject: Re: [PATCH] define binding for fsl mpic interrupt controllers
>
> Hi Stuart,
>
> > From: Stuart Yoder<stuart.yoder at freescale.com>
> >
> > define the binding for compatible = "fsl,mpic", including the
> > definition of 4-cell interrupt specifiers. The 3rd and 4th cells are
> > needed to define additional types of interrupt source outside the
> > "normal"
> > external and internal interrupts in FSL SoCs. Define error interrupt,
> > IPIs, and PIC timer sources.
>
> Recently I posted an RFC concerning a binding for the MPIC [1]. I also
> submitted a patch based on that feedback today (patch subject '[PATCH 1/2]
> powerpc: document the MPIC device tree binding'). You provide much more
> information concerning the interrupt sources and how interrupt specifiers
> are encoded. I had a few more properties listed: 'name', 'device_type',
> 'protected-sources', and 'no-reset'. The 'no-reset'
> property is a proposed addition to the binding, the others are already in
> use.
>
I'm not sure a complete merge into one binding makes sense. The thing that
motivated creating this new binding with 4 cells was a thread from
last year. See:
http://lists.ozlabs.org/pipermail/devicetree-discuss/2010-January/001489.html
Since Freescale had the need to represent additional information
in interrupt specifiers (error ints, timers, ipis), the suggestion
at the end was to add additional cells in a FSL-specific binding.
So this really is a Freescale specific thing and does belong in the
Freescale binding directory.
That being said, if there is consensus on the AMP related properties,
I should probably incorporate those as well-- protected-sources
and no-reset.
I'll separately comment on your mpic binding patch.
Stuart
More information about the devicetree-discuss
mailing list