Hi Heiko,
On 11/21/2011 03:09 PM, Heiko Schocher wrote:
> Holger Brunck wrote:
>> On 09/15/2011 07:34 AM, Heiko Schocher wrote:
>>> Signed-off-by: Heiko Schocher <h...@denx.de>
>>> Cc: Albert ARIBAUD <albert.u.b...@aribaud.net>
>>> ---
>>> post/post.c | 2 +-
>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/post/post.c b/post/post.c
>>> index 6c59f91..0bfca75 100644
>>> --- a/post/post.c
>>> +++ b/post/post.c
>>> @@ -488,7 +488,7 @@ void post_reloc(void)
>>> */
>>> unsigned long post_time_ms(unsigned long base)
>>> {
>>> -#ifdef CONFIG_PPC
>>> +#if defined(CONFIG_PPC) || defined(CONFIG_ARM)
>>> return (unsigned long)(get_ticks() / (get_tbclk() / CONFIG_SYS_HZ))
>>> - base;
>>> #else
>>
>> this commit breaks post test support for our keymile arm boards:
>
> :-(
>
> I am sure I tried a "MAKEALL arm" ... Hmm.. how did the post support
I guess that our post support wasn't mainlined at the time you test the patch.
> worked for you before this patch? You should have seen this warning message:
>
> #warning "Not implemented yet"
>
> or?
>
Yes we got these warning in the past but this wasn't a problem for us, because
we don't use these timer informations in our framework.
>>> make km_kirkwood
>> [...]
>> post/libpost.o: In function `post_time_ms':
>> /daten/ws_root/hbu/u-boot-km.git/post/post.c:504: undefined reference to
>> `get_ticks'
>> /daten/ws_root/hbu/u-boot-km.git/post/post.c:504: undefined reference to
>> `get_tbclk'
>>
>> So it seems that the kirkwood code has no implementation for these functions.
>> Any idea how to fix this?
>
> We must implement it ... Prafulla, can you comment this?
>
Or in the first step just add && !(CONFIG_MACH_KIRKWOOD) or similar to the
#ifdef until it is supported in the generic kirkwood code.
regards
Holger
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
