linux-kernel - Re: [PATCH] iio: at91: document ADC clock properties

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <52B729FD.7020507@kernel.org>
Date:	Sun, 22 Dec 2013 18:05:49 +0000
From:	Jonathan Cameron <jic23@...nel.org>
To:	Nicolas Ferre <nicolas.ferre@...el.com>,
	Boris BREZILLON <b.brezillon@...rkiz.com>,
	Rob Landley <rob@...dley.net>
CC:	Arnd Bergmann <arnd@...db.de>, devicetree@...r.kernel.org,
	linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org, linux-iio@...r.kernel.org
Subject: Re: [PATCH] iio: at91: document ADC clock properties

On 12/19/13 15:54, Nicolas Ferre wrote:
> On 17/12/2013 17:16, Boris BREZILLON :
>> Document the clock properties required by the at91 ADC driver.
>>
>> Signed-off-by: Boris BREZILLON <b.brezillon@...rkiz.com>
> 
> Acked-by: Nicolas Ferre <nicolas.ferre@...el.com>
Looks fine to me.  I'm just waiting on an ack from a device tree maintainer
(or the 3 weeks to pass so I can pick it up anyway ;)

> 
>> ---
>>  .../devicetree/bindings/arm/atmel-adc.txt          |    5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/arm/atmel-adc.txt b/Documentation/devicetree/bindings/arm/atmel-adc.txt
>> index d106146..9a1175b 100644
>> --- a/Documentation/devicetree/bindings/arm/atmel-adc.txt
>> +++ b/Documentation/devicetree/bindings/arm/atmel-adc.txt
>> @@ -5,6 +5,9 @@ Required properties:
>>      <chip> can be "at91sam9260", "at91sam9g45" or "at91sam9x5"
>>    - reg: Should contain ADC registers location and length
>>    - interrupts: Should contain the IRQ line for the ADC
>> +  - clock-names: tuple listing input clock names.
>> +	Required elements: "adc_clk", "adc_op_clk".
>> +  - clocks: phandles to input clocks.
>>    - atmel,adc-channels-used: Bitmask of the channels muxed and enable for this
>>      device
>>    - atmel,adc-startup-time: Startup Time of the ADC in microseconds as
>> @@ -44,6 +47,8 @@ adc0: adc@...b0000 {
>>  	compatible = "atmel,at91sam9260-adc";
>>  	reg = <0xfffb0000 0x100>;
>>  	interrupts = <20 4>;
>> +	clocks = <&adc_clk>, <&adc_op_clk>;
>> +	clock-names = "adc_clk", "adc_op_clk";
>>  	atmel,adc-channel-base = <0x30>;
>>  	atmel,adc-channels-used = <0xff>;
>>  	atmel,adc-drdy-mask = <0x10000>;
>>
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/