cmd: mmc: Expand bkops handling

Add more capable "bkops" command which allows enabling and disabling both
manual and automatic bkops. The existing 'mmc bkops-enable' subcommand is
poorly named to cover all the possibilities, hence the new-ish subcommand.
Note that both commands are wrappers around the same common code.

Signed-off-by: Marek Vasut <marex@denx.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
This commit is contained in:
Marek Vasut 2023-01-05 15:19:08 +01:00 committed by Jaehoon Chung
parent 30db474704
commit cf1f7355ae
3 changed files with 61 additions and 16 deletions

View file

@ -1020,16 +1020,12 @@ static int do_mmc_setdsr(struct cmd_tbl *cmdtp, int flag,
}
#ifdef CONFIG_CMD_BKOPS_ENABLE
static int do_mmc_bkops_enable(struct cmd_tbl *cmdtp, int flag,
int argc, char *const argv[])
static int mmc_bkops_common(char *device, bool autobkops, bool enable)
{
int dev;
struct mmc *mmc;
int dev;
if (argc != 2)
return CMD_RET_USAGE;
dev = dectoul(argv[1], NULL);
dev = dectoul(device, NULL);
mmc = init_mmc_device(dev, false);
if (!mmc)
@ -1040,7 +1036,41 @@ static int do_mmc_bkops_enable(struct cmd_tbl *cmdtp, int flag,
return CMD_RET_FAILURE;
}
return mmc_set_bkops_enable(mmc);
return mmc_set_bkops_enable(mmc, autobkops, enable);
}
static int do_mmc_bkops(struct cmd_tbl *cmdtp, int flag,
int argc, char * const argv[])
{
bool autobkops, enable;
if (argc != 4)
return CMD_RET_USAGE;
if (!strcmp(argv[2], "manual"))
autobkops = false;
else if (!strcmp(argv[2], "auto"))
autobkops = true;
else
return CMD_RET_FAILURE;
if (!strcmp(argv[3], "disable"))
enable = false;
else if (!strcmp(argv[3], "enable"))
enable = true;
else
return CMD_RET_FAILURE;
return mmc_bkops_common(argv[1], autobkops, enable);
}
static int do_mmc_bkops_enable(struct cmd_tbl *cmdtp, int flag,
int argc, char * const argv[])
{
if (argc != 2)
return CMD_RET_USAGE;
return mmc_bkops_common(argv[1], false, true);
}
#endif
@ -1102,6 +1132,7 @@ static struct cmd_tbl cmd_mmc[] = {
U_BOOT_CMD_MKENT(setdsr, 2, 0, do_mmc_setdsr, "", ""),
#ifdef CONFIG_CMD_BKOPS_ENABLE
U_BOOT_CMD_MKENT(bkops-enable, 2, 0, do_mmc_bkops_enable, "", ""),
U_BOOT_CMD_MKENT(bkops, 4, 0, do_mmc_bkops, "", ""),
#endif
};
@ -1188,6 +1219,8 @@ U_BOOT_CMD(
#ifdef CONFIG_CMD_BKOPS_ENABLE
"mmc bkops-enable <dev> - enable background operations handshake on device\n"
" WARNING: This is a write-once setting.\n"
"mmc bkops <dev> [auto|manual] [enable|disable]\n"
" - configure background operations handshake on device\n"
#endif
);

View file

@ -3127,9 +3127,10 @@ int mmc_init_device(int num)
#endif
#ifdef CONFIG_CMD_BKOPS_ENABLE
int mmc_set_bkops_enable(struct mmc *mmc)
int mmc_set_bkops_enable(struct mmc *mmc, bool autobkops, bool enable)
{
int err;
u32 bit = autobkops ? BIT(1) : BIT(0);
ALLOC_CACHE_ALIGN_BUFFER(u8, ext_csd, MMC_MAX_BLOCK_LEN);
err = mmc_send_ext_csd(mmc, ext_csd);
@ -3143,18 +3144,21 @@ int mmc_set_bkops_enable(struct mmc *mmc)
return -EMEDIUMTYPE;
}
if (ext_csd[EXT_CSD_BKOPS_EN] & 0x1) {
if (enable && (ext_csd[EXT_CSD_BKOPS_EN] & bit)) {
puts("Background operations already enabled\n");
return 0;
}
err = mmc_switch(mmc, EXT_CSD_CMD_SET_NORMAL, EXT_CSD_BKOPS_EN, 1);
err = mmc_switch(mmc, EXT_CSD_CMD_SET_NORMAL, EXT_CSD_BKOPS_EN,
enable ? bit : 0);
if (err) {
puts("Failed to enable manual background operations\n");
printf("Failed to %sable manual background operations\n",
enable ? "en" : "dis");
return err;
}
puts("Enabled manual background operations\n");
printf("%sabled %s background operations\n",
enable ? "En" : "Dis", autobkops ? "auto" : "manual");
return 0;
}

View file

@ -892,9 +892,17 @@ int mmc_rpmb_write(struct mmc *mmc, void *addr, unsigned short blk,
int mmc_rpmb_route_frames(struct mmc *mmc, void *req, unsigned long reqlen,
void *rsp, unsigned long rsplen);
#ifdef CONFIG_CMD_BKOPS_ENABLE
int mmc_set_bkops_enable(struct mmc *mmc);
#endif
/**
* mmc_set_bkops_enable() - enable background operations
* @param mmc Pointer to a MMC device struct
* @param autobkops Enable automatic bkops, not manual bkops
* @param enable Enable bkops, not disable
*
* Enable or disable automatic or manual background operation of the eMMC.
*
* Return: 0 on success, <0 on error.
*/
int mmc_set_bkops_enable(struct mmc *mmc, bool autobkops, bool enable);
/**
* Start device initialization and return immediately; it does not block on