[coreboot-gerrit] Change in em100[master]: Add more traceable SPI commands

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/em100/+/48584 )

Change subject: Add more traceable SPI commands
......................................................................

Patch Set 4:

(2 comments)

https://review.coreboot.org/c/em100/+/48584/4/trace.c 
File trace.c:

https://review.coreboot.org/c/em100/+/48584/4/trace.c@264 
PS4, Line 264: always decode 4 bytes
I am not sure I understand this comment.

https://review.coreboot.org/c/em100/+/48584/4/trace.c@321 
PS4, Line 321: address += 16;
Isn't this only for address_type != ADDR_NONE? Probably doesn't matter since it will not be used in the for loop above?