Re: [U-Boot] [PATCH 3/3] [OneNAND] Flex-OneNAND boundary settings

2009-11-19 Thread Amul Kumar Saha
Should I send the patch again with version updated? Regards, Amul Kumar Saha ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3] [OneNAND] Flex-OneNAND boundary settings

2009-11-10 Thread Amul Kumar Saha
Add command for changing Flex-OneNAND SLC / MLC boundary. Also onenand commands work for Flex-OneNAND. Signed-off-by: Rohit Hagargundgi Signed-off-by: Amul Kumar Saha --- common/cmd_onenand.c | 106 +++--- include/configs/apollon.h |2 2 files

Re: [U-Boot] [PATCH 3/3] [OneNAND] Flex-OneNAND boundary settings

2009-11-10 Thread Amul Kumar Saha
Hi Scott, > On Fri, Nov 06, 2009 at 05:17:44PM +0530, Amul Kumar Saha wrote: >> - printk("Bad blocks %d at 0x%x\n", >> -(u32)(ofs >> this->erase_shift), (u32)ofs); >> + printk("Bad blocks %lu at 0x%x\n", >> +(u32)onenand_blo

Re: [U-Boot] [PATCH 1/3] [OneNAND] Flex-OneNAND driver support

2009-11-03 Thread Amul Kumar Saha
. Note that this > function is not specified as inline in Linux. > Accepted and corrected > There are some other fairly significant differences with Linux later in > the patch -- is this due to missing functionality that u-boot doesn't > need, or something else? > Yes, there a

Re: [U-Boot] [PATCH 0/3] [OneNAND] Flex-OneNAND driver support

2009-11-03 Thread Amul Kumar Saha
Hi Scott, > > You seem to have stripped a copyright notice that was added in the Linux > version of patch 1/3. > Updated in the current patch. Regards, Amul Kumar Saha ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.den

Re: [U-Boot] [PATCH 3/3] [OneNAND] Flex-OneNAND boundary settings

2009-11-03 Thread Amul Kumar Saha
Add command for changing Flex-OneNAND SLC / MLC boundary. Also onenand commands work for Flex-OneNAND. Signed-off-by: Rohit Hagargundgi Signed-off-by: Amul Kumar Saha --- common/cmd_onenand.c | 90 +- include/configs/apollon.h |2 - 2

Re: [U-Boot] [PATCH 2/3] [OneNAND] ENV Variable support for Flex-OneNAND

2009-11-03 Thread Amul Kumar Saha
Define and use CONFIG_ENV_ADDR_FLEX and CONFIG_ENV_SIZE_FLEX for storing environment variables. Signed-off-by: Rohit Hagargundgi Signed-off-by: Amul Kumar Saha --- common/env_onenand.c | 10 ++ include/configs/apollon.h |2 ++ 2 files changed, 12 insertions(+) diff --git a

Re: [U-Boot] [PATCH 1/3] [OneNAND] Flex-OneNAND driver support

2009-11-03 Thread Amul Kumar Saha
This patch adds support for Flex-OneNAND devices. Signed-off-by: Rohit Hagargundgi Signed-off-by: Amul Kumar Saha --- drivers/mtd/onenand/onenand_base.c | 740 +++- drivers/mtd/onenand/onenand_bbt.c | 14 drivers/mtd/onenand/onenand_uboot.c |4 include

[U-Boot] [PATCH 2/3] [OneNAND] ENV Variable support for Flex-OneNAND

2009-09-23 Thread Amul Kumar Saha
Define and use CONFIG_ENV_ADDR_FLEX and CONFIG_ENV_SIZE_FLEX for storing environment variables. Signed-off-by: Rohit Hagargundgi Signed-off-by: Amul Kumar Saha --- common/env_onenand.c | 10 ++ include/configs/apollon.h |2 ++ 2 files changed, 12 insertions(+) diff --git a

[U-Boot] [PATCH 1/3] [OneNAND] Flex-OneNAND driver support

2009-09-23 Thread Amul Kumar Saha
This patch adds support for Flex-OneNAND devices. Signed-off-by: Rohit Hagargundgi Signed-off-by: Amul Kumar Saha --- drivers/mtd/onenand/onenand_base.c | 733 +++- drivers/mtd/onenand/onenand_bbt.c | 14 drivers/mtd/onenand/onenand_uboot.c |4 include

[U-Boot] [PATCH 3/3] [OneNAND] Flex-OneNAND boundary settings

2009-09-23 Thread Amul Kumar Saha
Add command for changing Flex-OneNAND SLC / MLC boundary. Also onenand commands work for Flex-OneNAND. Signed-off-by: Rohit Hagargundgi Signed-off-by: Amul Kumar Saha --- common/cmd_onenand.c | 90 +- include/configs/apollon.h |2 - 2

[U-Boot] [PATCH 0/3] [OneNAND] Flex-OneNAND driver support

2009-09-23 Thread Amul Kumar Saha
. Patch 3/3: Flex-OneNAND SLC / MLC boundary setting. Regards, Amul Kumar Saha ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v2 4/4] Flex-OneNAND boundary setting command

2009-03-26 Thread Amul Kumar Saha
OneNAND commands now work for Flex-OneNAND. Add command for changing Flex-OneNAND SLC / MLC boundary. Signed-off-by: Rohit Hagargundgi Signed-off-by: Amul Kumar Saha --- common/cmd_onenand.c | 103 -- include/configs/apollon.h |2 2 files

Re: [U-Boot] [PATCH v2 4/4] Flex-OneNAND boundary setting command

2009-03-26 Thread Amul Kumar Saha
Hi Scott, >> I believe that typecasting a UL to an int, is OK. >> Do let me know. > > The compiler will implicitly cast unsigned long to int. The explicit cast > is unnecessary clutter which makes it harder to find casts that aren't OK. Accepted and changed. Thank You Amul. ___

Re: [U-Boot] [PATCH v2 4/4] Flex-OneNAND boundary setting command

2009-03-25 Thread Amul Kumar Saha
Hi Scott, > >> + int blocks = (int) onenand_block(this, from + len) >> +- onenand_block(this, from); > > Why the (int) cast? onenand_block() already returns int. > Resolved >> - if (end_block > (mtd->size >> this->erase_shift)) >> - end_block = mtd->size >> this->erase_shift; >> + if (e

Re: [U-Boot] [PATCH v2 4/4] Flex-OneNAND boundary setting command

2009-03-23 Thread Amul Kumar Saha
t;> "to/from memory address 'addr', skipping bad blocks.\n" >> - "onenand erase [force] [off size] - erase 'size' bytes from\n" >> + "onenand erase [force] [off size] - erase 'size' bytes from off\n" > > Quotes aroun

Re: [U-Boot] [PATCH v2 1/4] Flex-OneNAND driver

2009-03-17 Thread Amul Kumar Saha
eshift++; >> + } else { >> + mtd->numeraseregions -= 1; >> + mtd->eraseregions[i].numblocks += >> + this->boundary[die] + 1; >> + ofs += (this->boundary[die] + 1) << (eraseshift - 1); >> + } >> + if (this->boundary[die] != maxbdry) { &