George-
Those two added lines do look good. It would be handy to have access to the second chip of user flash. Do you think it would be good to add some kind of support for the AMD boot flash? It claims to be CFI compliant, though tracking it as it moves through the memory map as the DIP switches are changed would be a bit tough. Perhaps if we updated the config file with #define's for the DIP switch settings, and based on that, it reasons out the correct memory settings for you?
-Michael Bendzick
-----Original Message----- From: 'George G. Davis' [mailto:gdavis@mvista.com] Sent: Tuesday, July 20, 2004 10:38 AM To: Michael Bendzick Cc: U-Boot-Users (E-mail) Subject: Re: [U-Boot-Users] No more omap1510inn/omap1610inn Maintainer?
On Mon, Jul 19, 2004 at 04:35:23PM -0500, Michael Bendzick wrote:
Attached is a patch that makes the omap1510inn board use the drivers/cfi_flash.c flash driver.
NOTE to Wolfgang: This patch is NOT for CVS yet.
Instead, it is for testing by other people to make sure this looks like a good change to make to this board configuration. If reports are positive,
I
will submit a patch for real.
It seems to work ok for me. How about adding this:
diff -u include/configs/omap1510inn.h include/configs/omap1510inn.h --- include/configs/omap1510inn.h 19 Jul 2004 20:50:28 -0000 +++ include/configs/omap1510inn.h 20 Jul 2004 15:18:48 -0000 @@ -159,7 +159,7 @@ /*----------------------------------------------------------------------- * FLASH and environment organization */ -#define CFG_MAX_FLASH_BANKS 1 /* max number of memory banks */ +#define CFG_MAX_FLASH_BANKS 2 /* max number of memory banks */ #define PHYS_FLASH_SIZE 0x01000000 /* 16MB */ #define CFG_MAX_FLASH_SECT (128) /* max number of sectors on one chip */ #define CFG_ENV_ADDR (CFG_FLASH_BASE + 0x020000) /* addr of environment */ @@ -168,6 +168,7 @@ #define CFG_FLASH_USE_BUFFER_WRITE 1 /* Use buffered writes (10x faster) */ #define CFG_MONITOR_BASE CFG_FLASH_BASE /* Monitor at beginning of flash */ #define CFG_MONITOR_LEN (128 << 10) /* Reserve 128kB */ +#define CFG_FLASH_BANKS_LIST { CFG_FLASH_BASE, CFG_FLASH_BASE + PHYS_FLASH_SIZE }
/* timeout values are in ticks */ #ifndef CFG_FLASH_CFI_DRIVER
FWIW, I'm having problems running U-Boot CVS HEAD on Innovator/OMAP1510 lately. The target hangs during early init. I haven't figured out what's wrong but I'm only able to use U-Boot CVS HEAD on Innovator/OMAP1510 while the BDI2000 is connected. If I disconnect the BDI2000, the darn target hangs during early init. Unfortunately I've no time to debug the early hang problem. But that problem is independent from your CFI patch anyway.
Thanks!
<snip>
-- Regards, George