Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/19790 )
Change subject: mb/lenovo/*/smihandler: Get rid of mainboard_io_trap_handler
......................................................................
Patch Set 1: Code-Review+2
Is that tp-smapi IO range even used somewhere (ACPI?)?
--
To view, visit https://review.coreboot.org/19790
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I33fd829a7e34aefa8f76ca6020cc8e802f7aab17
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-HasComments: No
Hello Shaunak Saha, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/19739
to look at the new patch set (#6).
Change subject: [WIP]mainboard/intel/glkrvp: Add GPE routing settings for GLKRVP.
......................................................................
[WIP]mainboard/intel/glkrvp: Add GPE routing settings for GLKRVP.
This patch sets the devicetree for gpe0_dw
configuration and also configures the GPIO lines for SCI.
The macros for LID switch is also set.
Change-Id: I56a99bbb70ac9edda8448fabf05e3ea90d18d93c
Signed-off-by: Shaunak Saha <shaunak.saha(a)intel.com>
Signed-off-by: Hannah Williams <hannah.williams(a)intel.com>
---
M src/mainboard/intel/glkrvp/dsdt.asl
M src/mainboard/intel/glkrvp/variants/baseboard/devicetree.cb
M src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/ec.h
M src/mainboard/intel/glkrvp/variants/baseboard/include/baseboard/gpio.h
4 files changed, 33 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/19739/6
--
To view, visit https://review.coreboot.org/19739
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I56a99bbb70ac9edda8448fabf05e3ea90d18d93c
Gerrit-PatchSet: 6
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Shaunak Saha <shaunak.saha(a)intel.com>
Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>