Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/27124 )
Change subject: mainboard/google/nocturne: Enable Image Unit
......................................................................
Patch Set 1:
This change needs to be rebased on top of its parent CL.
--
To view, visit https://review.coreboot.org/27124
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic2edf5ec7bde5c55ce1b13cf7b680094a9fffc6a
Gerrit-Change-Number: 27124
Gerrit-PatchSet: 1
Gerrit-Owner: Lijian Zhao <lijian.zhao(a)intel.com>
Gerrit-Reviewer: Chiranjeevi Rapolu <chiranjeevi.rapolu(a)intel.com>
Gerrit-Reviewer: Rajmohan Mani <rajmohan.mani(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Furquan Shaikh <furquan(a)google.com>
Gerrit-Comment-Date: Tue, 19 Jun 2018 07:03:46 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Nico Rikken has posted comments on this change. ( https://review.coreboot.org/27158 )
Change subject: mb/lenovo/t400: Change CMOS value name to Dual Graphics
......................................................................
Patch Set 1:
The found user manual has been addressed by a merge request:
https://notabug.org/libreboot/libreboot/pulls/473
--
To view, visit https://review.coreboot.org/27158
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I2e4d8bafbae5de97c78dab118f75fdefff1d7c37
Gerrit-Change-Number: 27158
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Rikken <nico(a)nicorikken.eu>
Gerrit-Reviewer: Nico Rikken <nico(a)nicorikken.eu>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 19 Jun 2018 06:17:23 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Nico Rikken has uploaded this change for review. ( https://review.coreboot.org/27158
Change subject: mb/lenovo/t400: Change CMOS value name to Dual Graphics
......................................................................
mb/lenovo/t400: Change CMOS value name to Dual Graphics
To uniform the naming of the 'Dual Graphics' mode amongst the ThinkPads that
support it, the T400 CMOS value needs a change. This was the outcome of a
discussion on another patch: https://review.coreboot.org/#/c/coreboot/+/23040/
This might cause breakage for automated NVRAM configuration scripts, and
manuals. I only found one manual using the previous 'Switchable' option:
https://libreboot.org/docs/install/r400_external.html#a-note-about-gpus
Change-Id: I2e4d8bafbae5de97c78dab118f75fdefff1d7c37
Signed-off-by: Nico Rikken <nico(a)nicorikken.eu>
---
M src/mainboard/lenovo/t400/cmos.layout
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/58/27158/1
diff --git a/src/mainboard/lenovo/t400/cmos.layout b/src/mainboard/lenovo/t400/cmos.layout
index 6592f40..62072dc 100644
--- a/src/mainboard/lenovo/t400/cmos.layout
+++ b/src/mainboard/lenovo/t400/cmos.layout
@@ -128,7 +128,7 @@
11 12 352M
12 0 Integrated Only
12 1 Discrete Only
-12 2 Switchable
+12 2 Dual Graphics
# -----------------------------------------------------------------
checksums
--
To view, visit https://review.coreboot.org/27158
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I2e4d8bafbae5de97c78dab118f75fdefff1d7c37
Gerrit-Change-Number: 27158
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Rikken <nico(a)nicorikken.eu>