Revert "device: Add Kconfig options for D3COLD_SUPPORT and NO_S0IX_SUPPORT"

This reverts commit d6e04aa00b.

Reason for revert: Breaks master.

Change-Id: If7daeaaffe3f9ae9f5e2fbecef5817b9b62827d3
Signed-off-by: Felix Singer <felixsinger@posteo.net>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72917
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Ivy Jian <ivy.jian@quanta.corp-partner.google.com>
Reviewed-by: Frank Wu <frank_wu@compal.corp-partner.google.com>
This commit is contained in:
Felix Singer 2023-02-09 01:54:17 +01:00
parent 4a7af6e148
commit 10d4753f40
11 changed files with 49 additions and 37 deletions

View File

@ -326,6 +326,12 @@ config SOC_INTEL_I2C_DEV_MAX
int
default 8
config SOC_INTEL_ALDERLAKE_S3
bool
default n
help
Select if using S3 instead of S0ix to disable D3Cold.
config ENABLE_SATA_TEST_MODE
bool "Enable test mode for SATA margining"
default n

View File

@ -583,7 +583,7 @@ Scope (\_SB.PCI0)
}
}
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_ALDERLAKE_S3)
Method (TCON, 0)
{
/* Reset IOM D3 cold bit if it is in D3 cold now. */
@ -654,7 +654,7 @@ Scope (\_SB.PCI0)
STAT = 0
}
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
/*
* TCSS xHCI device

View File

@ -28,16 +28,16 @@ Name (STAT, 0x1) /* Variable to save power state 1 - D0, 0 - D3C */
Method (_S0W, 0x0)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_ALDERLAKE_S3)
Return (0x04)
#else
Return (0x03)
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
}
Method (_PR0)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_ALDERLAKE_S3)
If (DUID == 0) {
Return (Package() { \_SB.PCI0.D3C, \_SB.PCI0.TBT0 })
} Else {
@ -49,12 +49,12 @@ Method (_PR0)
} Else {
Return (Package() { \_SB.PCI0.TBT1 })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
}
Method (_PR3)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_ALDERLAKE_S3)
If (DUID == 0) {
Return (Package() { \_SB.PCI0.D3C, \_SB.PCI0.TBT0 })
} Else {
@ -66,7 +66,7 @@ Method (_PR3)
} Else {
Return (Package() { \_SB.PCI0.TBT1 })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
}
/*

View File

@ -247,16 +247,16 @@ Method (_PS3, 0, Serialized)
Method (_S0W, 0x0, NotSerialized)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_ALDERLAKE_S3)
Return (0x4)
#else
Return (0x3)
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
}
Method (_PR0)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_ALDERLAKE_S3)
If ((TUID == 0) || (TUID == 1)) {
Return (Package() { \_SB.PCI0.D3C, \_SB.PCI0.TBT0 })
} Else {
@ -268,12 +268,12 @@ Method (_PR0)
} Else {
Return (Package() { \_SB.PCI0.TBT1 })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
}
Method (_PR3)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_ALDERLAKE_S3)
If ((TUID == 0) || (TUID == 1)) {
Return (Package() { \_SB.PCI0.D3C, \_SB.PCI0.TBT0 })
} Else {
@ -285,7 +285,7 @@ Method (_PR3)
} Else {
Return (Package() { \_SB.PCI0.TBT1 })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
}
/*

View File

@ -30,11 +30,11 @@ Method (_PS3, 0, Serialized)
Method (_S0W, 0x0, NotSerialized)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_ALDERLAKE_S3)
Return (0x4)
#else
Return (0x3)
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
}
/*
@ -43,7 +43,7 @@ Method (_S0W, 0x0, NotSerialized)
*/
Name (SD3C, 0)
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_ALDERLAKE_S3)
Method (_PR0)
{
Return (Package () { \_SB.PCI0.D3C })
@ -53,7 +53,7 @@ Method (_PR3)
{
Return (Package () { \_SB.PCI0.D3C })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
/*
* XHCI controller _DSM method

View File

@ -648,7 +648,7 @@ static void fill_fsps_tcss_params(FSP_S_CONFIG *s_cfg,
/* D3Hot and D3Cold for TCSS */
s_cfg->D3HotEnable = !config->tcss_d3_hot_disable;
s_cfg->D3ColdEnable = CONFIG(D3COLD_SUPPORT) && !config->tcss_d3_cold_disable;
s_cfg->D3ColdEnable = !CONFIG(SOC_INTEL_ALDERLAKE_S3) && !config->tcss_d3_cold_disable;
s_cfg->UsbTcPortEn = 0;
for (int i = 0; i < MAX_TYPE_C_PORTS; i++) {

View File

@ -205,6 +205,12 @@ config SOC_INTEL_I2C_DEV_MAX
int
default 6
config SOC_INTEL_TIGERLAKE_S3
bool
default n
help
Select if using S3 instead of S0ix to disable D3Cold
config SOC_INTEL_UART_DEV_MAX
int
default 3

View File

@ -676,7 +676,7 @@ Scope (\_SB.PCI0)
}
}
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_TIGERLAKE_S3)
Method (TCON, 0)
{
/* Reset IOM D3 cold bit if it is in D3 cold now. */
@ -787,7 +787,7 @@ Scope (\_SB.PCI0)
STAT = 0
}
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_TIGERLAKE_S3
/*
* TCSS xHCI device

View File

@ -27,11 +27,11 @@ Name (STAT, 0x1) /* Variable to save power state 1 - D0, 0 - D3C */
Method (_S0W, 0x0)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_TIGERLAKE_S3)
Return (0x04)
#else
Return (0x03)
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_TIGERLAKE_S3
}
/*
@ -40,7 +40,7 @@ Method (_S0W, 0x0)
*/
Method (_PR0)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_TIGERLAKE_S3)
If (DUID == 0) {
Return (Package() { \_SB.PCI0.D3C, \_SB.PCI0.TBT0 })
} Else {
@ -52,12 +52,12 @@ Method (_PR0)
} Else {
Return (Package() { \_SB.PCI0.TBT1 })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_TIGERLAKE_S3
}
Method (_PR3)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_TIGERLAKE_S3)
If (DUID == 0) {
Return (Package() { \_SB.PCI0.D3C, \_SB.PCI0.TBT0 })
} Else {
@ -69,7 +69,7 @@ Method (_PR3)
} Else {
Return (Package() { \_SB.PCI0.TBT1 })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_TIGERLAKE_S3
}
/*

View File

@ -247,16 +247,16 @@ Method (_PS3, 0, Serialized)
Method (_S0W, 0x0, NotSerialized)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_TIGERLAKE_S3)
Return (0x4)
#else
Return (0x3)
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_ALDERLAKE_S3
}
Method (_PR0)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_TIGERLAKE_S3)
If ((TUID == 0) || (TUID == 1)) {
Return (Package() { \_SB.PCI0.D3C, \_SB.PCI0.TBT0 })
} Else {
@ -268,12 +268,12 @@ Method (_PR0)
} Else {
Return (Package() { \_SB.PCI0.TBT1 })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_TIGERLAKE_S3
}
Method (_PR3)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_TIGERLAKE_S3)
If ((TUID == 0) || (TUID == 1)) {
Return (Package() { \_SB.PCI0.D3C, \_SB.PCI0.TBT0 })
} Else {
@ -285,7 +285,7 @@ Method (_PR3)
} Else {
Return (Package() { \_SB.PCI0.TBT1 })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_TIGERLAKE_S3
}
/*

View File

@ -30,11 +30,11 @@ Method (_PS3, 0, Serialized)
Method (_S0W, 0x0, NotSerialized)
{
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_TIGERLAKE_S3)
Return (0x4)
#else
Return (0x3)
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_TIGERLAKE_S3
}
/*
@ -43,7 +43,7 @@ Method (_S0W, 0x0, NotSerialized)
*/
Name (SD3C, 0)
#if CONFIG(D3COLD_SUPPORT)
#if !CONFIG(SOC_INTEL_TIGERLAKE_S3)
Method (_PR0)
{
Return (Package () { \_SB.PCI0.D3C })
@ -53,7 +53,7 @@ Method (_PR3)
{
Return (Package () { \_SB.PCI0.D3C })
}
#endif // D3COLD_SUPPORT
#endif // SOC_INTEL_TIGERLAKE_S3
/*
* XHCI controller _DSM method