Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42882 )
Change subject: documentation: Add documentation for Purism Librem Mini ......................................................................
Patch Set 10: Code-Review+1
(5 comments)
https://review.coreboot.org/c/coreboot/+/42882/10/Documentation/mainboard/pu... File Documentation/mainboard/purism/librem_mini.md:
https://review.coreboot.org/c/coreboot/+/42882/10/Documentation/mainboard/pu... PS10, Line 7: Whiskey Lake / Cannon Lake LP Cannon Point LP
https://review.coreboot.org/c/coreboot/+/42882/10/Documentation/mainboard/pu... PS10, Line 19: Three empty lines?
https://review.coreboot.org/c/coreboot/+/42882/10/Documentation/mainboard/pu... PS10, Line 75: trailing whitespace
https://review.coreboot.org/c/coreboot/+/42882/10/Documentation/mainboard/pu... File Documentation/mainboard/purism/librem_mini.png:
PS10: https://tinypng.com/ can halve the size of this image
https://review.coreboot.org/c/coreboot/+/42882/10/Documentation/mainboard/pu... File Documentation/mainboard/purism/librem_mini_flash.jpg:
PS10: http://jpeg-optimizer.com/ can more or less halve this image's size without resizing it