Johnny Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45798 )
Change subject: soc/intel/xeon_sp/cpx: Add save_dimm_info for SMBIOS type 17
......................................................................
Patch Set 11:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45798/4/src/soc/intel/xeon_sp/cpx/…
File src/soc/intel/xeon_sp/cpx/romstage.c:
https://review.coreboot.org/c/coreboot/+/45798/4/src/soc/intel/xeon_sp/cpx/…
PS4, Line 41: save_dimm_info
> The save_dimm_info() and the SystemMemoryMapHob struct are both Cooper Lake specific, they cannot be […]
Ack
--
To view, visit https://review.coreboot.org/c/coreboot/+/45798
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3cb72d18027d972140828970206834ff55b72022
Gerrit-Change-Number: 45798
Gerrit-PatchSet: 11
Gerrit-Owner: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Jingle Hsu <jingle_hsu(a)wiwynn.com>
Gerrit-Reviewer: John Looney <john.looney(a)gmail.com>
Gerrit-Reviewer: Jonathan Zhang <jonzhang(a)fb.com>
Gerrit-Reviewer: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Morgan Jang <Morgan_Jang(a)wiwynn.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <felixsinger(a)posteo.net>
Gerrit-CC: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Comment-Date: Wed, 07 Oct 2020 05:16:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Johnny Lin <Johnny_Lin(a)wiwynn.com>
Comment-In-Reply-To: Michael Niewöhner <foss(a)mniewoehner.de>
Gerrit-MessageType: comment
Kangheui Won has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45986 )
Change subject: Update vboot submodule to upstream master
......................................................................
Update vboot submodule to upstream master
Updating from commit id 4bb06cc1:
COIL: Change denylist to blocklist
to commit id 4c523ed1:
vboot2: Add support for modexp acceleration
This brings in 10 new commmits.
Change-Id: Iff6eb99c8ed3046b6fdb6c1e2892aab956f3b562
---
M 3rdparty/vboot
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/45986/1
diff --git a/3rdparty/vboot b/3rdparty/vboot
index 4bb06cc..4c523ed 160000
--- a/3rdparty/vboot
+++ b/3rdparty/vboot
@@ -1 +1 @@
-Subproject commit 4bb06cc1b73c738acf056d7fc0a96c4690e8ef8a
+Subproject commit 4c523ed10f25de872ac0513ebd6ca53d3970b9de
--
To view, visit https://review.coreboot.org/c/coreboot/+/45986
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iff6eb99c8ed3046b6fdb6c1e2892aab956f3b562
Gerrit-Change-Number: 45986
Gerrit-PatchSet: 1
Gerrit-Owner: Kangheui Won <khwon(a)chromium.org>
Gerrit-MessageType: newchange
Tim Chu has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/41604 )
Change subject: mb/ocp/tiogapass: Implement set POST start/end command
......................................................................
mb/ocp/tiogapass: Implement set POST start/end command
Implemented sending POST start and POST end command to BMC in ramstage.
Tested=Read POST command in OpenBMC,
if success message may show as below,
root@bmc:~# cat /var/log/messages |grep -i "POST"
Aug 10 00:11:24 bmc user.info ipmid: POST Start Event for Payload#1
Aug 10 00:11:28 bmc user.info ipmid: POST End Event for Payload#1
root@bmc:~#
Signed-off-by: TimChu <Tim.Chu(a)quantatw.com>
Change-Id: Ia283c5e8d539bfeb47a73d32d8bfb5e666f27170
---
M src/mainboard/ocp/tiogapass/Kconfig
M src/mainboard/ocp/tiogapass/devicetree.cb
2 files changed, 4 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/04/41604/1
diff --git a/src/mainboard/ocp/tiogapass/Kconfig b/src/mainboard/ocp/tiogapass/Kconfig
index 5cb2421..2ec47ed 100644
--- a/src/mainboard/ocp/tiogapass/Kconfig
+++ b/src/mainboard/ocp/tiogapass/Kconfig
@@ -9,6 +9,7 @@
select HAVE_ACPI_TABLES
select MAINBOARD_USES_FSP2_0
select IPMI_KCS
+ select IPMI_OCP
select SOC_INTEL_SKYLAKE_SP
select SUPERIO_ASPEED_AST2400
diff --git a/src/mainboard/ocp/tiogapass/devicetree.cb b/src/mainboard/ocp/tiogapass/devicetree.cb
index 19dba55..35e74b0 100644
--- a/src/mainboard/ocp/tiogapass/devicetree.cb
+++ b/src/mainboard/ocp/tiogapass/devicetree.cb
@@ -77,6 +77,9 @@
register "bmc_i2c_address" = "0x20"
register "bmc_boot_timeout" = "60"
end
+ chip drivers/ocp/ipmi # OCP specific IPMI porting
+ device pnp ca2.1 on end
+ end
end # Intel Corporation C621 Series Chipset LPC/eSPI Controller
device pci 1f.2 on end # Intel Corporation C620 Series Chipset Family Power Management Controller
device pci 1f.4 on end # Intel Corporation C620 Series Chipset Family SMBus
--
To view, visit https://review.coreboot.org/c/coreboot/+/41604
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia283c5e8d539bfeb47a73d32d8bfb5e666f27170
Gerrit-Change-Number: 41604
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Chu <Tim.Chu(a)quantatw.com>
Gerrit-MessageType: newchange
Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/28719 )
Change subject: sb/amd/pi/hudson: Add SPI controller support
......................................................................
Patch Set 5:
(3 comments)
https://review.coreboot.org/c/coreboot/+/28719/5/src/southbridge/amd/pi/hud…
File src/southbridge/amd/pi/hudson/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/28719/5/src/southbridge/amd/pi/hud…
PS5, Line 31: bootblock
Did you consider all-$()? For any system with elog support, then it can also be good to have spi in smm as well. Or, it looks like in soc/amd/common/block/spi, it looks like we've qualified it with CONFIG_SPI_FLASH_SMM.
https://review.coreboot.org/c/coreboot/+/28719/5/src/southbridge/amd/pi/hud…
File src/southbridge/amd/pi/hudson/pci_devs.h:
https://review.coreboot.org/c/coreboot/+/28719/5/src/southbridge/amd/pi/hud…
PS5, Line 18:
Was this file added to the commit by accident? It doesn't seem to fit with adding SPI.
https://review.coreboot.org/c/coreboot/+/28719/5/src/southbridge/amd/pi/hud…
File src/southbridge/amd/pi/hudson/spi.c:
https://review.coreboot.org/c/coreboot/+/28719/5/src/southbridge/amd/pi/hud…
PS5, Line 158: spi_write8(SPI_EXT_REG_INDX, SPI_EXT_REG_TXCOUNT);
: spi_write8(SPI_EXT_REG_DATA, bytesout);
Hmm, which device did you write this for? It looks like, for example, that Hudson 1 is completely different than Kabini. As a result, I think a lot of the definitions should be looked at more closely. Maybe break the steps out into their own functions?
--
To view, visit https://review.coreboot.org/c/coreboot/+/28719
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Icc6feb433a19337e09fc394cbf30288f53b195dd
Gerrit-Change-Number: 28719
Gerrit-PatchSet: 5
Gerrit-Owner: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Piotr Król <piotr.krol(a)3mdeb.com>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Marc Jones <marc(a)marcjonesconsulting.com>
Gerrit-Comment-Date: Wed, 07 Oct 2020 01:21:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Furquan Shaikh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/46029 )
Change subject: drivers/intel/wifi: Drop call to pci_dev_init
......................................................................
drivers/intel/wifi: Drop call to pci_dev_init
`pci_dev_init()` is used to load and run option ROM on VGA class
devices (PCI_CLASS_DISPLAY_VGA). WiFi device is not a VGA class device
and hence the call to `pci_dev_init()` is not required. This change
drops the call to `pci_dev_init()` from `wifi_pci_dev_init()` in Intel
WiFi driver.
BUG=b:169802515
BRANCH=zork
Change-Id: I6588ea0a5c848904088d05fd1cbdf677b2dc8ea9
Signed-off-by: Furquan Shaikh <furquan(a)google.com>
---
M src/drivers/intel/wifi/wifi.c
1 file changed, 0 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/46029/1
diff --git a/src/drivers/intel/wifi/wifi.c b/src/drivers/intel/wifi/wifi.c
index 0ad0e1c..e117ec0 100644
--- a/src/drivers/intel/wifi/wifi.c
+++ b/src/drivers/intel/wifi/wifi.c
@@ -63,8 +63,6 @@
static void wifi_pci_dev_init(struct device *dev)
{
- pci_dev_init(dev);
-
if (CONFIG(ELOG)) {
uint32_t val;
val = pci_read_config16(dev, PMCS_DR);
--
To view, visit https://review.coreboot.org/c/coreboot/+/46029
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6588ea0a5c848904088d05fd1cbdf677b2dc8ea9
Gerrit-Change-Number: 46029
Gerrit-PatchSet: 1
Gerrit-Owner: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-MessageType: newchange