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

[Patrick Rudolph (Code Review)]

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

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

Patch Set 3:

(2 comments)

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.

True, but looks looks like a bug to me.
Is there any historic reason not to pass the argument?

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

https://review.coreboot.org/#/c/33447/2/Documentation/lib/payloads/selfboot.... 
PS2, Line 3: static
> statically

Done