On Fri, Jun 13, 2014 at 12:26:10PM +0530, Sricharan R wrote: > On Thursday 12 June 2014 07:35 PM, Jason Cooper wrote: ... > > Do you have other changes outside of irqchip depending on this series? > > If so, I can set up a topic branch for you guys to base off of. > > Otherwise, I'll just apply them to irqchip/core when they're ready. > > > There are dts changes which are dependent upon this series. > > http://www.spinics.net/lists/linux-omap/msg108116.html In general, dts changes shouldn't depend on code changes or vice-versa. If they do, that's an indicator that we're breaking compatibility with older dtbs. Looking at the dra7.dtsi changes, we're redefining the interrupt property, which can't be good. :( Perhaps a better solution would be to add a property, say 'ti,cross-irq' that is the exact same format as 'interrupts', but is used by the crossbar driver? I'm not convinced of this yet, I suspect we may not actually have a dependency between the dtsi changes and the code changes. We would have the ugly "if you have the crossbar node, 'interrupts' means X, if not it means Y" in the binding docs. But the absence of the node prevents the crossbar driver from re-interpreting the interrupts property. Have you tried booting all the different scenarios? eg: old dtb, new driver new dtb, old driver old dtb, old driver new dtb, new driver thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
