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

[Jean-Jacques Hiblot]

That's right. I keep forgetting this.

2014-03-20 10:50 GMT+01:00 Boris BREZILLON <b.brezillon.dev@xxxxxxxxx>:
>
> Le 20/03/2014 10:24, Jean-Jacques Hiblot a écrit :
>
>> 2014-03-18 21:18 GMT+01:00 Boris BREZILLON <brezillonboris@xxxxxxxxx>:
>>>
>>> Define at91sam9261ek main crystal frequency.
>>>
>>> Signed-off-by: Boris BREZILLON <b.brezillon.dev@xxxxxxxxx>
>>> ---
>>>   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@xxxxxxxxxxxxxxx>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/