Hi,
On 24 November 2015 at 11:23, Simon Glass sjg@chromium.org wrote:
Hi Thomas,
On 24 November 2015 at 07:01, Thomas Chou thomas@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?
I've had a crack at this and will send an updated patch.
Regards, Simon