Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48811 )
Change subject: fmaptool: Add option to dump fmap ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/48811/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/48811/2//COMMIT_MSG@10 PS2, Line 10: coreboot does not include a utility : that lets you read the fmap from an image It does, actually? coreboot can build futility and `futility dump_fmap -h` does exactly that. Can we just stick to that rather than implementing a second one? (fmaptool in particular is intended to be a build-only tool that's not normally going to be installed on target hosts, so I think it's not that suitable for this.)