| Message ID | 69bfe9497ed2561e4c54a02c8d8b1142.squirrel@www.mm-sol.com |
|---|---|
| State | Accepted |
| Delegated to: | Pantelis Antoniou |
| Headers | show |
Hi Lubomic, > On Nov 11, 2014, at 12:25 , Lubomir Popov <lpopov@mm-sol.com> wrote: > > If print_mmc_devices() was called with a '\n' separator (as done > for example by the "mmc list" command), it offset the 2-nd and > all subsequent lines by one space. Fixing this. > > Signed-off-by: Lubomir Popov <l-popov@ti.com> > --- > drivers/mmc/mmc.c | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c > index 44a4feb..e0ea833 100644 > --- a/drivers/mmc/mmc.c > +++ b/drivers/mmc/mmc.c > @@ -1401,8 +1401,11 @@ void print_mmc_devices(char separator) > > printf("%s: %d", m->cfg->name, m->block_dev.dev); > > - if (entry->next != &mmc_devices) > - printf("%c ", separator); > + if (entry->next != &mmc_devices) { > + printf("%c", separator); > + if (separator != '\n') > + puts (" "); > + } > } > > printf("\n"); > -- > 1.7.9.5 > Applied, thanks — Pantelis
diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index 44a4feb..e0ea833 100644 --- a/drivers/mmc/mmc.c +++ b/drivers/mmc/mmc.c @@ -1401,8 +1401,11 @@ void print_mmc_devices(char separator) printf("%s: %d", m->cfg->name, m->block_dev.dev); - if (entry->next != &mmc_devices) - printf("%c ", separator); + if (entry->next != &mmc_devices) { + printf("%c", separator); + if (separator != '\n') + puts (" "); + } } printf("\n");
If print_mmc_devices() was called with a '\n' separator (as done for example by the "mmc list" command), it offset the 2-nd and all subsequent lines by one space. Fixing this. Signed-off-by: Lubomir Popov <l-popov@ti.com> --- drivers/mmc/mmc.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)