Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31252 )
Change subject: Documentation: update/improve distribution listing ......................................................................
Patch Set 1:
(9 comments)
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md File Documentation/distributions.md:
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@15 PS1, Line 15: blob-free OS and coreboot firmware Huh? Don't they use the FSP?
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@21 PS1, Line 21: on onwards, maybe?
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@22 PS1, Line 22: running *on* the
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@28 PS1, Line 28: [Libretrend](https://libretrend.com) sells the Librebox, a NUC-like PC This line is noticeably shorter than the rest
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@29 PS1, Line 29: chips ships
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@37 PS1, Line 37: / This slash looks terrible in here
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@43 PS1, Line 43: Mr Chromebox Maybe add the period after "Mr"
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@48 PS1, Line 48: ,, double comma? should be dot and comma (.,) to make "i.e.,"
https://review.coreboot.org/#/c/31252/1/Documentation/distributions.md@59 PS1, Line 59: a good deal maybe use "significantly" instead