Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: 7WnbDC4CB1Ht/O+ejlXjjshyau8DxEBCTntnI2S7h30aatPiXh6LooW8LQ4w8XzJcBUXCDypek
 73EBBIL9KAGMFT+/bYn+q9COTYnULmeGFTnRSp5IeIW8LXwylYmMp782f2eyt536L84GRBV/eT
 GT/HKgrGnsXoemoWzTkU4VxhyI/dauNJeSi/iMa8I+yF1MZZicguCA+Wd735c2edFDrtzjvQwU
 j2Q4XY6OrCYsvUMf0rsRfWKBQCsW2mC6ev7ER5Vjoav0rZwKsLcDHkjEPlFeji0+Qo+Wj6wlOx
 gdY=
From:   Avri Altman <Avri.Altman@wdc.com>
To:     "oracleks043021@gmail.com" <oracleks043021@gmail.com>,
        "beanhuo@micron.com" <beanhuo@micron.com>,
        "kenny.gibbons@oracle.com" <kenny.gibbons@oracle.com>,
        "kimito.sakata@oracle.com" <kimito.sakata@oracle.com>,
        "rkamdar@micron.com" <rkamdar@micron.com>,
        "chris@printf.net" <chris@printf.net>,
        "ulf.hansson@linaro.org" <ulf.hansson@linaro.org>
CC:     "linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH RESEND v4] Clean up and show effect of ERASE_GROUP_DEF
Thread-Topic: [PATCH RESEND v4] Clean up and show effect of ERASE_GROUP_DEF
Thread-Index: AQHXQigUvNZJ98bPLkavQe25bgO60arV/IoA
Date:   Thu, 6 May 2021 06:44:06 +0000
Message-ID: <DM6PR04MB6575DB4933121EBE949FB297FC589@DM6PR04MB6575.namprd04.prod.outlook.com>
References: <20210506032945.939-1-oracleks043021@gmail.com>
In-Reply-To: <20210506032945.939-1-oracleks043021@gmail.com>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 28eeb000-1a54-4545-2cf8-08d9105a584b
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 May 2021 06:44:06.2913
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: WXr4EUdAT+8cDUnudUrmquVtiDKhi4Hbf1cs0wvQw9l7eNQ/Q9ssqxzoZnjo0TLy/9XCA7EEsegw6IMN1JVULg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR04MB6509
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

This patch is only the incremental change from your last version.
I guess you forgot to amend your changes.

To merge your changes after you tested them, you can use:
#git status
It will show you which file you changed
#git add <the file you changed>
#git commit --amend
Will open your default editor with the commit message.
Update your commit history, with the appropriate v3 -> v4 changes.

Also add my reviewed-by tag, below Kenneth Gibbons:
Reviewed-by: Avri Altman <avri.altman@wdc.com>

Save the patch to some directory:
#git format-patch -1 -v4 -o <some path>

Now send the patch.  Use git send email:
#cd <the directory that contains the patch>
#git send-email <patch file name> --to 'Ulf Hansson <ulf.hansson@linaro.org=
>' --to linux-mmc@vger.kernel.org --cc linux-kernel@vger.kernel.org

It should have the same subject as before, be v4, without the RESEND label.
So before sending you should see the subject:
[PATCH v4] mmc-utils: Add eMMC erase command support

git send-email will automatically cc me, Bean, and  Kenneth.

Thanks,
Avri

P.S.
If by accident you saved those changes as a separate commit, instead of ame=
nding them,
Just squash them into the previous one.
Ping me privately  if you need help with that.

>=20
> From: Kimito Sakata <kimito.sakata@oracle.com>
>=20
> Replaced unused pointer with NULL on calls to strtol().
> Added logic to print High Capacity mode parameters of Erase unit size,
> Erase Timeout, Write protect Group Size if the EXT_CSD_ERASE_GrOUP_DEF
> bit 0 is enabled.
>=20
> Tested on X86 but the changes should work on all platforms.
>=20
> Co-developed-by: Bean Huo <beanhuo@micron.com>
> Signed-off-by: Kimito Sakata <kimito.sakata@oracle.com>
> Reviewed-by: Kenneth Gibbons <kenny.gibbons@oracle.com>
>=20
> Changelog:
> V3--V4:
>     1. Replace unused pointer var with NULL.
>     2. Added msg if ERASE_GROUP_DEF enabled for HC.
> v2--v3:
>     1. Remove redundant ifndef
>=20
> V1--V2:
>     1. refactor Kimito's original patch
>     2. change to use MMC_IOC_MULTI_CMD
>     3. add checkup if eMMC devie supports secure erase/trim
>=20
>=20
> ---
>  mmc_cmds.c | 25 ++++++++++++++++++++-----
>  1 file changed, 20 insertions(+), 5 deletions(-)
>=20
> diff --git a/mmc_cmds.c b/mmc_cmds.c
> index 3e36ff2..afa85b7 100644
> --- a/mmc_cmds.c
> +++ b/mmc_cmds.c
> @@ -2518,6 +2518,22 @@ static int erase(int dev_fd, __u32 argin, __u32
> start, __u32 end)
>  {
>         int ret =3D 0;
>         struct mmc_ioc_multi_cmd *multi_cmd;
> +       __u8 ext_csd[512];
> +
> +
> +       ret =3D read_extcsd(dev_fd, ext_csd);
> +       if (ret) {
> +               fprintf(stderr, "Could not read EXT_CSD\n");
> +               exit(1);
> +       }
> +       if (ext_csd[EXT_CSD_ERASE_GROUP_DEF] & 0x01) {
> +         fprintf(stderr, "High Capacity Erase Unit Size=3D%d bytes\n" \
> +                          "High Capacity Erase Timeout=3D%d ms\n" \
> +                          "High Capacity Write Protect Group Size=3D%d b=
ytes\n",
> +                          ext_csd[224]*0x80000,
> +                          ext_csd[223]*300,
> +                           ext_csd[221]*ext_csd[224]*0x80000);
> +       }
>=20
>         multi_cmd =3D calloc(1, sizeof(struct mmc_ioc_multi_cmd) +
>                            3 * sizeof(struct mmc_ioc_cmd));
> @@ -2559,7 +2575,6 @@ int do_erase(int nargs, char **argv)
>  {
>         int dev_fd, ret;
>         char *print_str;
> -       char **eptr =3D NULL;
>         __u8 ext_csd[512], checkup_mask =3D 0;
>         __u32 arg, start, end;
>=20
> @@ -2569,14 +2584,14 @@ int do_erase(int nargs, char **argv)
>         }
>=20
>         if (strstr(argv[2], "0x") || strstr(argv[2], "0X"))
> -               start =3D strtol(argv[2], eptr, 16);
> +               start =3D strtol(argv[2], NULL, 16);
>         else
> -               start =3D strtol(argv[2], eptr, 10);
> +               start =3D strtol(argv[2], NULL, 10);
>=20
>         if (strstr(argv[3], "0x") || strstr(argv[3], "0X"))
> -               end =3D strtol(argv[3], eptr, 16);
> +               end =3D strtol(argv[3], NULL, 16);
>         else
> -               end =3D strtol(argv[3], eptr, 10);
> +               end =3D strtol(argv[3], NULL, 10);
>=20
>         if (end < start) {
>                 fprintf(stderr, "erase start [0x%08x] > erase end [0x%08x=
]\n",
> --
> 2.31.1