[coreboot-gerrit] Change in ...coreboot[master]: Documentation: Explain SimpleELF

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33447 )

Change subject: Documentation: Explain SimpleELF
......................................................................

Patch Set 1:

(1 comment)

When I think of SELF documentation, I think of the stuff in https://review.coreboot.org/cgit/coreboot.git/tree/Documentation/cbfs.txt#n3..., which still has a lot more details about the actual format than you have here. Shouldn't this be integrated there instead?

https://review.coreboot.org/#/c/33447/1/Documentation/lib/payloads/selfboot.... 
File Documentation/lib/payloads/selfboot.md:

https://review.coreboot.org/#/c/33447/1/Documentation/lib/payloads/selfboot.... 
PS1, Line 23: argument.
I don't think this is true on x86, the coreboot table address is still hardcoded there.