diff mbox

[U-Boot,v3,2/2] cmd_sf: Add print mesgs on sf read/write commands

Message ID 50a124ba-ac79-4b9f-9747-90aac19362da@CO9EHSMHS026.ehs.local
State Accepted
Delegated to: Jagannadha Sutradharudu Teki
Headers show

Commit Message

Jagannadha Sutradharudu Teki May 27, 2013, 8:11 p.m. UTC
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(-)

Comments

Jagan Teki June 2, 2013, 6:43 p.m. UTC | #1
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.
diff mbox

Patch

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;
 }