build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33400 )
Change subject: util/amdfwtool: Update for Family 17h ......................................................................
Patch Set 1:
(11 comments)
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c File util/amdfwtool/amdfwtool.c:
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@197 PS1, Line 197: printf("-U | --token-unlock Reserve space for debug token\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@202 PS1, Line 202: printf("-I | --instance <number> Sets instance field for the next BIOS firmware\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@203 PS1, Line 203: printf("-a | --apcb <FILE> Add AGESA PSP customization block\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@204 PS1, Line 204: printf("-Q | --apob-base <HEX_VAL> Destination for AGESA PSP output block\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@207 PS1, Line 207: printf("-y | --pmu-inst <FILE> Add PMU firmware instruction portion\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@208 PS1, Line 208: printf("-G | --pmu-data <FILE> Add PMU firmware data portion\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@211 PS1, Line 211: printf("-V | --bios-bin <FILE> Add compressed image; auto source address\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@212 PS1, Line 212: printf("-e | --bios-bin-src <HEX_VAL> Address in flash of source if -V not used\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@213 PS1, Line 213: printf("-v | --bios-bin-dest <HEX_VAL> Destination for uncompressed BIOS\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@214 PS1, Line 214: printf("-j | --bios-uncomp-size <HEX> Uncompressed size of BIOS image\n"); line over 80 characters
https://review.coreboot.org/#/c/33400/1/util/amdfwtool/amdfwtool.c@945 PS1, Line 945: 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:h"; line over 80 characters