Attention is currently required from: Bill XIE.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/54335
to look at the new patch set (#2).
Change subject: mb/asus/p8x7x: Turn P8Z77-M PRO into a variant of P8x7x baseboard
......................................................................
mb/asus/p8x7x: Turn P8Z77-M PRO into a variant of P8x7x baseboard
Other variants would be added later.
MAINBOARD_HAS_TPM1 should not be selected, since the module is
replaceable.
Signed-off-by: Bill XIE <persmule(a)hardenedlinux.org>
Change-Id: Iaf0bd4c993debe4d5d7ca695473438bd4c954290
---
A src/mainboard/asus/p8x7x/Kconfig
R src/mainboard/asus/p8x7x/Kconfig.name
A src/mainboard/asus/p8x7x/Makefile.inc
R src/mainboard/asus/p8x7x/acpi/ec.asl
R src/mainboard/asus/p8x7x/acpi/platform.asl
R src/mainboard/asus/p8x7x/acpi/superio.asl
A src/mainboard/asus/p8x7x/board_info.txt
R src/mainboard/asus/p8x7x/cmos.default
R src/mainboard/asus/p8x7x/cmos.layout
A src/mainboard/asus/p8x7x/devicetree.cb
R src/mainboard/asus/p8x7x/dsdt.asl
A src/mainboard/asus/p8x7x/hda_verb.c
R src/mainboard/asus/p8x7x/variants/z77-m_pro/acpi_tables.c
A src/mainboard/asus/p8x7x/variants/z77-m_pro/board_info.txt
R src/mainboard/asus/p8x7x/variants/z77-m_pro/data.vbt
R src/mainboard/asus/p8x7x/variants/z77-m_pro/early_init.c
R src/mainboard/asus/p8x7x/variants/z77-m_pro/gma-mainboard.ads
R src/mainboard/asus/p8x7x/variants/z77-m_pro/gpio.c
R src/mainboard/asus/p8x7x/variants/z77-m_pro/hda_verb.c
R src/mainboard/asus/p8x7x/variants/z77-m_pro/mainboard.c
A src/mainboard/asus/p8x7x/variants/z77-m_pro/overridetree.cb
D src/mainboard/asus/p8z77-m_pro/Kconfig
D src/mainboard/asus/p8z77-m_pro/Makefile.inc
D src/mainboard/asus/p8z77-m_pro/board_info.txt
D src/mainboard/asus/p8z77-m_pro/devicetree.cb
25 files changed, 184 insertions(+), 139 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/35/54335/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/54335
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iaf0bd4c993debe4d5d7ca695473438bd4c954290
Gerrit-Change-Number: 54335
Gerrit-PatchSet: 2
Gerrit-Owner: Bill XIE <persmule(a)hardenedlinux.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Bill XIE <persmule(a)hardenedlinux.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Bill XIE.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/54338
to look at the new patch set (#2).
Change subject: mb/asus/p8x7x: Add P8Z77-V as a variant of P8x7x
......................................................................
mb/asus/p8x7x: Add P8Z77-V as a variant of P8x7x
Tested:
- PS/2 keyboard with SeaBIOS
- Integrated NIC
- S3 Suspend to RAM
- USB2 on rear
- USB3 (Z77's and Asmedia's works)
- Integrated SATA
- CPU Temp sensors (tested PSensor on GNU/Linux)
- TPM on TPM-header (tested tpm-tools with TPM 1.2 Infineon SLB9635TT12)
- Native raminit
- Integrated graphics with both libgfxinit (VGA/DVI-D/HDMI tested and working)
- PCIe GPU in PCIe-16x/8x slots (tested using an S3 Matrix)
- Debug output from serial port
- Atheros AR9485 adapted with Wi-Fi Go! Adapter
- Default PCIe config
Signed-off-by: Bill XIE <persmule(a)hardenedlinux.org>
Change-Id: Ic56ac0e5f93a6e818ef0666e41996718471b1cf6
---
A Documentation/mainboard/asus/p8z77-v.jpg
A Documentation/mainboard/asus/p8z77-v.md
M Documentation/mainboard/index.md
M src/mainboard/asus/p8x7x/Kconfig
M src/mainboard/asus/p8x7x/Kconfig.name
A src/mainboard/asus/p8x7x/variants/z77-v/acpi_tables.c
A src/mainboard/asus/p8x7x/variants/z77-v/board_info.txt
A src/mainboard/asus/p8x7x/variants/z77-v/data.vbt
A src/mainboard/asus/p8x7x/variants/z77-v/early_init.c
A src/mainboard/asus/p8x7x/variants/z77-v/gma-mainboard.ads
A src/mainboard/asus/p8x7x/variants/z77-v/gpio.c
A src/mainboard/asus/p8x7x/variants/z77-v/hda_verb.c
A src/mainboard/asus/p8x7x/variants/z77-v/mainboard.c
A src/mainboard/asus/p8x7x/variants/z77-v/overridetree.cb
14 files changed, 514 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/54338/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/54338
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic56ac0e5f93a6e818ef0666e41996718471b1cf6
Gerrit-Change-Number: 54338
Gerrit-PatchSet: 2
Gerrit-Owner: Bill XIE <persmule(a)hardenedlinux.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Bill XIE <persmule(a)hardenedlinux.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Bill XIE.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/54336
to look at the new patch set (#2).
Change subject: mb/asus/p8x7x: Use native ram init and depricate mrc.bin code
......................................................................
mb/asus/p8x7x: Use native ram init and depricate mrc.bin code
Signed-off-by: Bill XIE <persmule(a)hardenedlinux.org>
Change-Id: I79dc64c25cb36c630bf9994bf04a0dfe8654eb3c
---
M src/mainboard/asus/p8x7x/Kconfig
M src/mainboard/asus/p8x7x/cmos.default
M src/mainboard/asus/p8x7x/cmos.layout
M src/mainboard/asus/p8x7x/variants/z77-m_pro/early_init.c
4 files changed, 45 insertions(+), 214 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/36/54336/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/54336
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I79dc64c25cb36c630bf9994bf04a0dfe8654eb3c
Gerrit-Change-Number: 54336
Gerrit-PatchSet: 2
Gerrit-Owner: Bill XIE <persmule(a)hardenedlinux.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Bill XIE <persmule(a)hardenedlinux.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Tim Wawrzynczak, Patrick Rudolph.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54334 )
Change subject: soc/intel/alderlake: Add configurable value for UsbTcPortEn
......................................................................
Patch Set 1:
(6 comments)
File src/soc/intel/alderlake/chip.h:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119381):
https://review.coreboot.org/c/coreboot/+/54334/comment/9cafc16f_4283d43a
PS1, Line 44: * Specifies which Type-C Ports are enabled on the system
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119381):
https://review.coreboot.org/c/coreboot/+/54334/comment/c3f1b92b_e976ba34
PS1, Line 45: * each bit represents a port starting at 0
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119381):
https://review.coreboot.org/c/coreboot/+/54334/comment/bb7d57d0_4ac4c002
PS1, Line 46: * Example: set value to 0x3 for ports 0 and 1 to be enabled
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119381):
https://review.coreboot.org/c/coreboot/+/54334/comment/f2d66be8_84fc221d
PS1, Line 47: */
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119381):
https://review.coreboot.org/c/coreboot/+/54334/comment/d216df58_1a597977
PS1, Line 48: uint8_t UsbTcPortEn;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-119381):
https://review.coreboot.org/c/coreboot/+/54334/comment/d3f5a1f7_417864e3
PS1, Line 48: uint8_t UsbTcPortEn;
please, no spaces at the start of a line
--
To view, visit https://review.coreboot.org/c/coreboot/+/54334
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic5cd0690945b9296c105ade2d99f68bbf0ee22bd
Gerrit-Change-Number: 54334
Gerrit-PatchSet: 1
Gerrit-Owner: Sridhar Siricilla <sridhar.siricilla(a)intel.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Sun, 16 May 2021 13:56:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Sridhar Siricilla has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/54334 )
Change subject: soc/intel/alderlake: Add configurable value for UsbTcPortEn
......................................................................
soc/intel/alderlake: Add configurable value for UsbTcPortEn
As a requirement of TCSS this setting needs to be correctly set
to determine what Type-C ports are enabled on the platform.
BUG=b:184324979
TEST=Verified on brya
Signed-off-by: Sridhar Siricilla <sridhar.siricilla(a)intel.com>
Change-Id: Ic5cd0690945b9296c105ade2d99f68bbf0ee22bd
---
M src/soc/intel/alderlake/chip.h
M src/soc/intel/alderlake/fsp_params.c
2 files changed, 9 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/54334/1
diff --git a/src/soc/intel/alderlake/chip.h b/src/soc/intel/alderlake/chip.h
index fb9dd73..d433a00 100644
--- a/src/soc/intel/alderlake/chip.h
+++ b/src/soc/intel/alderlake/chip.h
@@ -39,6 +39,14 @@
/* Enable S0iX support */
int s0ix_enable;
+
+ /*
+ * Specifies which Type-C Ports are enabled on the system
+ * each bit represents a port starting at 0
+ * Example: set value to 0x3 for ports 0 and 1 to be enabled
+ */
+ uint8_t UsbTcPortEn;
+
/* Support for TCSS xhci, xdci, TBT PCIe root ports and DMA controllers */
uint8_t TcssD3HotDisable;
/* Support for TBT PCIe root ports and DMA controllers with D3Hot->D3Cold */
diff --git a/src/soc/intel/alderlake/fsp_params.c b/src/soc/intel/alderlake/fsp_params.c
index 2ab1825..03173b3 100644
--- a/src/soc/intel/alderlake/fsp_params.c
+++ b/src/soc/intel/alderlake/fsp_params.c
@@ -128,6 +128,7 @@
params->D3HotEnable = !config->TcssD3HotDisable;
params->D3ColdEnable = !config->TcssD3ColdDisable;
+ params->UsbTcPortEn = config->UsbTcPortEn;
params->TcssAuxOri = config->TcssAuxOri;
for (i = 0; i < 8; i++)
params->IomTypeCPortPadCfg[i] = config->IomTypeCPortPadCfg[i];
--
To view, visit https://review.coreboot.org/c/coreboot/+/54334
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic5cd0690945b9296c105ade2d99f68bbf0ee22bd
Gerrit-Change-Number: 54334
Gerrit-PatchSet: 1
Gerrit-Owner: Sridhar Siricilla <sridhar.siricilla(a)intel.com>
Gerrit-MessageType: newchange