[PATCH] drivers: mmc: support secure erase.

Jaehoon Chung jh80.chung at samsung.com
Sun Dec 27 23:40:08 CET 2020


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

> 



More information about the U-Boot mailing list