Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39598 )
Change subject: util/amdfwtool: Fix file open error msg ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/39598/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/39598/1//COMMIT_MSG@7 PS1, Line 7:
nit: spurious space?
Done
https://review.coreboot.org/c/coreboot/+/39598/1//COMMIT_MSG@17 PS1, Line 17: https://chromium-review.googlesource.com/c/chromiumos/third_party/coreboot/+...
Can shorten this: https://chromium-review.googlesource. […]
Done
https://review.coreboot.org/c/coreboot/+/39598/1//COMMIT_MSG@19 PS1, Line 19: Commit-Queue: Rob Barnes robbarnes@google.com
Is this tag needed?
Done. No idea what that tag is for though