Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752056AbdFUQ1Q convert rfc822-to-8bit (ORCPT ); Wed, 21 Jun 2017 12:27:16 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:39808 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751640AbdFUQ1O (ORCPT ); Wed, 21 Jun 2017 12:27:14 -0400 From: Gregory CLEMENT To: Marc Zyngier Cc: Thomas Petazzoni , 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> <87r2yd1h5x.fsf@free-electrons.com> <245df82c-60b1-bc4b-e343-5435713c453c@arm.com> Date: Wed, 21 Jun 2017 17:50:20 +0200 In-Reply-To: <245df82c-60b1-bc4b-e343-5435713c453c@arm.com> (Marc Zyngier's message of "Wed, 21 Jun 2017 16:47:20 +0100") Message-ID: <87k2451gsj.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: 2838 Lines: 73 Hi Marc, On mer., juin 21 2017, Marc Zyngier wrote: > On 21/06/17 16:42, Gregory CLEMENT wrote: >> 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. > > Hmmm. You mean patches 5 and 6, right? And I queue 1 to 4? Yes that was what I meant! (it's too hot here and my brain is melting...) Thanks, Gregory > > Thanks, > > M. > -- > Jazz is not dead. It just smells funny... -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com