From mboxrd@z Thu Jan 1 00:00:00 1970 From: miquel.raynal@bootlin.com (Miquel Raynal) Date: Sat, 28 Apr 2018 12:42:04 +0200 Subject: [PATCH 13/17] dt-bindings/interrupt-controller: update Marvell ICU bindings In-Reply-To: <20180427204723.hz2ef5f7fhaid3aj@rob-hp-laptop> References: <20180421135537.24716-1-miquel.raynal@bootlin.com> <20180421135537.24716-14-miquel.raynal@bootlin.com> <20180427204723.hz2ef5f7fhaid3aj@rob-hp-laptop> Message-ID: <20180428124204.3f305405@xps13> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Rob, On Fri, 27 Apr 2018 15:47:23 -0500, Rob Herring 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 > > --- > > .../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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Miquel Raynal Subject: Re: [PATCH 13/17] dt-bindings/interrupt-controller: update Marvell ICU bindings Date: Sat, 28 Apr 2018 12:42:04 +0200 Message-ID: <20180428124204.3f305405@xps13> References: <20180421135537.24716-1-miquel.raynal@bootlin.com> <20180421135537.24716-14-miquel.raynal@bootlin.com> <20180427204723.hz2ef5f7fhaid3aj@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20180427204723.hz2ef5f7fhaid3aj@rob-hp-laptop> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Rob Herring Cc: Mark Rutland , Andrew Lunn , Jason Cooper , devicetree@vger.kernel.org, Marc Zyngier , Catalin Marinas , Gregory Clement , Haim Boot , Will Deacon , Maxime Chevallier , Nadav Haklai , Antoine Tenart , Thomas Petazzoni , Thomas Gleixner , Hanna Hawa , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth List-Id: devicetree@vger.kernel.org SGkgUm9iLAoKT24gRnJpLCAyNyBBcHIgMjAxOCAxNTo0NzoyMyAtMDUwMCwgUm9iIEhlcnJpbmcg PHJvYmhAa2VybmVsLm9yZz4gd3JvdGU6Cgo+IE9uIFNhdCwgQXByIDIxLCAyMDE4IGF0IDAzOjU1 OjMzUE0gKzAyMDAsIE1pcXVlbCBSYXluYWwgd3JvdGU6Cj4gPiBDaGFuZ2UgdGhlIGRvY3VtZW50 YXRpb24gdG8gcmVmbGVjdCB0aGUgbmV3IGJpbmRpbmdzIHVzZWQgZm9yIE1hcnZlbGwKPiA+IElD VS4gVGhpcyBpbnZvbHZlcyBkZXNjcmliaW5nIGVhY2ggaW50ZXJydXB0IGdyb3VwIGFzIGEgc3Vi bm9kZSBvZiB0aGUKPiA+IElDVSBub2RlLiBFYWNoIG9mIHRoZW0gaGF2aW5nIHRoZWlyIG93biBj b21wYXRpYmxlLiAgCj4gCj4gV2h5Pwo+IAo+IFRoaXMgYnJlYWtzIGNvbXBhdGliaWxpdHkuCgpJ dCBkb2VzIGluZGVlZC4KClRoZSBwcm9ibGVtIGNvbWVzIGZyb20gdGhlIGluYWJpbGl0eSB0byBo YW5kbGUgbXVsdGlwbGUgTVNJIHBhcmVudHMKZm9yIG9uZSBkZXZpY2UgYW5kIHRoZSBmYWN0IHRo YXQgdGhlIEFQSSB3YXMgbm90IGRlc2lnbmVkIGluIHRoaXMgd2F5CmF0IGFsbC4gSSB3b3VsZCBu b3QgcmlzayBteXNlbGYgdG8gZXhwbGFpbiB0aGUgTVNJIGFyY2FuYSBidXQgSSBzdWdnZXN0Cnlv dSB0byBoYXZlIGEgbG9vayBhdCB0aGlzIHRocmVhZCB3aGljaCBsZWQgdG8gdGhpcyBzZXJpZXM6 CgpodHRwczovL3d3dy5zcGluaWNzLm5ldC9saXN0cy9hcm0ta2VybmVsL21zZzY0NTExNS5odG1s CgpPZiBjb3Vyc2UgSSBhbSBvcGVuIHRvIHN1Z2dlc3Rpb25zLgoKVGhhbmtzLApNaXF1w6hsCiAK PiAKPiA+IAo+ID4gU2lnbmVkLW9mZi1ieTogTWlxdWVsIFJheW5hbCA8bWlxdWVsLnJheW5hbEBi b290bGluLmNvbT4KPiA+IC0tLQo+ID4gIC4uLi9iaW5kaW5ncy9pbnRlcnJ1cHQtY29udHJvbGxl ci9tYXJ2ZWxsLGljdS50eHQgIHwgNjAgKysrKysrKysrKysrKysrKy0tLS0tLQo+ID4gIDEgZmls ZSBjaGFuZ2VkLCA0MyBpbnNlcnRpb25zKCspLCAxNyBkZWxldGlvbnMoLSkgIAoKCgotLSAKTWlx dWVsIFJheW5hbCwgQm9vdGxpbiAoZm9ybWVybHkgRnJlZSBFbGVjdHJvbnMpCkVtYmVkZGVkIExp bnV4IGFuZCBLZXJuZWwgZW5naW5lZXJpbmcKaHR0cHM6Ly9ib290bGluLmNvbQoKX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KbGludXgtYXJtLWtlcm5lbCBt YWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9s aXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtYXJtLWtlcm5lbAo=