[U-Boot] should print_mmc_devices() not add an extraneous blank between devices?

Lubomir Popov lpopov at mm-sol.com
Wed Aug 21 08:34:26 CEST 2013


Hi Robert,

On 20/08/13 23:46, Robert P. J. Day wrote:
> 
>   playing on my beaglebone black and:
> 
> U-Boot# mmc list
> OMAP SD/MMC: 0
>  OMAP SD/MMC: 1     <-- ???
> U-Boot#
> 
>   puzzled as to why the second line is indented, and discovered, first
> from cmd_mmc.c:
> 
>         print_mmc_devices('\n');
> 
> which reasonably defines a newline as a separator, but then there's
> this in drivers/mmc/mmc.c:
> 
>         list_for_each(entry, &mmc_devices) {
>                 m = list_entry(entry, struct mmc, link);
> 
>                 printf("%s: %d", m->name, m->block_dev.dev);
> 
>                 if (entry->next != &mmc_devices)
>                         printf("%c ", separator);
>         }
> 
> where that loop inserts the separator *and* a blank. is that
> deliberate? it just looks weird.
> 
I have used to fix this as follows on some older U-Boot versions:

		if (entry->next != &mmc_devices) {
			printf("%c", separator);
			if (separator != '\n')
				puts (" ");
		}

But as this is just cosmetics, never bothered to submit a patch, nor
am fixing it anymore locally. If you are a style perfectionist, do
submit one... ;)

Best regards,
Lubomir



More information about the U-Boot mailing list