Bao Zheng has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45286 )
Change subject: amdfwtool: Clean up the Makefile of amdfwtool ......................................................................
Patch Set 9:
(1 comment)
Patch Set 9:
(5 comments)
compiling the standalone version fails with this:
felix@felix-VirtualBox:~/coreboot/util/amdfwtool$ make cc -O2 -Wall -Wextra -Wshadow -Wno-sign-compare -c -o amdfwtool.o amdfwtool.c amdfwtool.c: In function ‘main’: amdfwtool.c:1756:3: warning: ignoring return value of ‘write’, declared with attribute warn_unused_result [-Wunused-result] write(targetfd, amd_romsig, ctx.current - romsig_offset); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc amdfwtool.o -o amdfwtool
Done. Fixed in set 10
https://review.coreboot.org/c/coreboot/+/45286/7/util/amdfwtool/Makefile File util/amdfwtool/Makefile:
https://review.coreboot.org/c/coreboot/+/45286/7/util/amdfwtool/Makefile@10 PS7, Line 10: #data_parse.c
remove; that one gets added in the next patch, so no need to have a commented out version here
Done. Removed.