build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42044 )
Change subject: util/amdfwtool: Add option for setting PSP/FW shared memory location ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/42044/1/util/amdfwtool/amdfwtool.c File util/amdfwtool/amdfwtool.c:
https://review.coreboot.org/c/coreboot/+/42044/1/util/amdfwtool/amdfwtool.c@... PS1, Line 216: printf("-P | --sharedmem-size Maximum size of the PSP/FW shared memory area\n"); line over 96 characters
https://review.coreboot.org/c/coreboot/+/42044/1/util/amdfwtool/amdfwtool.c@... PS1, Line 1041: 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:"; line over 96 characters