Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33193 )
Change subject: soc/intel/{cml, whl}: Add option to skip HECI disable in SMM
......................................................................
Patch Set 6:
(1 comment)
https://review.coreboot.org/#/c/33193/6/src/soc/intel/common/block/smm/Kconf...
File src/soc/intel/common/block/smm/Kconfig:
https://review.coreboot.org/#/c/33193/6/src/soc/intel/common/block/smm/Kconf...
PS6, Line 32: Only select this option if FSP
: doesn't provides a dedicated UPD to perform HECI disable.
yeah, that what we wish to do even fsp has this support :)
I meant this option can be selected in more cases than what is indicated here in the help text. "Only select ... if FSP doesn't provide ..." seems to be too restrictive.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/33193
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If3b064f3c32877235916f966a01beb525156d188
Gerrit-Change-Number: 33193
Gerrit-PatchSet: 6
Gerrit-Owner: Subrata Banik
subrata.banik@intel.com
Gerrit-Reviewer: Aamir Bohra
aamir.bohra@intel.com
Gerrit-Reviewer: Aaron Durbin
adurbin@chromium.org
Gerrit-Reviewer: Bora Guvendik
bora.guvendik@intel.com
Gerrit-Reviewer: Duncan Laurie
dlaurie@chromium.org
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Rizwan Qureshi
rizwan.qureshi@intel.com
Gerrit-Reviewer: Shelley Chen
shchen@google.com
Gerrit-Reviewer: Subrata Banik
subrata.banik@intel.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Nico Huber
nico.h@gmx.de
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Wed, 12 Jun 2019 06:30:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik
subrata.banik@intel.com
Comment-In-Reply-To: Furquan Shaikh
furquan@google.com
Gerrit-MessageType: comment