Matt DeVillier has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/44555 )
Change subject: docs/purism/librem_mini: Fixup CPU model, markup links ......................................................................
docs/purism/librem_mini: Fixup CPU model, markup links
Change-Id: I26c0936c912490fc0ba28ee53139e3a1f3a00911 Signed-off-by: Matt DeVillier matt.devillier@puri.sm --- M Documentation/mainboard/purism/librem_mini.md 1 file changed, 5 insertions(+), 3 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/55/44555/1
diff --git a/Documentation/mainboard/purism/librem_mini.md b/Documentation/mainboard/purism/librem_mini.md index 240dcff..be9a3e5 100644 --- a/Documentation/mainboard/purism/librem_mini.md +++ b/Documentation/mainboard/purism/librem_mini.md @@ -1,8 +1,10 @@ # Purism Librem Mini
+This page describes how to run coreboot on the [Purism Librem Mini]. + ```eval_rst +------------------+--------------------------------------------------+ -| CPU | Intel Core i7-8665U | +| CPU | Intel Core i7-8565U | +------------------+--------------------------------------------------+ | PCH | Whiskey Lake / Cannon Point LP | +------------------+--------------------------------------------------+ @@ -80,7 +82,7 @@
The m.2 SSD will need to be removed if the Wi-Fi antenna are connected to an internal Wi-Fi/BT module. Use a SOIC-8 chip clip to program the chip. -Specifically, it's a Winbond W25Q128JV (3.3V) -[datasheet][W25Q128JV]. +Specifically, it's a Winbond W25Q128JV (3.3V) - [datasheet][W25Q128JV].
The EC firmware is stored on a separate SOIC-8 chip (a Winbond W25Q80DV), but is not protected by a diode and therefore cannot be read/written to without @@ -115,6 +117,6 @@
[Purism Librem Mini]: https://puri.sm/products/librem-mini/ -[purism-blobs] : https://source.puri.sm/coreboot/purism-blobs +[purism-blobs]: https://source.puri.sm/coreboot/purism-blobs [W25Q128JV]: https://www.winbond.com/resource-files/w25q128jv%20revf%2003272018%20plus.pd... [flashrom]: https://flashrom.org/Flashrom
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44555 )
Change subject: docs/purism/librem_mini: Fixup CPU model, markup links ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
https://review.coreboot.org/c/coreboot/+/44555/1/Documentation/mainboard/pur... File Documentation/mainboard/purism/librem_mini.md:
https://review.coreboot.org/c/coreboot/+/44555/1/Documentation/mainboard/pur... PS1, Line 85: [datasheet][W25Q128JV] Does this mark up the link correctly?
Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44555 )
Change subject: docs/purism/librem_mini: Fixup CPU model, markup links ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44555/1/Documentation/mainboard/pur... File Documentation/mainboard/purism/librem_mini.md:
https://review.coreboot.org/c/coreboot/+/44555/1/Documentation/mainboard/pur... PS1, Line 85: [datasheet][W25Q128JV]
Does this mark up the link correctly?
yes it does (see existing Protectli docs)
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44555 )
Change subject: docs/purism/librem_mini: Fixup CPU model, markup links ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
https://review.coreboot.org/c/coreboot/+/44555/1/Documentation/mainboard/pur... File Documentation/mainboard/purism/librem_mini.md:
https://review.coreboot.org/c/coreboot/+/44555/1/Documentation/mainboard/pur... PS1, Line 85: [datasheet][W25Q128JV]
yes it does (see existing Protectli docs)
Ack
Matt DeVillier has submitted this change. ( https://review.coreboot.org/c/coreboot/+/44555 )
Change subject: docs/purism/librem_mini: Fixup CPU model, markup links ......................................................................
docs/purism/librem_mini: Fixup CPU model, markup links
Change-Id: I26c0936c912490fc0ba28ee53139e3a1f3a00911 Signed-off-by: Matt DeVillier matt.devillier@puri.sm Reviewed-on: https://review.coreboot.org/c/coreboot/+/44555 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com --- M Documentation/mainboard/purism/librem_mini.md 1 file changed, 5 insertions(+), 3 deletions(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved
diff --git a/Documentation/mainboard/purism/librem_mini.md b/Documentation/mainboard/purism/librem_mini.md index 240dcff..be9a3e5 100644 --- a/Documentation/mainboard/purism/librem_mini.md +++ b/Documentation/mainboard/purism/librem_mini.md @@ -1,8 +1,10 @@ # Purism Librem Mini
+This page describes how to run coreboot on the [Purism Librem Mini]. + ```eval_rst +------------------+--------------------------------------------------+ -| CPU | Intel Core i7-8665U | +| CPU | Intel Core i7-8565U | +------------------+--------------------------------------------------+ | PCH | Whiskey Lake / Cannon Point LP | +------------------+--------------------------------------------------+ @@ -80,7 +82,7 @@
The m.2 SSD will need to be removed if the Wi-Fi antenna are connected to an internal Wi-Fi/BT module. Use a SOIC-8 chip clip to program the chip. -Specifically, it's a Winbond W25Q128JV (3.3V) -[datasheet][W25Q128JV]. +Specifically, it's a Winbond W25Q128JV (3.3V) - [datasheet][W25Q128JV].
The EC firmware is stored on a separate SOIC-8 chip (a Winbond W25Q80DV), but is not protected by a diode and therefore cannot be read/written to without @@ -115,6 +117,6 @@
[Purism Librem Mini]: https://puri.sm/products/librem-mini/ -[purism-blobs] : https://source.puri.sm/coreboot/purism-blobs +[purism-blobs]: https://source.puri.sm/coreboot/purism-blobs [W25Q128JV]: https://www.winbond.com/resource-files/w25q128jv%20revf%2003272018%20plus.pd... [flashrom]: https://flashrom.org/Flashrom
-
Angel Pons (Code Review) -
Matt DeVillier (Code Review)