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

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