[U-Boot] [PATCH 1/5] imx: Add get_tbclk() function for imx25

Matthias Weißer weisserm at arcor.de
Mon Jul 4 07:56:48 CEST 2011


Hi Stefano

Am 30.06.2011 17:38, schrieb Stefano Babic:
> On 06/30/2011 11:57 AM, Matthias Weisser wrote:
>> Need this function for autoboot keyd
>>
>
> Hi Matthias,
>
>> +/*
>> + * This function is derived from PowerPC code (timebase clock frequency).
>> + * On ARM it returns the number of timer ticks per second.
>> + */
>> +ulong get_tbclk(void)
>> +{
>> +	ulong tbclk;
>> +
>> +	tbclk = CONFIG_MX25_CLK32;
>> +	return tbclk;
>> +}
>
> Which is the advantage to add this function instead using directly
> CONFIG_MX25_CLK32 in the caller ? It is not me so clear..

The caller is in common code (see common/main.c line 76 -> 97). I think 
if I add CONFIG_MX25_CLK32 there whis will break a couple of boards 
which I don't want to. ;-)

Also, we will have all this stuff cleaned up when the timer redesign 
comes in.

Regards,
Matthias


More information about the U-Boot mailing list