Hi Rob, On Fri, 27 Apr 2018 15:47:23 -0500, Rob Herring <robh@xxxxxxxxxx> wrote: > On Sat, Apr 21, 2018 at 03:55:33PM +0200, Miquel Raynal wrote: > > Change the documentation to reflect the new bindings used for Marvell > > ICU. This involves describing each interrupt group as a subnode of the > > ICU node. Each of them having their own compatible. > > Why? > > This breaks compatibility. It does indeed. The problem comes from the inability to handle multiple MSI parents for one device and the fact that the API was not designed in this way at all. I would not risk myself to explain the MSI arcana but I suggest you to have a look at this thread which led to this series: https://www.spinics.net/lists/arm-kernel/msg645115.html Of course I am open to suggestions. Thanks, Miquèl > > > > > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx> > > --- > > .../bindings/interrupt-controller/marvell,icu.txt | 60 ++++++++++++++++------ > > 1 file changed, 43 insertions(+), 17 deletions(-) -- Miquel Raynal, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
