Dear Tom, In message <20180718125351.GE4609@bill-the-cat> you wrote: > > > Loading Environment from FAT... > > --> MMC: no card present > > --> ** Bad device mmc 0 ** > > --> Failed (-5) > > Loading Environment from MMC... > > --> OK > > > > instead of: > > > > Loading Environment from FAT... MMC: no card present > > ** Bad device mmc 0 ** > > Failed (-5) > > Loading Environment from MMC... OK > > So, I think maybe (and given Wolfgang's comments) we should think about > how the output might want to look, and how to get there without GD > changes. Perhaps: > Attempting to load Environment from FAT (do we have more easily > available info at this point?): > MMC: no card present > ** Bad device mmc 0 ** > Failed (-5) > Loading Environment from MMC... > Attempting to load Environment from MMC: > Succeeded
Just my 0.02€: In the non-error case, the output should be a single (ideally short) line. Rationale: to many lines of ourput clutter your screen and make you miss context faster; to many/long lines take time to print so they make booting slower. In the error case, the user should be able to understand what the problem was and decide if it was critical or can be ignored (like here when intentionally booting without SDCard). Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de "Wish not to seem, but to be, the best." - Aeschylus _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
