linux-kernel - Re: [PATCH v2 00/11] Improve cp110 clk support on Marvell Armada 7K/8K

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <87lgpbsuga.fsf@free-electrons.com>
Date:   Thu, 01 Jun 2017 15:34:13 +0200
From:   Gregory CLEMENT <gregory.clement@...e-electrons.com>
To:     Mike Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...eaurora.org>
Cc:     linux-clk@...r.kernel.org, linux-kernel@...r.kernel.org,
        Jason Cooper <jason@...edaemon.net>,
        Andrew Lunn <andrew@...n.ch>,
        Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>,
        Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>,
        linux-arm-kernel@...ts.infradead.org,
        Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
        Nadav Haklai <nadavh@...vell.com>,
        Kostya Porotchkin <kostap@...vell.com>,
        Neta Zur Hershkovits <neta@...vell.com>,
        Marcin Wojtas <mw@...ihalf.com>,
        Omri Itach <omrii@...vell.com>,
        Shadi Ammouri <shadi@...vell.com>
Subject: Re: [PATCH v2 00/11] Improve cp110 clk support on Marvell Armada 7K/8K

Hi ,

Please do not apply this version. I was just pointed that I misread the
documentation and that the GOP clock are really also used by the
SD/eMMC.

So I am about sending a new version of the series removing the 2 first
patchs and fixing the patch 5.

Gregory

 On mer., mai 31 2017, Gregory CLEMENT <gregory.clement@...e-electrons.com> wrote:

> Hi,
>
> We got more information about the clock controllers and the clock tree
> of the CP110 part that we find in the Marvell Armada 7K/8K SoCs.
>
> The main 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.
>
> However the last two patches modifying the device tree _must_ be
> merged through the mvebu tree to avoid future conflict (especially
> with the recent ICU series sent by Thomas Petazzoni).
>
> For the record:
>
> The clock tree needs to be fixed, indeed the GOP clock is only the
> clock for the GOIP and not for the SDIO, it is fixed in the first
> patch.
>
> The third patch does a small improve of the code itself.
>
> The purpose of the fifth patch is to have an easier way to add new
> clocks without breaking the backward compatibility. Indeed, we expect
> to continue to have new update on the datasheet and this patch will
> allow integrating this change easier and faster.
>
> The  patch 7 is needed to ease the integration of the pinctrl node
> in the device tree.
>
> The patch 9 adds a new clock for the SDIO, it also fixes the clock
> tree.
>
> In this series, 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 (10):
>   dt-bindings: cp110: fix name of the GOP gate clock
>   clk: mvebu: cp110 fix name of the GOP gate clock
>   clk: mvebu: cp110: make failure labels more meaningful
>   dt-bindings: cp110: do not depend anymore of the *-clock-output-names
>   clk: mvebu: cp110: do not depend anymore of the *-clock-output-names
>   dt-bindings: cp110: introduce a new binding
>   clk: mvebu: cp110: introduce a new binding
>   dt-bindings: cp110: add sdio clock to cp-110 system controller
>   arm64: dts: marvell: remove *-clock-output-names on cp110
>   arm64: dts: marvell: use new binding for the system controller on cp110
>
> Konstantin Porotchkin (1):
>   clk: mvebu: cp110: add sdio clock to cp-110 system controller
>
>  Documentation/devicetree/bindings/arm/marvell/cp110-system-controller0.txt |  35 +++++-------
>  arch/arm64/boot/dts/marvell/armada-cp110-master.dtsi                       |  54 +++++++------------
>  arch/arm64/boot/dts/marvell/armada-cp110-slave.dtsi                        |  48 ++++++-----------
>  drivers/clk/mvebu/cp110-system-controller.c                                | 202 +++++++++++++++++++++++++++++++++++++++++++++++++-----------------------
>  4 files changed, 196 insertions(+), 143 deletions(-)
>
> base-commit: 2ea659a9ef488125eb46da6eb571de5eae5c43f6
> -- 
> git-series 0.9.1

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com