
23 Feb
2010
23 Feb
'10
11:51 p.m.
Hi Stefan,
Stefan Roese sr@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