linux-kernel - Re: [PATCH v2 1/2] clk: add device tree binding for Artpec-6 clock controller

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: Windows password security audit tool. GUI, reports in PDF.

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

Message-ID: <20160223204307.GA19883@rob-hp-laptop>
Date:	Tue, 23 Feb 2016 14:43:07 -0600
From:	Rob Herring <robh@...nel.org>
To:	Lars Persson <lars.persson@...s.com>
Cc:	devicetree@...r.kernel.org, linux-clk@...r.kernel.org,
	mturquette@...libre.com, sboyd@...eaurora.org, pawel.moll@....com,
	mark.rutland@....com, ijc+devicetree@...lion.org.uk,
	galak@...eaurora.org, linux-kernel@...r.kernel.org,
	Lars Persson <larper@...s.com>
Subject: Re: [PATCH v2 1/2] clk: add device tree binding for Artpec-6 clock
 controller

On Fri, Feb 19, 2016 at 03:30:14PM +0100, Lars Persson wrote:
> Add device tree documentation for the main clock controller in the
> Artpec-6 SoC.
> 
> Signed-off-by: Lars Persson <larper@...s.com>
> ---
>  .../devicetree/bindings/clock/artpec6.txt          | 41 ++++++++++++++++++++++
>  1 file changed, 41 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/clock/artpec6.txt
> 
> diff --git a/Documentation/devicetree/bindings/clock/artpec6.txt b/Documentation/devicetree/bindings/clock/artpec6.txt
> new file mode 100644
> index 0000000..76e3e7f
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/clock/artpec6.txt
> @@ -0,0 +1,41 @@
> +* Clock bindings for Axis ARTPEC-6 chip
> +
> +The bindings are based on the clock provider binding in
> +Documentation/devicetree/bindings/clock/clock-bindings.txt
> +
> +External clocks:
> +----------------
> +
> +There are two external inputs to the main clock controller which should be
> +provided using the common clock bindings.
> +- "sys_refclk": External 50 Mhz oscillator (required)
> +- "i2s_refclk": Alternate audio reference clock (optional).
> +
> +Main clock controller
> +---------------------
> +
> +Required properties:
> +- #clock-cells: Should be <1>
> +  See dt-bindings/clock/axis,artpec6-clkctrl.h for the list of valid identifiers.

This header should be part of this patch. Otherwise:

Acked-by: Rob Herring <robh@...nel.org>

> +- compatible: Should be "axis,artpec6-clkctrl"
> +- reg: Must contain the base address and length of the system controller
> +- clocks:  Must contain a phandle entry for each clock in clock-names
> +- clock-names: Must include the external oscillator ("sys_refclk"). Optional
> +  ones are the audio reference clock ("i2s_refclk") and the audio fractional
> +  divider ("frac_clk").
> +
> +Examples:
> +
> +ext_clk: ext_clk {
> +	#clock-cells = <0>;
> +	compatible = "fixed-clock";
> +	clock-frequency = <50000000>;
> +};
> +
> +clkctrl: clkctrl {
> +	#clock-cells = <1>;
> +	compatible = "axis,artpec6-clkctrl";
> +	reg = <0xf8000000 0x48>;
> +	clocks = <&ext_clk>;
> +	clock-names = "sys_refclk";
> +};
> -- 
> 2.1.4
>