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

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.
-Michael Bendzick
-----Original Message----- From: 'George G. Davis' [mailto:gdavis@mvista.com] Sent: Friday, July 16, 2004 7:05 PM To: Michael Bendzick Cc: U-Boot-Users (E-mail) Subject: Re: [U-Boot-Users] No more omap1510inn/omap1610inn Maintainer?
On Fri, Jul 16, 2004 at 05:11:20PM -0500, Michael Bendzick wrote:
George-
Mostly I was interested in seeing if he would be interested in submitting
a
patch to the omap1510inn board to use the driver/cfi_flash.c driver. I tested that configuration in developing my recent patch for the file, and found it worked quite well. I certainly can't complain about writing to flash about 10x as fast as before.
Sounds agreeable. Just submit it and we can all give it a try. : )
If Kshitij is no longer around,
I'm sure he's probably still around but probably quite busy.
and others are interested in trying it out,
I'll give it a try.
I can make a patch for the modifications I made to that board
configuration.
Please do send a patch.
I don't see a reason to stick with the non-central, slow flash writing
code
for omap1510inn if a better option is available.
Agreed, it sounds good to me to use the common flash driver when/where possible.
FWIW, the OMAP1610 H2 uses Intel L18 flash parts. I'm not sure if the common cfi_flash.c driver supports this part yet.
-- Regards, George
-Michael Bendzick
-----Original Message----- From: George G. Davis [mailto:gdavis@mvista.com] Sent: Friday, July 16, 2004 4:43 PM To: Michael Bendzick Cc: 'wd@denx.de'; U-Boot-Users (E-mail) Subject: Re: [U-Boot-Users] No more omap1510inn/omap1610inn Maintainer?
On Thu, Jul 15, 2004 at 02:44:35PM -0500, Michael Bendzick wrote:
Wolfgang-
It appears that Kshitij Gupta, the maintainer for the omap1510inn & omap1610inn, is no longer reachable at his email address listed in the MAINTAINERS file. Is there anyone else that has been closely associated with this board,
I'm casually aquainted with the OMAP1610 H2 and Innovator/OMAP1510.
What's the problem?
or does anyone know of a new way to contact Kshitij?
Perhaps you can ask questions about those boards here and he or someone else may answer eventually? I'm sure he is just as busy as the rest us
are.
-- Regards, George
-Michael Bendzick Systems and Software Engineering Logic Product Development michael.b@logicpd.com 612-436-5122 www.logicpd.com
This SF.Net email is sponsored by BEA Weblogic Workshop FREE Java Enterprise J2EE developer tools! Get your free copy of BEA WebLogic Workshop 8.1 today. http://ads.osdn.com/?ad_id=4721&alloc_id=10040&op=click _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
This SF.Net email is sponsored by BEA Weblogic Workshop FREE Java Enterprise J2EE developer tools! Get your free copy of BEA WebLogic Workshop 8.1 today. http://ads.osdn.com/?ad_id=4721&alloc_id=10040&op=click _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users

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

On Tue, Jul 20, 2004 at 11:37:58AM -0400, 'George G. Davis' wrote:
<snip>
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.
Ok, nevermind, it seems to be working OK now w/o the BDI2000. I'll attribute this flaky behavior to my impatience with the usual POR problems I've had with this target since the very beginning.
<snip>
-- Regards, George
participants (2)
-
'George G. Davis'
-
Michael Bendzick