[U-Boot] [PATCH v3 05/11] dm: timer: Support 64-bit counter

Simon Glass sjg at chromium.org
Tue Nov 24 19:23:44 CET 2015


Hi Thomas,

On 24 November 2015 at 07:01, Thomas Chou <thomas at wytron.com.tw> wrote:
> Hi Simon,
>
> On 2015年11月24日 18:09, Simon Glass wrote:
>>
>>
>> Unfortunately this causes build errors for a few avr32 boards - e.g.
>> grasshopper.
>>
>> I think the problem is the gd declaration in the timer.h header file.
>> I don't think that is a good idea. Can we move it to the C file?
>
>
> If you meant, "warning: input is not relaxable", it is fixed after commit
> f300dccde433 ("i2c, avr32: fix compiler warning "input is not relaxable"").
> The u-boot-dm/testing branch does not have such warning.

Please try u-boot-dm/testing. This seems to be a different problem:

     avr32:  +   grasshopper
+lib/time.c:20: warning: register used for two global register variables

We should not declare the global_data pointer in a header file.

Bin, are you able to take a look? Perhaps the function that needs it
should move to the C file?

Regards,
Simon


More information about the U-Boot mailing list