Re: [PATCH 09/11] ARM: at91/dt: define main xtal frequency of the at91sam9261ek board

[Boris BREZILLON <b.brezillon.dev@gmail.com>]

Le 20/03/2014 10:24, Jean-Jacques Hiblot a écrit :
> 2014-03-18 21:18 GMT+01:00 Boris BREZILLON <brezillonboris@gmail.com>:
>> Define at91sam9261ek main crystal frequency.
>>
>> Signed-off-by: Boris BREZILLON <b.brezillon.dev@gmail.com>
>> ---
>>   arch/arm/boot/dts/at91sam9261ek.dts |    4 ++++
>>   1 file changed, 4 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/at91sam9261ek.dts b/arch/arm/boot/dts/at91sam9261ek.dts
>> index 2ce527e..7175986 100644
>> --- a/arch/arm/boot/dts/at91sam9261ek.dts
>> +++ b/arch/arm/boot/dts/at91sam9261ek.dts
>> @@ -29,6 +29,10 @@
>>                          compatible = "atmel,osc", "fixed-clock";
>>                          clock-frequency = <18432000>;
>>                  };
>> +
>> +               main_xtal: main_xtal {
>> +                       clock-frequency = <18432000>;
>> +               };
>>          };
>>
>>          ahb {
>> --
>> 1.7.9.5
>>
> You forgot to remove the old fixed-clock definition for the main_clock: clock@0

We need this clock when CCF is disabled (when using the old at91 clk 
implementation).
I'm afraid we'll have to keep it until we move all sam9 SoCs and boards 
to CCF (and DT).

Best Regards,

Boris
>
> Otherwise:
> Acked-by: Jean-Jacques HIBLOT <jjhiblot@traphandler.com>