build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42566 )
Change subject: util/amdfwtool: Add support for EFS SPI values for F17h and F15h ......................................................................
Patch Set 6:
(3 comments)
https://review.coreboot.org/c/coreboot/+/42566/6/util/amdfwtool/amdfwtool.c File util/amdfwtool/amdfwtool.c:
https://review.coreboot.org/c/coreboot/+/42566/6/util/amdfwtool/amdfwtool.c@... PS6, Line 219: printf(" Stoneyridge, Raven, Picasso, Renoir or Lucienne"); line over 96 characters
https://review.coreboot.org/c/coreboot/+/42566/6/util/amdfwtool/amdfwtool.c@... PS6, Line 241: printf(" 0x2 Micron parts optional, this option is only\n"); line over 96 characters
https://review.coreboot.org/c/coreboot/+/42566/6/util/amdfwtool/amdfwtool.c@... PS6, Line 1093: static const char *optstring = "x:i:g:AMS:p:b:s:r:k:c:n:d:t:u:w:m:T:z:J:B:K:L:Y:N:UW:I:a:Q:V:e:v:j:y:G:O:X:F:H:o:f:l:hZ:qR:P:C:"; line over 96 characters