Re: [PATCH] drivers: mmc: support secure erase.

Hi Dennis,

On 12/26/20 5:44 PM, dennis laplacian1 wrote:

Hi Jaehoon Chung, for your comment: "ditto. if (IS_ENABLED()) instead of #if. And i don't know why doesn't check this in mmc_erase_t(). Then it doesn't need to pass arg as argument."

I check the support of secure erase in the mmc_berase and not in mmc_erase_t because mmc_erase_t is called multiple times for each block. So to prevent unnecessary checks, I check the support in mmc_berase function.

It there any problem to check a condition in mmc_erase_t()? or any dramatic degradation? And CONFIG_MMC_SECURE_ERASE is not enabled by default. So It doesn't need to add a variable.

Best Regards, Jaehoon Chung