Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46984 )
Change subject: mb/purism/librem_cnl: Add new variant 'Librem Mini v2' ......................................................................
Patch Set 2: Code-Review+1
(3 comments)
https://review.coreboot.org/c/coreboot/+/46984/2/Documentation/mainboard/pur... File Documentation/mainboard/purism/librem_mini_v2.md:
https://review.coreboot.org/c/coreboot/+/46984/2/Documentation/mainboard/pur... PS2, Line 67: nit: trailing space
https://review.coreboot.org/c/coreboot/+/46984/2/Documentation/mainboard/pur... PS2, Line 67: an *c*an
https://review.coreboot.org/c/coreboot/+/46984/2/src/mainboard/purism/librem... File src/mainboard/purism/librem_cnl/variants/librem_mini_v2/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/46984/2/src/mainboard/purism/librem... PS2, Line 22: SaGv_Enabled SaGv_FixedHigh for better bootup times