Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753065AbdFUQWS convert rfc822-to-8bit (ORCPT ); Wed, 21 Jun 2017 12:22:18 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:39725 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752413AbdFUQWO (ORCPT ); Wed, 21 Jun 2017 12:22:14 -0400 From: Gregory CLEMENT To: Marc Zyngier , Thomas Petazzoni 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 , linux-arm-kernel@lists.infradead.org, Nadav Haklai , Hanna Hawa , Yehuda Yitschak , Antoine Tenart , =?utf-8?Q?Miqu?= =?utf-8?Q?=C3=A8l?= Raynal Subject: Re: [PATCH v5 0/6] Add support for the ICU unit in Marvell Armada 7K/8K References: <20170621132917.14187-1-thomas.petazzoni@free-electrons.com> <20170621171809.34bcce1e@windsurf.home> Date: Wed, 21 Jun 2017 17:42:18 +0200 In-Reply-To: <20170621171809.34bcce1e@windsurf.home> (Thomas Petazzoni's message of "Wed, 21 Jun 2017 17:18:09 +0200") Message-ID: <87r2yd1h5x.fsf@free-electrons.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) 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 Content-Length: 2720 Lines: 66 Hi Marc and Thomas, On mer., juin 21 2017, Thomas Petazzoni wrote: > 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. Indeed it makes more sens to merge the patch 4 through the arm-soc subsystem. Thanks, Gregory > > 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 -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com