[PATCH 0/3] Code improvements and argument checks for mmc dev subcommand

Anshul Dalal anshuld at ti.com
Mon Feb 2 08:51:31 CET 2026


On Fri Jan 30, 2026 at 11:33 PM IST, Yao Zi wrote:
> This series reduces code duplication by utilizing fallthrough, then adds
> extra checks for arguments of mmc dev to ensure they're valid numbers,
> preventing unexpected behavior.
>
> The last patch replaces an integer literal returned in do_mmc_dev() by
> its corresponding symbolic constant, CMD_RET_FAILURE to improve
> readability, no function change is involved.

The overall patch series looks good to me, thanks for the cleanup!

Tested-by: Anshul Dalal <anshuld at ti.com>

>
> Yao Zi (3):
>   cmd: mmc: Simplify dev subcommand handling
>   cmd: mmc: Check whether arguments are valid numbers in dev subcommand
>   cmd: mmc: Return symbolic value when part switching fails in mmc dev
>
>  cmd/mmc.c | 55 ++++++++++++++++++++++++++++++++-----------------------
>  1 file changed, 32 insertions(+), 23 deletions(-)



More information about the U-Boot mailing list