Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/23564 )
Change subject: mb/lenovo/w520: Add ThinkPad W520 support
......................................................................
Patch Set 1:
Please use one of the existing schemes: Copy&paste T520 to W520 folder or the preferred: create a variants subfolder in T520 folder.
Are you sure that the GPIO config is the same as on T520 ?
What doesn't work?
What payloads did you test ?
What OS did you test ?
--
To view, visit https://review.coreboot.org/23564
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: I1deb0436a807950c605dcd590deedcb3169bf8c5
Gerrit-Change-Number: 23564
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Rikken <nico(a)nicorikken.eu>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Fri, 02 Feb 2018 19:13:43 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Hello Jenny Tc,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/23572
to look at the new patch set (#2).
Change subject: google/lars: Turn on keyboard backlight in romstage
......................................................................
google/lars: Turn on keyboard backlight in romstage
Cherry-pick from Chromium:
a60ac10 [Lars: Turn on keyboard backlight in romstage]
Use the keyboard backlight to provide indication that the system is
booting. This is useful for determining that a system is in S0 and
is running BIOS code.
TEST=boot on Lars and see keyboard backlight come on early
Original-Change-Id: I4fede6cff85f4487cedfbccf6cc24c6380d905e0
Original-Signed-off-by: Jenny TC <jenny.tc(a)intel.com>
Original-Tested-by: Divagar Mohandass <divagar.mohandass(a)intel.com>
Original-Reviewed-by: Duncan Laurie <dlaurie(a)chromium.org>
Change-Id: I4b1fed10d9bd1ae1b265e848417836f816f252f3
Signed-off-by: Matt DeVillier <matt.devillier(a)gmail.com>
---
M src/mainboard/google/lars/romstage.c
1 file changed, 5 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/23572/2
--
To view, visit https://review.coreboot.org/23572
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4b1fed10d9bd1ae1b265e848417836f816f252f3
Gerrit-Change-Number: 23572
Gerrit-PatchSet: 2
Gerrit-Owner: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Jenny Tc <jenny.tc(a)intel.com>