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 <[email protected]> > > 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(-)
