From mboxrd@z Thu Jan  1 00:00:00 1970
From: Boris BREZILLON <b.brezillon.dev@gmail.com>
Subject: Re: [PATCH v2 02/11] clk: at91: update main clk documentation
Date: Fri, 18 Apr 2014 15:00:37 +0200
Message-ID: <535121F5.10307@gmail.com>
References: <1395649643-9146-1-git-send-email-b.brezillon.dev@gmail.com> <1395649643-9146-3-git-send-email-b.brezillon.dev@gmail.com> <53511F60.3040301@atmel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Return-path: <linux-doc-owner@vger.kernel.org>
In-Reply-To: <53511F60.3040301@atmel.com>
Sender: linux-doc-owner@vger.kernel.org
To: Nicolas Ferre <nicolas.ferre@atmel.com>, Boris BREZILLON <brezillonboris@gmail.com>, Mike Turquette <mturquette@linaro.org>, Alexandre Belloni <alexandre.belloni@free-electrons.com>, Jean-Jacques Hiblot <jjhiblot@traphandler.com>, Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
Cc: devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org
List-Id: devicetree@vger.kernel.org


On 18/04/2014 14:49, Nicolas Ferre wrote:
> On 24/03/2014 09:27, Boris BREZILLON :
>> Update main clk documentation to match main clk implementation rework.
>>
>> Signed-off-by: Boris BREZILLON <b.brezillon.dev@gmail.com>
>> ---
>>  .../devicetree/bindings/clock/at91-clock.txt       |   56 ++++++++++++++++----
>>  1 file changed, 47 insertions(+), 9 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/clock/at91-clock.txt b/Documentation/devicetree/bindings/clock/at91-clock.txt
>> index cd5e239..9a7025b 100644
>> --- a/Documentation/devicetree/bindings/clock/at91-clock.txt
>> +++ b/Documentation/devicetree/bindings/clock/at91-clock.txt
>> @@ -15,8 +15,13 @@ Required properties:
>>  		All at91 specific clocks (clocks defined below) must be child
>>  		node of the PMC node.
>>  
>> +	"atmel,at91rm9200-clk-main-osc"
>> +	"atmel,at91sam9x5-clk-main-rc-osc"
>> +		at91 main clk sources
>> +
>> +	"atmel,at91sam9x5-clk-main"
>>  	"atmel,at91rm9200-clk-main":
>> -		at91 main oscillator
>> +		at91 main clock
>>  
>>  	"atmel,at91rm9200-clk-master" or
>>  	"atmel,at91sam9x5-clk-master":
>> @@ -85,24 +90,57 @@ For example:
>>  		/* put at91 clocks here */
>>  	};
>>  
>> +Required properties for main clock oscillator:
>> +- interrupt-parent : must reference the PMC node.
>> +- interrupts : shall be set to "<0>".
>> +- clock-frequency : define the internal RC oscillator frequency.
>> +
>> +Optional properties:
>> +- clock-accuracy : define the internal RC oscillator accuracy.
>> +
>> +For example:
>> +	main_rc_osc: main_rc_osc {
>> +		compatible = "atmel,at91sam9x5-clk-main-rc-osc";
>> +		interrupt-parent = <&pmc>;
>> +		interrupts = <0>;
>> +		clock-frequency = <12000000>;
>> +		clock-accuracy = <50000000>;
>> +	};
> Don't you swapped this example above with the description below?

Actually I only swapped the "Required properties" lines, I'll fix it.

Thanks,

Boris

>
>
>> +Required properties for main clock internal RC oscillator:
>> +- interrupt-parent : must reference the PMC node.
>> +- interrupts : shall be set to "<0>".
>> +- #clock-cells : from common clock binding; shall be set to 0.
>> +- clocks : shall encode the main osc source clk sources (see atmel datasheet).
>> +
>> +Optional properties:
>> +- atmel,osc-bypass : boolean property. Specified if a clock signal is provided
>> +  on XIN.
>> +
>> +  clock signal is directly provided on XIN pin.
>> +
>> +For example:
>> +	main_osc: main_osc {
>> +		compatible = "atmel,at91rm9200-clk-main-osc";
>> +		interrupt-parent = <&pmc>;
>> +		interrupts = <0>;
>> +		#clock-cells = <0>;
>> +		clocks = <&main_xtal>;
>> +	};
>> +
>>  Required properties for main clock:
>>  - interrupt-parent : must reference the PMC node.
>>  - interrupts : shall be set to "<0>".
>>  - #clock-cells : from common clock binding; shall be set to 0.
>> -- clocks (optional if clock-frequency is provided) : shall be the slow clock
>> -	phandle. This clock is used to calculate the main clock rate if
>> -	"clock-frequency" is not provided.
>> -- clock-frequency : the main oscillator frequency.Prefer the use of
>> -	"clock-frequency" over automatic clock rate calculation.
>> +- clocks : shall encode the main clk sources (see atmel datasheet).
>>  
>>  For example:
>>  	main: mainck {
>> -		compatible = "atmel,at91rm9200-clk-main";
>> +		compatible = "atmel,at91sam9x5-clk-main";
>>  		interrupt-parent = <&pmc>;
>>  		interrupts = <0>;
>>  		#clock-cells = <0>;
>> -		clocks = <&ck32k>;
>> -		clock-frequency = <18432000>;
>> +		clocks = <&main_rc_osc &main_osc>;
>>  	};
>>  
>>  Required properties for master clock:
>>
>