diff mbox

[RESEND,2/2] mmc-utils: Check for ext_csd_rev only once

Message ID 20170320134301.2909-2-alexander.stein@systec-electronic.com (mailing list archive)
State New, archived
Headers show

Commit Message

Alexander Stein March 20, 2017, 1:43 p.m. UTC 
Signed-off-by: Alexander Stein <alexander.stein@systec-electronic.com>
---
 mmc_cmds.c | 9 ---------
 1 file changed, 9 deletions(-)

Comments

Jaehoon Chung March 21, 2017, 10:27 p.m. UTC | #1 
On 03/20/2017 10:43 PM, Alexander Stein wrote:
> Signed-off-by: Alexander Stein <alexander.stein@systec-electronic.com>

Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>

> ---
>  mmc_cmds.c | 9 ---------
>  1 file changed, 9 deletions(-)
> 
> diff --git a/mmc_cmds.c b/mmc_cmds.c
> index 728ad77..d7215bb 100644
> --- a/mmc_cmds.c
> +++ b/mmc_cmds.c
> @@ -1736,19 +1736,10 @@ int do_read_extcsd(int nargs, char **argv)
>  	if (ext_csd_rev >= 7) {
>  		printf("eMMC Firmware Version: %s\n",
>  			(char*)&ext_csd[EXT_CSD_FIRMWARE_VERSION]);
> -	}
> -
> -	if (ext_csd_rev >= 7) {
>  		printf("eMMC Life Time Estimation A [EXT_CSD_DEVICE_LIFE_TIME_EST_TYP_A]: 0x%02x\n",
>  			ext_csd[EXT_CSD_DEVICE_LIFE_TIME_EST_TYP_A]);
> -	}
> -
> -	if (ext_csd_rev >= 7) {
>  		printf("eMMC Life Time Estimation B [EXT_CSD_DEVICE_LIFE_TIME_EST_TYP_B]: 0x%02x\n",
>  			ext_csd[EXT_CSD_DEVICE_LIFE_TIME_EST_TYP_B]);
> -	}
> -
> -	if (ext_csd_rev >= 7) {
>  		printf("eMMC Pre EOL information [EXT_CSD_PRE_EOL_INFO]: 0x%02x\n",
>  			ext_csd[EXT_CSD_PRE_EOL_INFO]);
>  	}
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/mmc_cmds.c b/mmc_cmds.c
index 728ad77..d7215bb 100644
--- a/mmc_cmds.c
+++ b/mmc_cmds.c
@@ -1736,19 +1736,10 @@  int do_read_extcsd(int nargs, char **argv)
 	if (ext_csd_rev >= 7) {
 		printf("eMMC Firmware Version: %s\n",
 			(char*)&ext_csd[EXT_CSD_FIRMWARE_VERSION]);
-	}
-
-	if (ext_csd_rev >= 7) {
 		printf("eMMC Life Time Estimation A [EXT_CSD_DEVICE_LIFE_TIME_EST_TYP_A]: 0x%02x\n",
 			ext_csd[EXT_CSD_DEVICE_LIFE_TIME_EST_TYP_A]);
-	}
-
-	if (ext_csd_rev >= 7) {
 		printf("eMMC Life Time Estimation B [EXT_CSD_DEVICE_LIFE_TIME_EST_TYP_B]: 0x%02x\n",
 			ext_csd[EXT_CSD_DEVICE_LIFE_TIME_EST_TYP_B]);
-	}
-
-	if (ext_csd_rev >= 7) {
 		printf("eMMC Pre EOL information [EXT_CSD_PRE_EOL_INFO]: 0x%02x\n",
 			ext_csd[EXT_CSD_PRE_EOL_INFO]);
 	}