[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