Nico Huber has posted comments on this change. ( https://review.coreboot.org/19614 )
Change subject: cli_common: Dont request reports from known-bad chip operations ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/19614/1/cli_common.c File cli_common.c:
https://review.coreboot.org/#/c/19614/1/cli_common.c@111 PS1, Line 111: if (untested) { : msg_cinfo("The test status of this chip may have been updated in the latest development\n" : "version of flashrom. If you are running the latest development version,\n" : "please email a report to flashrom@flashrom.org if any of the above operations\n" : "work correctly for you with this flash chip. Please include the flashrom log\n" : "file for all operations you tested (see the man page for details), and mention\n" : "which mainboard or programmer you tested in the subject line.\n" : "Thanks for your help!\n"); : } else { : msg_cinfo("The test status of this chip may have been updated in the latest development\n" : "version of flashrom.\n"); : } I was once told, that we have a 112 columns limit [1]. Can't find anything about that in the wiki though.
[1] https://mail.coreboot.org/pipermail/flashrom/2012-June/009387.html