cedarhouse1@comcast.net has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38765 )
Change subject: [UNTESTED] intel/stm: Place resource list right below MSEG
......................................................................
Patch Set 1: Code-Review+1
I've tested this patch with no problems. The console log showed the expected placing of the BIOS resource list, the STM had no problems processing the list and when the SMI handler ran the change in the stack location had no apparent issues.
This was tested on a Purism librem 15v4 (kabylake).
--
To view, visit
https://review.coreboot.org/c/coreboot/+/38765
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9c5970433a3851b5c9662c8870efaf2943498d27
Gerrit-Change-Number: 38765
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Arthur Heymans
arthur@aheymans.xyz
Gerrit-Reviewer: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-Reviewer: Nico Huber
nico.h@gmx.de
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Reviewer: cedarhouse1@comcast.net
Gerrit-Comment-Date: Fri, 07 Feb 2020 21:34:26 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment