Message-ID: <56F256A9.6040405@axis.com>
Date: Wed, 23 Mar 2016 09:41:13 +0100
From: Lars Persson <lars.persson@axis.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.8.0
MIME-Version: 1.0
To: <mturquette@baylibre.com>, <sboyd@codeaurora.org>
CC: <devicetree@vger.kernel.org>, <linux-clk@vger.kernel.org>,
        <robh+dt@kernel.org>, <pawel.moll@arm.com>, <mark.rutland@arm.com>,
        <ijc+devicetree@hellion.org.uk>, <galak@codeaurora.org>,
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v4 0/2] clk: Add Artpec-6 SoC support
References: <cover.1456411894.git.larper@axis.com>
In-Reply-To: <cover.1456411894.git.larper@axis.com>
Content-Type: text/plain; charset="windows-1252"; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1994
Lines: 52

Michael & Stephen,

Should we resubmit this series for 4.7 ?

Also to follow up on your previous request to use the platform driver 
framework. We have noticed recent submissions use a style that mixes 
early clocks registered with CLK_OF_DECLARE and the remaining clocks 
registered through a platform driver. Is this the style we should 
implement ?

Best Regards,
  Lars Persson


On 02/25/2016 03:58 PM, Lars Persson wrote:
> Add clock support for the Artpec-6 SoC port. The ARM parts are in the series
> "arm: Add Artpec-6 SoC" and it goes through the arm-soc tree.
>
> Changes since v3:
> - The binding was corrected to handle two fractional divider clocks as input to
>    the clock controller.
> - Updated clk-artpec6.c to handle a distinct fractional divider input for each
>    i2s clock mux.
>
> Changes since v2:
> - Moved axis,artpec6-clkctrl.h to the first patch with the DT bindings.
>
> Changes since v1:
>
> - The driver now provides all clocks from the main clock controller block
>    through one DT node.
> - Added a header file for the clock indexes.
> - Refer to clock-bindings.txt in the bindings document.
>
> A platform driver was not possible because the clocks are needed earlier in the
> kernel startup.
>
> Lars Persson (2):
>    clk: add device tree binding for Artpec-6 clock controller
>    clk: add artpec-6 clock controller
>
>   .../devicetree/bindings/clock/artpec6.txt          |  41 +++++
>   drivers/clk/Makefile                               |   1 +
>   drivers/clk/axis/Makefile                          |   1 +
>   drivers/clk/axis/clk-artpec6.c                     | 189 +++++++++++++++++++++
>   include/dt-bindings/clock/axis,artpec6-clkctrl.h   |  38 +++++
>   5 files changed, 270 insertions(+)
>   create mode 100644 Documentation/devicetree/bindings/clock/artpec6.txt
>   create mode 100644 drivers/clk/axis/Makefile
>   create mode 100644 drivers/clk/axis/clk-artpec6.c
>   create mode 100644 include/dt-bindings/clock/axis,artpec6-clkctrl.h
>