Hello, On Sat, 28 Apr 2018 12:42:04 +0200, Miquel Raynal wrote: > 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. Well, yes and no: your driver supports both the old and new binding, so the compatibility is not broken. So, yes the binding is changed in an incompatible way, but the driver continues to support the old binding. Perhaps the binding documentation should continue to document the legacy binding, mark it as still supported but deprecated. Best regards, Thomas -- Thomas Petazzoni, CTO, 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
