From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Subject: Re: [PATCH v2 12/16] dt-bindings/interrupt-controller: update Marvell ICU bindings Date: Tue, 5 Jun 2018 22:35:10 +0200 Message-ID: <20180605223510.0249b86c@windsurf> References: <20180522094042.24770-1-miquel.raynal@bootlin.com> <20180522094042.24770-13-miquel.raynal@bootlin.com> <20180605202902.GA8875@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20180605202902.GA8875@rob-hp-laptop> Sender: linux-kernel-owner@vger.kernel.org To: Rob Herring Cc: Miquel Raynal , Thomas Gleixner , Jason Cooper , Marc Zyngier , Catalin Marinas , Will Deacon , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Mark Rutland , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Antoine Tenart , Maxime Chevallier , Nadav Haklai , Haim Boot , Hanna Hawa , linux-kernel@vger.kernel.org List-Id: devicetree@vger.kernel.org Hello, On Tue, 5 Jun 2018 14:29:02 -0600, Rob Herring wrote: > On Tue, May 22, 2018 at 11:40:38AM +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. > > Need to explain why you need to do this and why breaking backwards > compatibility is okay. It does not break backward compatibility. The driver changes keep support for the previous DT binding where there was a single node with no subnodes. The DT binding documentation still documents the legacy binding, because it is still supported, and still works. Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com