From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753193AbdFUQhR convert rfc822-to-8bit (ORCPT ); Wed, 21 Jun 2017 12:37:17 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:40001 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753163AbdFUQhO (ORCPT ); Wed, 21 Jun 2017 12:37:14 -0400 Date: Wed, 21 Jun 2017 17:18:09 +0200 From: Thomas Petazzoni To: Marc Zyngier Cc: Thomas Gleixner , Jason Cooper , linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Rob Herring , Ian Campbell , Pawel Moll , Mark Rutland , Kumar Gala , Andrew Lunn , Sebastian Hesselbarth , Gregory Clement , linux-arm-kernel@lists.infradead.org, Nadav Haklai , Hanna Hawa , Yehuda Yitschak , Antoine Tenart , =?UTF-8?B?TWlxdcOobA==?= Raynal Subject: Re: [PATCH v5 0/6] Add support for the ICU unit in Marvell Armada 7K/8K Message-ID: <20170621171809.34bcce1e@windsurf.home> In-Reply-To: References: <20170621132917.14187-1-thomas.petazzoni@free-electrons.com> Organization: Free Electrons X-Mailer: Claws Mail 3.14.1 (GTK+ 2.24.31; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, On Wed, 21 Jun 2017 16:14:21 +0100, Marc Zyngier wrote: > > Thomas Petazzoni (6): > > dt-bindings: interrupt-controller: add DT binding for the Marvell GICP > > dt-bindings: interrupt-controller: add DT binding for the Marvell ICU > > irqchip: irq-mvebu-gicp: new driver for Marvell GICP > > irqchip: irq-mvebu-icu: new driver for Marvell ICU > > arm64: marvell: enable ICU and GICP drivers > > arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K > > > > .../bindings/interrupt-controller/marvell,gicp.txt | 27 ++ > > .../bindings/interrupt-controller/marvell,icu.txt | 51 ++++ > > arch/arm64/Kconfig.platforms | 2 + > > arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 7 + > > .../boot/dts/marvell/armada-cp110-master.dtsi | 59 +++-- > > .../arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 55 ++-- > > drivers/irqchip/Kconfig | 6 + > > drivers/irqchip/Makefile | 2 + > > drivers/irqchip/irq-mvebu-gicp.c | 279 ++++++++++++++++++++ > > drivers/irqchip/irq-mvebu-gicp.h | 12 + > > drivers/irqchip/irq-mvebu-icu.c | 289 +++++++++++++++++++++ > > .../dt-bindings/interrupt-controller/mvebu-icu.h | 15 ++ > > 12 files changed, 756 insertions(+), 48 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,gicp.txt > > create mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,icu.txt > > create mode 100644 drivers/irqchip/irq-mvebu-gicp.c > > create mode 100644 drivers/irqchip/irq-mvebu-gicp.h > > create mode 100644 drivers/irqchip/irq-mvebu-icu.c > > create mode 100644 include/dt-bindings/interrupt-controller/mvebu-icu.h > > > > It all looks good to me. How do we merge this? I take the first five > patches and Gregory takes the last one? I think you should take the first four patches, and Grégory takes the last two ones. But I'll let Grégory confirm. Once again, thanks a lot for your help on this series, it's in a much better shape now compared to what the original submission was! Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Subject: Re: [PATCH v5 0/6] Add support for the ICU unit in Marvell Armada 7K/8K Date: Wed, 21 Jun 2017 17:18:09 +0200 Message-ID: <20170621171809.34bcce1e@windsurf.home> References: <20170621132917.14187-1-thomas.petazzoni@free-electrons.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: 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: Marc Zyngier Cc: Mark Rutland , devicetree@vger.kernel.org, Yehuda Yitschak , Jason Cooper , Pawel Moll , Ian Campbell , Hanna Hawa , =?UTF-8?B?TWlxdcOobA==?= Raynal , linux-kernel@vger.kernel.org, Nadav Haklai , Rob Herring , Andrew Lunn , Kumar Gala , Gregory Clement , Thomas Gleixner , Antoine Tenart , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth List-Id: devicetree@vger.kernel.org SGVsbG8sCgpPbiBXZWQsIDIxIEp1biAyMDE3IDE2OjE0OjIxICswMTAwLCBNYXJjIFp5bmdpZXIg d3JvdGU6Cgo+ID4gVGhvbWFzIFBldGF6em9uaSAoNik6Cj4gPiAgIGR0LWJpbmRpbmdzOiBpbnRl cnJ1cHQtY29udHJvbGxlcjogYWRkIERUIGJpbmRpbmcgZm9yIHRoZSBNYXJ2ZWxsIEdJQ1AKPiA+ ICAgZHQtYmluZGluZ3M6IGludGVycnVwdC1jb250cm9sbGVyOiBhZGQgRFQgYmluZGluZyBmb3Ig dGhlIE1hcnZlbGwgSUNVCj4gPiAgIGlycWNoaXA6IGlycS1tdmVidS1naWNwOiBuZXcgZHJpdmVy IGZvciBNYXJ2ZWxsIEdJQ1AKPiA+ICAgaXJxY2hpcDogaXJxLW12ZWJ1LWljdTogbmV3IGRyaXZl ciBmb3IgTWFydmVsbCBJQ1UKPiA+ICAgYXJtNjQ6IG1hcnZlbGw6IGVuYWJsZSBJQ1UgYW5kIEdJ Q1AgZHJpdmVycwo+ID4gICBhcm02NDogZHRzOiBtYXJ2ZWxsOiBlbmFibGUgR0lDUCBhbmQgSUNV IG9uIEFybWFkYSA3Sy84Swo+ID4gCj4gPiAgLi4uL2JpbmRpbmdzL2ludGVycnVwdC1jb250cm9s bGVyL21hcnZlbGwsZ2ljcC50eHQgfCAgMjcgKysKPiA+ICAuLi4vYmluZGluZ3MvaW50ZXJydXB0 LWNvbnRyb2xsZXIvbWFydmVsbCxpY3UudHh0ICB8ICA1MSArKysrCj4gPiAgYXJjaC9hcm02NC9L Y29uZmlnLnBsYXRmb3JtcyAgICAgICAgICAgICAgICAgICAgICAgfCAgIDIgKwo+ID4gIGFyY2gv YXJtNjQvYm9vdC9kdHMvbWFydmVsbC9hcm1hZGEtYXA4MDYuZHRzaSAgICAgIHwgICA3ICsKPiA+ ICAuLi4vYm9vdC9kdHMvbWFydmVsbC9hcm1hZGEtY3AxMTAtbWFzdGVyLmR0c2kgICAgICB8ICA1 OSArKystLQo+ID4gIC4uLi9hcm02NC9ib290L2R0cy9tYXJ2ZWxsL2FybWFkYS1jcDExMC1zbGF2 ZS5kdHNpIHwgIDU1ICsrLS0KPiA+ICBkcml2ZXJzL2lycWNoaXAvS2NvbmZpZyAgICAgICAgICAg ICAgICAgICAgICAgICAgICB8ICAgNiArCj4gPiAgZHJpdmVycy9pcnFjaGlwL01ha2VmaWxlICAg ICAgICAgICAgICAgICAgICAgICAgICAgfCAgIDIgKwo+ID4gIGRyaXZlcnMvaXJxY2hpcC9pcnEt bXZlYnUtZ2ljcC5jICAgICAgICAgICAgICAgICAgIHwgMjc5ICsrKysrKysrKysrKysrKysrKysr Cj4gPiAgZHJpdmVycy9pcnFjaGlwL2lycS1tdmVidS1naWNwLmggICAgICAgICAgICAgICAgICAg fCAgMTIgKwo+ID4gIGRyaXZlcnMvaXJxY2hpcC9pcnEtbXZlYnUtaWN1LmMgICAgICAgICAgICAg ICAgICAgIHwgMjg5ICsrKysrKysrKysrKysrKysrKysrKwo+ID4gIC4uLi9kdC1iaW5kaW5ncy9p bnRlcnJ1cHQtY29udHJvbGxlci9tdmVidS1pY3UuaCAgIHwgIDE1ICsrCj4gPiAgMTIgZmlsZXMg Y2hhbmdlZCwgNzU2IGluc2VydGlvbnMoKyksIDQ4IGRlbGV0aW9ucygtKQo+ID4gIGNyZWF0ZSBt b2RlIDEwMDY0NCBEb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvaW50ZXJydXB0LWNv bnRyb2xsZXIvbWFydmVsbCxnaWNwLnR4dAo+ID4gIGNyZWF0ZSBtb2RlIDEwMDY0NCBEb2N1bWVu dGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvaW50ZXJydXB0LWNvbnRyb2xsZXIvbWFydmVsbCxp Y3UudHh0Cj4gPiAgY3JlYXRlIG1vZGUgMTAwNjQ0IGRyaXZlcnMvaXJxY2hpcC9pcnEtbXZlYnUt Z2ljcC5jCj4gPiAgY3JlYXRlIG1vZGUgMTAwNjQ0IGRyaXZlcnMvaXJxY2hpcC9pcnEtbXZlYnUt Z2ljcC5oCj4gPiAgY3JlYXRlIG1vZGUgMTAwNjQ0IGRyaXZlcnMvaXJxY2hpcC9pcnEtbXZlYnUt aWN1LmMKPiA+ICBjcmVhdGUgbW9kZSAxMDA2NDQgaW5jbHVkZS9kdC1iaW5kaW5ncy9pbnRlcnJ1 cHQtY29udHJvbGxlci9tdmVidS1pY3UuaAo+ID4gICAKPiAKPiBJdCBhbGwgbG9va3MgZ29vZCB0 byBtZS4gSG93IGRvIHdlIG1lcmdlIHRoaXM/IEkgdGFrZSB0aGUgZmlyc3QgZml2ZQo+IHBhdGNo ZXMgYW5kIEdyZWdvcnkgdGFrZXMgdGhlIGxhc3Qgb25lPwoKSSB0aGluayB5b3Ugc2hvdWxkIHRh a2UgdGhlIGZpcnN0IGZvdXIgcGF0Y2hlcywgYW5kIEdyw6lnb3J5IHRha2VzIHRoZQpsYXN0IHR3 byBvbmVzLiBCdXQgSSdsbCBsZXQgR3LDqWdvcnkgY29uZmlybS4KCk9uY2UgYWdhaW4sIHRoYW5r cyBhIGxvdCBmb3IgeW91ciBoZWxwIG9uIHRoaXMgc2VyaWVzLCBpdCdzIGluIGEgbXVjaApiZXR0 ZXIgc2hhcGUgbm93IGNvbXBhcmVkIHRvIHdoYXQgdGhlIG9yaWdpbmFsIHN1Ym1pc3Npb24gd2Fz IQoKQmVzdCByZWdhcmRzLAoKVGhvbWFzCi0tIApUaG9tYXMgUGV0YXp6b25pLCBDVE8sIEZyZWUg RWxlY3Ryb25zCkVtYmVkZGVkIExpbnV4IGFuZCBLZXJuZWwgZW5naW5lZXJpbmcKaHR0cDovL2Zy ZWUtZWxlY3Ryb25zLmNvbQoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX18KbGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgtYXJtLWtlcm5lbEBs aXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3JnL21haWxtYW4vbGlz dGluZm8vbGludXgtYXJtLWtlcm5lbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: thomas.petazzoni@free-electrons.com (Thomas Petazzoni) Date: Wed, 21 Jun 2017 17:18:09 +0200 Subject: [PATCH v5 0/6] Add support for the ICU unit in Marvell Armada 7K/8K In-Reply-To: References: <20170621132917.14187-1-thomas.petazzoni@free-electrons.com> Message-ID: <20170621171809.34bcce1e@windsurf.home> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello, On Wed, 21 Jun 2017 16:14:21 +0100, Marc Zyngier wrote: > > Thomas Petazzoni (6): > > dt-bindings: interrupt-controller: add DT binding for the Marvell GICP > > dt-bindings: interrupt-controller: add DT binding for the Marvell ICU > > irqchip: irq-mvebu-gicp: new driver for Marvell GICP > > irqchip: irq-mvebu-icu: new driver for Marvell ICU > > arm64: marvell: enable ICU and GICP drivers > > arm64: dts: marvell: enable GICP and ICU on Armada 7K/8K > > > > .../bindings/interrupt-controller/marvell,gicp.txt | 27 ++ > > .../bindings/interrupt-controller/marvell,icu.txt | 51 ++++ > > arch/arm64/Kconfig.platforms | 2 + > > arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 7 + > > .../boot/dts/marvell/armada-cp110-master.dtsi | 59 +++-- > > .../arm64/boot/dts/marvell/armada-cp110-slave.dtsi | 55 ++-- > > drivers/irqchip/Kconfig | 6 + > > drivers/irqchip/Makefile | 2 + > > drivers/irqchip/irq-mvebu-gicp.c | 279 ++++++++++++++++++++ > > drivers/irqchip/irq-mvebu-gicp.h | 12 + > > drivers/irqchip/irq-mvebu-icu.c | 289 +++++++++++++++++++++ > > .../dt-bindings/interrupt-controller/mvebu-icu.h | 15 ++ > > 12 files changed, 756 insertions(+), 48 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,gicp.txt > > create mode 100644 Documentation/devicetree/bindings/interrupt-controller/marvell,icu.txt > > create mode 100644 drivers/irqchip/irq-mvebu-gicp.c > > create mode 100644 drivers/irqchip/irq-mvebu-gicp.h > > create mode 100644 drivers/irqchip/irq-mvebu-icu.c > > create mode 100644 include/dt-bindings/interrupt-controller/mvebu-icu.h > > > > It all looks good to me. How do we merge this? I take the first five > patches and Gregory takes the last one? I think you should take the first four patches, and Gr?gory takes the last two ones. But I'll let Gr?gory confirm. Once again, thanks a lot for your help on this series, it's in a much better shape now compared to what the original submission was! Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com