Hi Stefan,

Stefan Roese <s...@denx.de> wrote:

> > -   printf("mtdparts: %s\n",
> > -           mtdparts_default ? mtdparts_default : "none");
> > +   puts("mtdparts: ");
> > +   puts(mtdparts_default ? mtdparts_default : "none");
> > +   puts("\n");
> 
> It might make sense to add a comment here, describing why you use these 
> puts() 
> calls instead of one printf(). Otherwise somebody else might trip into this 
> problem again in a later time.

This makes sense. I'll add a comment in next patch version.

Best regards,
Anatolij
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to