Re: [PATCH v3 0/3] clk: clock driver for the Marvell Armada AP806

Gregory CLEMENT
2016-03-08T08:57:47Z

Hi Thomas,

On ven., mars 04 2016, Thomas Petazzoni <thomas.petazzoni@free-electrons.com> wrote:

> Michael, Stephen,
>
> Here is a new version of the clock driver and its DT binding for the
> Marvell Armada AP806. As you will see, there is no longer any
> reference to the DFX Server and changes in the clock tree compared to
> the v2. This is due to the fact that the HW has changed from this
> point of view between the test chips and the final revision, and this
> new version aims at supporting only the final revision.
>
> We therefore have a very simple DT binding, with one single node that
> covers the entire "system controller" registers, associated to one
> driver that provides the clocks. This system controller will later be
> used to support other features such as pin-muxing, but for now we only
> support the clocks aspects of it.
>
> Please let me know if this approach works for you.
>
> Note that this patch series is based on clk-net, because I wanted to
> avoid introducing a new use of CLK_IS_ROOT. To do this, I needed the
> patches currently in clk-next that remove the need for CLK_IS_ROOT.
>
> Thanks!

The series is clean and small, I didn't see any problem with it, so for
the whole series:

Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>

Thanks,

Gregory

>
> Thomas
>
> Thomas Petazzoni (3):
> dt-bindings: arm: add DT binding for Marvell AP806 system controller
> clk: unconditionally recurse into clk/mvebu/
> clk: mvebu: new driver for Armada AP806 system controller
>
> .../arm/marvell/ap806-system-controller.txt | 34 +++++++
> drivers/clk/Makefile | 2 +-
> drivers/clk/mvebu/Kconfig | 3 +
> drivers/clk/mvebu/Makefile | 1 +
> drivers/clk/mvebu/ap806-system-controller.c | 106 +++++++++++++++++++++
> 5 files changed, 145 insertions(+), 1 deletion(-)
> create mode 100644 Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt
> create mode 100644 drivers/clk/mvebu/ap806-system-controller.c
>
> --
> 2.6.4
>

--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.