[U-Boot] [PATCH ARM] updates the at91 main_clock calculation

Tom Tom.Rix at windriver.com
Wed Feb 17 22:17:38 CET 2010


Daniel Gorsulowski wrote:
> Am 16.02.2010 16:23, schrieb Tom:
>> Daniel Gorsulowski wrote:
>>> Jens Scharsig wrote:
>>>>  * updates the conditional main_clock calculation (if AT91_MAIN_CLOCK defined) to c structure SoC access
>>>>  * add need register flags
>>>>
>>>>
>>>> Signed-off-by: Jens Scharsig <js_at_ng at scharsoft.de>
>>>> ---
>>>>  cpu/arm926ejs/at91/clock.c           |    7 ++++---
>>>>  include/asm-arm/arch-at91/at91_pmc.h |    3 +++
>>>>  2 files changed, 7 insertions(+), 3 deletions(-)
>>>>
>>> ...
>>>
>>> Thank you, now the updated otc570 builds without errors.
>>> I didn't check, whether the board boots, but I guess it does.
>>>
>> Were you going to check in the next couple of days ?
>> Tom
>>
>>
> Checked... board boots.
> 

Thanks!

> Btw. there are some warnings during build:
> mkimage.c: In function ‘main’:
> mkimage.c:204: warning: dereferencing type-punned pointer will break
> strict-aliasing rules
> mkimage.c:222: warning: dereferencing type-punned pointer will break

In general I look for new compiler warnings and errors.
Any old warnings should be fixed but it is unlikely that
this change introduced them.

Of course results will vary depending on toolchain.

Thanks,
Tom

> 
> Regards,
> Daniel



More information about the U-Boot mailing list