[PATCH] cmd: mmc: Drop trailing space in Name: before newline
Mattijs Korpershoek
mkorpershoek at kernel.org
Tue Feb 17 10:50:21 CET 2026
Hi Marek,
Thank you for the patch.
On Sun, Feb 08, 2026 at 15:51, Marek Vasut <marek.vasut+renesas at mailbox.org> wrote:
> The Name: line of 'mmc info' command prints a trailing space before
> newline. This is not useful and shows up as trailing space e.g. when
> the output is checked into documentation. Remove the trailing space.
>
> Signed-off-by: Marek Vasut <marek.vasut+renesas at mailbox.org>
Reviewed-by: Mattijs Korpershoek <mkorpershoek at kernel.org>
> ---
> Cc: Jaehoon Chung <jh80.chung at samsung.com>
> Cc: Jerome Forissier <jerome.forissier at arm.com> <jerome at forissier.org>
> Cc: Mattijs Korpershoek <mkorpershoek at kernel.org>
> Cc: Peng Fan <peng.fan at nxp.com>
> Cc: Tom Rini <trini at konsulko.com>
> Cc: Yao Zi <me at ziyao.cc>
> Cc: u-boot at lists.denx.de
> ---
> cmd/mmc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/cmd/mmc.c b/cmd/mmc.c
> index 6cb1ef5dc23..81b1ca4ad84 100644
> --- a/cmd/mmc.c
> +++ b/cmd/mmc.c
> @@ -29,12 +29,12 @@ static void print_mmcinfo(struct mmc *mmc)
> printf("Manufacturer ID: %x\n", mmc->cid[0] >> 24);
> if (IS_SD(mmc)) {
> printf("OEM: %x\n", (mmc->cid[0] >> 8) & 0xffff);
> - printf("Name: %c%c%c%c%c \n", mmc->cid[0] & 0xff,
> + printf("Name: %c%c%c%c%c\n", mmc->cid[0] & 0xff,
> (mmc->cid[1] >> 24), (mmc->cid[1] >> 16) & 0xff,
> (mmc->cid[1] >> 8) & 0xff, mmc->cid[1] & 0xff);
> } else {
> printf("OEM: %x\n", (mmc->cid[0] >> 8) & 0xff);
> - printf("Name: %c%c%c%c%c%c \n", mmc->cid[0] & 0xff,
> + printf("Name: %c%c%c%c%c%c\n", mmc->cid[0] & 0xff,
> (mmc->cid[1] >> 24), (mmc->cid[1] >> 16) & 0xff,
> (mmc->cid[1] >> 8) & 0xff, mmc->cid[1] & 0xff,
> (mmc->cid[2] >> 24));
> --
> 2.51.0
More information about the U-Boot
mailing list