[U-Boot-Users] [PATCH] ppc4xx: Fix Katmai CFG_MONITOR_LEN

Signed-off-by: Stefan Roese sr@denx.de --- include/configs/katmai.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/include/configs/katmai.h b/include/configs/katmai.h index 122b700..aad28fb 100644 --- a/include/configs/katmai.h +++ b/include/configs/katmai.h @@ -48,7 +48,7 @@ * Base addresses -- Note these are effective addresses where the * actual resources get mapped (not physical addresses) *----------------------------------------------------------------------*/ -#define CFG_MONITOR_LEN (256 * 1024) /* Reserve 256 kB for Mon */ +#define CFG_MONITOR_LEN (384 * 1024) /* Reserve 384 kB for Mon */ #define CFG_MALLOC_LEN (512 * 1024) /* Reserve 512 kB for malloc */
#define CFG_SDRAM_BASE 0x00000000 /* _must_ be 0 */

In message 1209471160-4453-1-git-send-email-sr@denx.de you wrote:
Signed-off-by: Stefan Roese sr@denx.de
...
-#define CFG_MONITOR_LEN (256 * 1024) /* Reserve 256 kB for Mon */ +#define CFG_MONITOR_LEN (384 * 1024) /* Reserve 384 kB for Mon */
Hm... maybe we should become defensive and use
#define CFG_MONITOR_LEN (0xFFFFFFFF - CFG_MONITOR_BASE + 1)
instead?
Best regards,
Wolfgang Denk

On Tuesday 29 April 2008, Wolfgang Denk wrote:
In message 1209471160-4453-1-git-send-email-sr@denx.de you wrote:
Signed-off-by: Stefan Roese sr@denx.de
...
-#define CFG_MONITOR_LEN (256 * 1024) /* Reserve 256 kB for Mon */ +#define CFG_MONITOR_LEN (384 * 1024) /* Reserve 384 kB for Mon */
Hm... maybe we should become defensive and use
#define CFG_MONITOR_LEN (0xFFFFFFFF - CFG_MONITOR_BASE + 1)
instead?
Yep, even better. I'll make the necessary changes.
Thanks.
Best regards, Stefan
===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office@denx.de =====================================================================
participants (2)
-
Stefan Roese
-
Wolfgang Denk