Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/em100/+/48548 )
Change subject: Simple tool to parse flash reads using em100 trace ......................................................................
Patch Set 3:
Patch Set 3:
Since we are integrating the support in em100 directly, do you still see value in having a separate tool? Is this mostly for offline analysis in that case?
I've been leaving this here for reference to get some input whether folks prefer this or the builtin version. I can also drop the builtin version. The nice thing about this version is it supports grouping the writes.