* [U-Boot] [PATCH] mmc: fix the output format @ 2016-07-19 10:14 ` Ziyuan Xu 2016-07-19 10:35 ` Jaehoon Chung 0 siblings, 1 reply; 3+ messages in thread From: Ziyuan Xu @ 2016-07-19 10:14 UTC (permalink / raw) To: u-boot It's nicer to see this: => mmc list dwmmc at ff0c0000: 0 dwmmc at ff0f0000: 1 (eMMC) than this: => mmc list dwmmc at ff0c0000: 0dwmmc at ff0f0000: 1 (eMMC) With the latter, it's much clearer which mmc devices are on. Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com> --- drivers/mmc/mmc-uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mmc/mmc-uclass.c b/drivers/mmc/mmc-uclass.c index 38ced41..beebc70 100644 --- a/drivers/mmc/mmc-uclass.c +++ b/drivers/mmc/mmc-uclass.c @@ -169,7 +169,7 @@ void print_mmc_devices(char separator) for (uclass_first_device(UCLASS_MMC, &dev); dev; - uclass_next_device(&dev)) { + uclass_next_device(&dev), first = false) { struct mmc *m = mmc_get_mmc_dev(dev); if (!first) { -- 1.9.1 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] mmc: fix the output format 2016-07-19 10:14 ` [U-Boot] [PATCH] mmc: fix the output format Ziyuan Xu @ 2016-07-19 10:35 ` Jaehoon Chung 2016-07-19 13:56 ` Ziyuan Xu 0 siblings, 1 reply; 3+ messages in thread From: Jaehoon Chung @ 2016-07-19 10:35 UTC (permalink / raw) To: u-boot Hi Ziyuan, On 07/19/2016 07:14 PM, Ziyuan Xu wrote: > It's nicer to see this: > > => mmc list > dwmmc at ff0c0000: 0 > dwmmc at ff0f0000: 1 (eMMC) > > than this: > > => mmc list > dwmmc at ff0c0000: 0dwmmc at ff0f0000: 1 (eMMC) > > With the latter, it's much clearer which mmc devices are on. You're right. After applied your patch, it may display like mmc-legacy type. If add the comment in more detail, How about "Display mmc list information like mmc-legacy type." ? :) Best Regards, Jaehoon Chung > > Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com> > --- > > drivers/mmc/mmc-uclass.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mmc/mmc-uclass.c b/drivers/mmc/mmc-uclass.c > index 38ced41..beebc70 100644 > --- a/drivers/mmc/mmc-uclass.c > +++ b/drivers/mmc/mmc-uclass.c > @@ -169,7 +169,7 @@ void print_mmc_devices(char separator) > > for (uclass_first_device(UCLASS_MMC, &dev); > dev; > - uclass_next_device(&dev)) { > + uclass_next_device(&dev), first = false) { > struct mmc *m = mmc_get_mmc_dev(dev); > > if (!first) { > ^ permalink raw reply [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] mmc: fix the output format 2016-07-19 10:35 ` Jaehoon Chung @ 2016-07-19 13:56 ` Ziyuan Xu 0 siblings, 0 replies; 3+ messages in thread From: Ziyuan Xu @ 2016-07-19 13:56 UTC (permalink / raw) To: u-boot Hi Jaehoon, On 2016?07?19? 18:35, Jaehoon Chung wrote: > Hi Ziyuan, > > On 07/19/2016 07:14 PM, Ziyuan Xu wrote: >> It's nicer to see this: >> >> => mmc list >> dwmmc at ff0c0000: 0 >> dwmmc at ff0f0000: 1 (eMMC) >> >> than this: >> >> => mmc list >> dwmmc at ff0c0000: 0dwmmc at ff0f0000: 1 (eMMC) >> >> With the latter, it's much clearer which mmc devices are on. > You're right. After applied your patch, it may display like mmc-legacy type. > If add the comment in more detail, > How about "Display mmc list information like mmc-legacy type." ? :) Looks good for me.I will change commit to mmc: display mmc list informaton like mmc_legacy type. Thanks! > > Best Regards, > Jaehoon Chung > >> Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com> >> --- >> >> drivers/mmc/mmc-uclass.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/mmc/mmc-uclass.c b/drivers/mmc/mmc-uclass.c >> index 38ced41..beebc70 100644 >> --- a/drivers/mmc/mmc-uclass.c >> +++ b/drivers/mmc/mmc-uclass.c >> @@ -169,7 +169,7 @@ void print_mmc_devices(char separator) >> >> for (uclass_first_device(UCLASS_MMC, &dev); >> dev; >> - uclass_next_device(&dev)) { >> + uclass_next_device(&dev), first = false) { >> struct mmc *m = mmc_get_mmc_dev(dev); >> >> if (!first) { >> > > > ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-07-19 13:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CGME20160719101454epcas1p34f5d8e05bd3fe8b8d229795b6626332b@epcas1p3.samsung.com>
2016-07-19 10:14 ` [U-Boot] [PATCH] mmc: fix the output format Ziyuan Xu
2016-07-19 10:35 ` Jaehoon Chung
2016-07-19 13:56 ` Ziyuan Xu
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.