[U-Boot-Users] No more omap1510inn/omap1610inn Maintainer?

[Michael Bendzick]

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 at 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