Attention is currently required from: Angel Pons. Zoltan Baldaszti has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45829 )
Change subject: Documentation: Add more payload links and descriptions ......................................................................
Patch Set 2:
(4 comments)
File Documentation/payloads.md:
https://review.coreboot.org/c/coreboot/+/45829/comment/510a6e0c_7fdf2274 PS2, Line 41: which can be : installed in a boot ROM and used to initialize
That doesn't refer to its use as a payload though, and neither does the URL above
I'm no position to describe U-Boot, text taken from the repo's README verbatim. https://gitlab.denx.de/u-boot/u-boot/-/blob/master/README About the link, that's its official url. The doc url in payloads/external/U-Boot/Kconfig.name is a dead-link. If you have a better one, let me know.
https://review.coreboot.org/c/coreboot/+/45829/comment/fc2b9413_b36d998f PS2, Line 56: version
implementation?
Yeah, why not?
https://review.coreboot.org/c/coreboot/+/45829/comment/b44f9ff5_d22d4de2 PS2, Line 61: [Linux](https://kernel.org)
I think the more important aspect would be links to the projects using Linux, not a link to the kern […]
This is a link to the Linux kernel, because the context refers to the Linux kernel. Distros should be listed separately IMHO (just like Heads). But if there's a list of Linux distros compiled for coreboot, let me know the url.
https://review.coreboot.org/c/coreboot/+/45829/comment/aaad92a9_75da4d54 PS2, Line 82: http
https?
I haven't checked this (the diff is because of the newline), but now that you mentioned it I did. It has an invalid cert, SSL_ERROR_BAD_CERT_DOMAIN. I guess that's the reason why it was http in the first place. Do you still want https knowing it's not working? I can update if you want.