On Mon, 16 Jan 2023 at 12:19, Christian Löhle <CLoehle@xxxxxxxxxxxxxx> wrote:
>
> Check for erase specific R1 errors so e.g. an OOR erase is not
> reported as successful when it never executed.
>
> There could be checks for more error bits but R1_ERASE_SEQ_ERROR
> on CMD38 should catch all that are reported by hardware anyway.
>
> Signed-off-by: Christian Loehle <cloehle@xxxxxxxxxxxxxx>
Applied to git.kernel.org/pub/scm/utils/mmc/mmc-utils.git master, thanks!
Kind regards
Uffe
> ---
> -v3: Check for ERASE_PARAM on CMD36
> -v2: Remove unneeded error bit checking
> mmc_cmds.c | 13 ++++++++++++-
> 1 file changed, 12 insertions(+), 1 deletion(-)
>
> diff --git a/mmc_cmds.c b/mmc_cmds.c
> index e6d3273..6c50bbe 100644
> --- a/mmc_cmds.c
> +++ b/mmc_cmds.c
> @@ -54,7 +54,6 @@
> #define WPTYPE_PWRON 2
> #define WPTYPE_PERM 3
>
> -
> int read_extcsd(int fd, __u8 *ext_csd)
> {
> int ret = 0;
> @@ -2668,6 +2667,18 @@ static int erase(int dev_fd, __u32 argin, __u32 start, __u32 end)
> if (ret)
> perror("Erase multi-cmd ioctl");
>
> + /* Does not work for SPI cards */
> + if (multi_cmd->cmds[1].response[0] & R1_ERASE_PARAM) {
> + fprintf(stderr, "Erase start response: 0x%08x\n",
> + multi_cmd->cmds[0].response[0]);
> + ret = -EIO;
> + }
> + if (multi_cmd->cmds[2].response[0] & R1_ERASE_SEQ_ERROR) {
> + fprintf(stderr, "Erase response: 0x%08x\n",
> + multi_cmd->cmds[2].response[0]);
> + ret = -EIO;
> + }
> +
> free(multi_cmd);
> return ret;
> }
> --
> 2.37.3
>
>
> Hyperstone GmbH | Reichenaustr. 39a | 78467 Konstanz
> Managing Director: Dr. Jan Peter Berns.
> Commercial register of local courts: Freiburg HRB381782
>
