From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751195AbdFBNxD (ORCPT ); Fri, 2 Jun 2017 09:53:03 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:42106 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751125AbdFBNxB (ORCPT ); Fri, 2 Jun 2017 09:53:01 -0400 From: Gregory CLEMENT To: Michael Turquette Cc: "Stephen Boyd" , linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Thomas Petazzoni , Andrew Lunn , Jason Cooper , devicetree@vger.kernel.org, Omri Itach , Nadav Haklai , Kostya Porotchkin , Rob Herring , Neta Zur Hershkovits , Shadi Ammouri , Marcin Wojtas , linux-arm-kernel@lists.infradead.org, Sebastian Hesselbarth Subject: Re: [PATCH v2 0/7] Improve ap806 clk support on Marvell Armada 7K/8K References: <149631063623.33065.5881396919137461999@resonance> Date: Fri, 02 Jun 2017 15:52:48 +0200 In-Reply-To: <149631063623.33065.5881396919137461999@resonance> (Michael Turquette's message of "Thu, 01 Jun 2017 18:50:36 +0900") Message-ID: <87mv9qpkcv.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 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Michael, On jeu., juin 01 2017, Michael Turquette wrote: > Hi Gregory, > > Quoting Gregory CLEMENT (2017-05-31 23:07:21) >> Hi, >> >> This series modifies the device tree binding of the clock of the AP806 >> part that we find in the Marvell Armada 7K/8K SoCs. >> >> As for the previsous series the only change in this second version is >> about the binding documentation: all the documentation related changes >> are now move in their own patches. It allows to provide a stable -dt >> branch for theses changes. I also added the acked-by from Rob Herring. >> >> Here again the last two patches modifying the device tree _must_ be >> merged through the mvebu tree to avoid future conflict. > > The approach (and the patches) look good to me. I took the DT binding > description changes (patches 2 & 4) into clk-ap806-dt stable branch: > > https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git/log/?h=clk-ap806-dt > > Anyone can merge the above branch if they need it as a dep. > > The driver changes (patches 1, 3 & 5) are merged on into clk-ap806, > which is based on top of clk-ap806-dt: > > https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git/log/?h=clk-ap806 > > Everything has been merged into clk-next. I did not touch the .dts files > at all. > Perfect I will take care of the dts patch. What about the other series on cp110? a clk_cp110-dt branch will be needed too for the pinctrl series. Thanks, Gregory > Regards, > Mike > >> >> For the record: >> As for the CP110, we want to be able to ease the integration of new >> clocks without breaking the backward compatibility. It is done in >> patch 3. >> >> We also want to ease the integration of the pinctrl node in the device >> tree. It is the purpose of the patch 5. >> >> In this series (as in the previous one for CP110), even if there is >> some change in the device tree binding we paid attention to the >> backward compatibility, and the driver can still work with the old >> device tree. >> >> Thanks, >> >> Gregory >> >> Gregory CLEMENT (7): >> clk: mvebu: ap806: cosmetic improvement >> dt-bindings: ap806: do not depend anymore of the *-clock-output-names >> clk: mvebu: ap806: do not depend anymore of the *-clock-output-names >> dt-bindings: ap806: introduce a new binding >> clk: mvebu: ap806: introduce a new binding >> arm64: dts: marvell: remove clock-output-names on ap806 >> arm64: dts: marvell: use new binding for the system controller on ap806 >> >> Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt | 24 ++++++++++------ >> arch/arm64/boot/dts/marvell/armada-ap806.dtsi | 23 +++++++-------- >> drivers/clk/mvebu/ap806-system-controller.c | 107 +++++++++++++++++++++++++++++++++++++++++++++++------------------------- >> 3 files changed, 97 insertions(+), 57 deletions(-) >> >> base-commit: 4139fcd6c66df1c3d3fa0a0a7cf7f8a8c601a16c >> -- >> git-series 0.9.1 > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com