[U-Boot] [patch] rm9200 lowevel_init: don't touch reserved/readonly registers
Jean-Christophe PLAGNIOL-VILLARD
plagnioj at jcrosoft.com
Mon Aug 3 09:29:08 CEST 2009
On 18:40 Thu 16 Jul , David Brownell wrote:
> For some reason the AT91rm9200 lowlevel init writes to a bunch of
> reserved or read-only addresses. All the boards seem to define the
> value-to-be-written values as zero ... but they shouldn't actually
> be writing *anything* there.
>
> No documented erratum justifies these accesses. It looks like maybe
> some pre-release BDI-2000 setup code has been carried along by cargo
> cult programming since at least late 2004 (per GIT history).
>
> Here's a patch disabling what seems to be bogosity. Tested on a
> csb337; there were no behavioral changes.
> ---
> cpu/arm920t/at91rm9200/lowlevel_init.S | 14 ++------------
> include/configs/at91rm9200dk.h | 5 -----
> include/configs/at91rm9200ek.h | 5 -----
> include/configs/cmc_pu2.h | 5 -----
> include/configs/csb637.h | 5 -----
> include/configs/m501sk.h | 5 -----
> include/configs/mp2usb.h | 5 -----
> 7 files changed, 2 insertions(+), 42 deletions(-)
>
Tested on rm9200ek
applied to u-boot-arm
Best Regards,
J.
More information about the U-Boot
mailing list