[U-Boot] BeagleBone u-boot bug in arch/arm/include/asm/arch-am33xx/ddr_defs.h

In arch/arm/include/asm/arch-am33xx/ddr_defs.h:
For the 256MB BeagleBone u-boot initializes the EMIF SDRAM_CONFIG register with #define MT47H128M16RT25E_EMIF_SDCFG 0x41805332
This is incorrect since it programs a row size of 6 (15 real) where the MT47H128M16RT25E only had 14 rows [13:0].
It should be using a value of 0x418052B2 (0x41805332 - 0x80) to correctly address 256MB of RAM instead of the current 512MB.

On Mon, May 06, 2013 at 04:21:13PM -0700, Matt Thomas wrote:
In arch/arm/include/asm/arch-am33xx/ddr_defs.h:
For the 256MB BeagleBone u-boot initializes the EMIF SDRAM_CONFIG register with #define MT47H128M16RT25E_EMIF_SDCFG 0x41805332
This is incorrect since it programs a row size of 6 (15 real) where the MT47H128M16RT25E only had 14 rows [13:0].
It should be using a value of 0x418052B2 (0x41805332 - 0x80) to correctly address 256MB of RAM instead of the current 512MB.
Sounds good, please send a patch, thanks!
participants (2)
-
Matt Thomas
-
Tom Rini