Attention is currently required from: Zheng Bao.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/73910 )
Change subject: amdfwtool: Move some funtions to other source files ......................................................................
Patch Set 1:
(1 comment)
File util/amdfwtool/signed_psp.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-172154): https://review.coreboot.org/c/coreboot/+/73910/comment/af558858_fbdb3c98 PS1, Line 217: S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH); Symbolic permissions 'S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH' are not preferred. Consider using octal permissions '0666'.