From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Wed, 28 Jan 2009 13:54:40 +0100 Subject: [U-Boot] [PATCH 12/31] 82xx, mgcoge: corrected wrong flash sizes In-Reply-To: <20090128120154.53390832E416@gemini.denx.de> References: <498027D6.3070600@denx.de> <20090128120154.53390832E416@gemini.denx.de> Message-ID: <49805590.5040405@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Wolfgang, Wolfgang Denk wrote: > In message <498027D6.3070600@denx.de> you wrote: >> flash size of mgcoge corrected >> boot to 4M (was 32) >> cfg to 32M (was 64) >> >> Signed-off-by: Thomas Herzmann >> Signed-off-by: Heiko Schocher >> --- >> include/configs/mgcoge.h | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/include/configs/mgcoge.h b/include/configs/mgcoge.h >> index a754f0b..f307382 100644 >> --- a/include/configs/mgcoge.h >> +++ b/include/configs/mgcoge.h >> @@ -99,14 +99,14 @@ >> >> #define CONFIG_SYS_SDRAM_BASE 0x00000000 >> #define CONFIG_SYS_FLASH_BASE 0xFE000000 >> -#define CONFIG_SYS_FLASH_SIZE 32 >> +#define CONFIG_SYS_FLASH_SIZE 4 > > That's broken. The flash size should never be defined. Why don;t you > use auto-detection as U-Boot was designed for? I use the CFI driver ... have to check for what this defines are used (Hopefully they are no more used). Argh... i used it. I change this. thanks Heiko -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany