From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ravik Hasija Date: Wed, 31 Mar 2021 11:15:49 -0700 Subject: [PATCH v2] cmd: gpt: remove redundant print messages In-Reply-To: <1617140080-24756-1-git-send-email-rahasij@linux.microsoft.com> References: <1617140080-24756-1-git-send-email-rahasij@linux.microsoft.com> Message-ID: <1617214549-26153-1-git-send-email-rahasij@linux.microsoft.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Removing printfs from do_gpt API as the messages are generic and does not specify error codes for failure cases. Signed-off-by: Ravik Hasija Reviewed-by: Patrick Delaunay --- Changes for v2: - Updated usage message for gpt read - Added removed printfs for write & verify - Rearranged verify printf --- cmd/gpt.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/cmd/gpt.c b/cmd/gpt.c index 76a95ade6c..ba0fa4c69c 100644 --- a/cmd/gpt.c +++ b/cmd/gpt.c @@ -971,9 +971,11 @@ static int do_gpt(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) if ((strcmp(argv[1], "write") == 0) && (argc == 5)) { printf("Writing GPT: "); ret = gpt_default(blk_dev_desc, argv[4]); + ret ? printf("error!\n") : printf("success!\n"); } else if ((strcmp(argv[1], "verify") == 0)) { - ret = gpt_verify(blk_dev_desc, argv[4]); printf("Verify GPT: "); + ret = gpt_verify(blk_dev_desc, argv[4]); + ret ? printf("error!\n") : printf("success!\n"); } else if ((strcmp(argv[1], "setenv") == 0)) { ret = gpt_setenv(blk_dev_desc, argv[4]); } else if ((strcmp(argv[1], "enumerate") == 0)) { @@ -992,11 +994,9 @@ static int do_gpt(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) } if (ret) { - printf("error!\n"); return CMD_RET_FAILURE; } - printf("success!\n"); return CMD_RET_SUCCESS; } @@ -1017,7 +1017,7 @@ U_BOOT_CMD(gpt, CONFIG_SYS_MAXARGS, 1, do_gpt, " gpt_partition_name, gpt_partition_entry\n" " gpt enumerate mmc 0\n" " - store list of partitions to gpt_partition_list environment variable\n" - " read \n" + " gpt read \n" " - read GPT into a data structure for manipulation\n" " gpt guid \n" " - print disk GUID\n" -- 2.17.1