* [U-Boot] [PATCH v3 1/2] cmd_sf: Add print mesg for 'sf erase' command [not found] <1369685506-30740-1-git-send-email-jaganna@xilinx.com> @ 2013-05-27 20:11 ` Jagannadha Sutradharudu Teki 2013-06-02 18:44 ` [U-Boot] [U-Boot, v3, " Jagan Teki 2013-05-27 20:11 ` [U-Boot] [PATCH v3 2/2] cmd_sf: Add print mesgs on sf read/write commands Jagannadha Sutradharudu Teki 1 sibling, 1 reply; 4+ messages in thread From: Jagannadha Sutradharudu Teki @ 2013-05-27 20:11 UTC (permalink / raw) To: u-boot This patch adds a print messages while using 'sf erase' command to make sure that how many bytes erased in flash device. Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com> --- common/cmd_sf.c | 8 +++----- drivers/mtd/spi/spi_flash.c | 7 ++----- 2 files changed, 5 insertions(+), 10 deletions(-) diff --git a/common/cmd_sf.c b/common/cmd_sf.c index 1daff70..12d1aca 100644 --- a/common/cmd_sf.c +++ b/common/cmd_sf.c @@ -305,12 +305,10 @@ static int do_spi_flash_erase(int argc, char * const argv[]) } ret = spi_flash_erase(flash, offset, len); - if (ret) { - printf("SPI flash %s failed\n", argv[0]); - return 1; - } + printf("SF: %zu bytes @ %#x Erased: %s\n", (size_t)len, (u32)offset, + ret ? "ERROR" : "OK"); - return 0; + return ret == 0 ? 0 : 1; } #ifdef CONFIG_CMD_SF_TEST diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c index d2bee3a..77938d3 100644 --- a/drivers/mtd/spi/spi_flash.c +++ b/drivers/mtd/spi/spi_flash.c @@ -206,7 +206,7 @@ int spi_flash_cmd_wait_ready(struct spi_flash *flash, unsigned long timeout) int spi_flash_cmd_erase(struct spi_flash *flash, u32 offset, size_t len) { - u32 start, end, erase_size; + u32 end, erase_size; int ret; u8 cmd[4]; @@ -226,8 +226,7 @@ int spi_flash_cmd_erase(struct spi_flash *flash, u32 offset, size_t len) cmd[0] = CMD_ERASE_4K; else cmd[0] = CMD_ERASE_64K; - start = offset; - end = start + len; + end = offset + len; while (offset < end) { spi_flash_addr(offset, cmd); @@ -251,8 +250,6 @@ int spi_flash_cmd_erase(struct spi_flash *flash, u32 offset, size_t len) goto out; } - debug("SF: Successfully erased %zu bytes @ %#x\n", len, start); - out: spi_release_bus(flash->spi); return ret; -- 1.7.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [U-Boot, v3, 1/2] cmd_sf: Add print mesg for 'sf erase' command 2013-05-27 20:11 ` [U-Boot] [PATCH v3 1/2] cmd_sf: Add print mesg for 'sf erase' command Jagannadha Sutradharudu Teki @ 2013-06-02 18:44 ` Jagan Teki 0 siblings, 0 replies; 4+ messages in thread From: Jagan Teki @ 2013-06-02 18:44 UTC (permalink / raw) To: u-boot On 27-05-2013 15:41, Jagannadha Sutradharudu Teki wrote: > This patch adds a print messages while using 'sf erase' command > to make sure that how many bytes erased in flash device. > > Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com> > > --- > common/cmd_sf.c | 8 +++----- > drivers/mtd/spi/spi_flash.c | 7 ++----- > 2 files changed, 5 insertions(+), 10 deletions(-) > > diff --git a/common/cmd_sf.c b/common/cmd_sf.c > index 1daff70..12d1aca 100644 > --- a/common/cmd_sf.c > +++ b/common/cmd_sf.c > @@ -305,12 +305,10 @@ static int do_spi_flash_erase(int argc, char * const argv[]) > } > > ret = spi_flash_erase(flash, offset, len); > - if (ret) { > - printf("SPI flash %s failed\n", argv[0]); > - return 1; > - } > + printf("SF: %zu bytes @ %#x Erased: %s\n", (size_t)len, (u32)offset, > + ret ? "ERROR" : "OK"); > > - return 0; > + return ret == 0 ? 0 : 1; > } > > #ifdef CONFIG_CMD_SF_TEST > diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c > index d2bee3a..77938d3 100644 > --- a/drivers/mtd/spi/spi_flash.c > +++ b/drivers/mtd/spi/spi_flash.c > @@ -206,7 +206,7 @@ int spi_flash_cmd_wait_ready(struct spi_flash *flash, unsigned long timeout) > > int spi_flash_cmd_erase(struct spi_flash *flash, u32 offset, size_t len) > { > - u32 start, end, erase_size; > + u32 end, erase_size; > int ret; > u8 cmd[4]; > > @@ -226,8 +226,7 @@ int spi_flash_cmd_erase(struct spi_flash *flash, u32 offset, size_t len) > cmd[0] = CMD_ERASE_4K; > else > cmd[0] = CMD_ERASE_64K; > - start = offset; > - end = start + len; > + end = offset + len; > > while (offset < end) { > spi_flash_addr(offset, cmd); > @@ -251,8 +250,6 @@ int spi_flash_cmd_erase(struct spi_flash *flash, u32 offset, size_t len) > goto out; > } > > - debug("SF: Successfully erased %zu bytes @ %#x\n", len, start); > - > out: > spi_release_bus(flash->spi); > return ret; Applied to u-boot-spi/master -- Thanks, Jagan. ^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH v3 2/2] cmd_sf: Add print mesgs on sf read/write commands [not found] <1369685506-30740-1-git-send-email-jaganna@xilinx.com> 2013-05-27 20:11 ` [U-Boot] [PATCH v3 1/2] cmd_sf: Add print mesg for 'sf erase' command Jagannadha Sutradharudu Teki @ 2013-05-27 20:11 ` Jagannadha Sutradharudu Teki 2013-06-02 18:43 ` [U-Boot] [U-Boot, v3, " Jagan Teki 1 sibling, 1 reply; 4+ messages in thread From: Jagannadha Sutradharudu Teki @ 2013-05-27 20:11 UTC (permalink / raw) To: u-boot This patch adds a print messages while using 'sf read' and 'sf write' commands to make sure that how many bytes read/written from/into flash device. Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com> --- common/cmd_sf.c | 26 +++++++++++++++----------- drivers/mtd/spi/spi_flash.c | 3 --- 2 files changed, 15 insertions(+), 14 deletions(-) diff --git a/common/cmd_sf.c b/common/cmd_sf.c index 12d1aca..3a40444 100644 --- a/common/cmd_sf.c +++ b/common/cmd_sf.c @@ -234,7 +234,7 @@ static int do_spi_flash_read_write(int argc, char * const argv[]) unsigned long len; void *buf; char *endp; - int ret; + int ret = 1; if (argc < 4) return -1; @@ -264,19 +264,23 @@ static int do_spi_flash_read_write(int argc, char * const argv[]) if (strcmp(argv[0], "update") == 0) ret = spi_flash_update(flash, offset, len, buf); - else if (strcmp(argv[0], "read") == 0) - ret = spi_flash_read(flash, offset, len, buf); - else - ret = spi_flash_write(flash, offset, len, buf); + else if (strncmp(argv[0], "read", 4) == 0 || + strncmp(argv[0], "write", 5) == 0) { + int read; + + read = strncmp(argv[0], "read", 4) == 0; + if (read) + ret = spi_flash_read(flash, offset, len, buf); + else + ret = spi_flash_write(flash, offset, len, buf); + + printf("SF: %zu bytes @ %#x %s: %s\n", (size_t)len, (u32)offset, + read ? "Read" : "Written", ret ? "ERROR" : "OK"); + } unmap_physmem(buf, len); - if (ret) { - printf("SPI flash %s failed\n", argv[0]); - return 1; - } - - return 0; + return ret == 0 ? 0 : 1; } static int do_spi_flash_erase(int argc, char * const argv[]) diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c index 77938d3..aeb1ccb 100644 --- a/drivers/mtd/spi/spi_flash.c +++ b/drivers/mtd/spi/spi_flash.c @@ -125,9 +125,6 @@ int spi_flash_cmd_write_multi(struct spi_flash *flash, u32 offset, } } - debug("SF: program %s %zu bytes @ %#x\n", - ret ? "failure" : "success", len, offset); - spi_release_bus(flash->spi); return ret; } -- 1.7.4 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [U-Boot, v3, 2/2] cmd_sf: Add print mesgs on sf read/write commands 2013-05-27 20:11 ` [U-Boot] [PATCH v3 2/2] cmd_sf: Add print mesgs on sf read/write commands Jagannadha Sutradharudu Teki @ 2013-06-02 18:43 ` Jagan Teki 0 siblings, 0 replies; 4+ messages in thread From: Jagan Teki @ 2013-06-02 18:43 UTC (permalink / raw) To: u-boot On 27-05-2013 15:41, Jagannadha Sutradharudu Teki wrote: > This patch adds a print messages while using 'sf read' and > 'sf write' commands to make sure that how many bytes read/written > from/into flash device. > > Signed-off-by: Jagannadha Sutradharudu Teki <jaganna@xilinx.com> > > --- > common/cmd_sf.c | 26 +++++++++++++++----------- > drivers/mtd/spi/spi_flash.c | 3 --- > 2 files changed, 15 insertions(+), 14 deletions(-) > > diff --git a/common/cmd_sf.c b/common/cmd_sf.c > index 12d1aca..3a40444 100644 > --- a/common/cmd_sf.c > +++ b/common/cmd_sf.c > @@ -234,7 +234,7 @@ static int do_spi_flash_read_write(int argc, char * const argv[]) > unsigned long len; > void *buf; > char *endp; > - int ret; > + int ret = 1; > > if (argc < 4) > return -1; > @@ -264,19 +264,23 @@ static int do_spi_flash_read_write(int argc, char * const argv[]) > > if (strcmp(argv[0], "update") == 0) > ret = spi_flash_update(flash, offset, len, buf); > - else if (strcmp(argv[0], "read") == 0) > - ret = spi_flash_read(flash, offset, len, buf); > - else > - ret = spi_flash_write(flash, offset, len, buf); > + else if (strncmp(argv[0], "read", 4) == 0 || > + strncmp(argv[0], "write", 5) == 0) { > + int read; > + > + read = strncmp(argv[0], "read", 4) == 0; > + if (read) > + ret = spi_flash_read(flash, offset, len, buf); > + else > + ret = spi_flash_write(flash, offset, len, buf); > + > + printf("SF: %zu bytes @ %#x %s: %s\n", (size_t)len, (u32)offset, > + read ? "Read" : "Written", ret ? "ERROR" : "OK"); > + } > > unmap_physmem(buf, len); > > - if (ret) { > - printf("SPI flash %s failed\n", argv[0]); > - return 1; > - } > - > - return 0; > + return ret == 0 ? 0 : 1; > } > > static int do_spi_flash_erase(int argc, char * const argv[]) > diff --git a/drivers/mtd/spi/spi_flash.c b/drivers/mtd/spi/spi_flash.c > index 77938d3..aeb1ccb 100644 > --- a/drivers/mtd/spi/spi_flash.c > +++ b/drivers/mtd/spi/spi_flash.c > @@ -125,9 +125,6 @@ int spi_flash_cmd_write_multi(struct spi_flash *flash, u32 offset, > } > } > > - debug("SF: program %s %zu bytes @ %#x\n", > - ret ? "failure" : "success", len, offset); > - > spi_release_bus(flash->spi); > return ret; > } Applied to u-boot-spi/master -- Thanks, Jagan. ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-06-02 18:44 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1369685506-30740-1-git-send-email-jaganna@xilinx.com>
2013-05-27 20:11 ` [U-Boot] [PATCH v3 1/2] cmd_sf: Add print mesg for 'sf erase' command Jagannadha Sutradharudu Teki
2013-06-02 18:44 ` [U-Boot] [U-Boot, v3, " Jagan Teki
2013-05-27 20:11 ` [U-Boot] [PATCH v3 2/2] cmd_sf: Add print mesgs on sf read/write commands Jagannadha Sutradharudu Teki
2013-06-02 18:43 ` [U-Boot] [U-Boot, v3, " Jagan Teki
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox