soc/intel/skylake: Enable LAN depending on devicetree configuration

Currently LAN gets enabled by the option EnableLan, but this
duplicates the devicetree on/off options. Therefore use the on/off
options for the enablement of the LAN controller.

I checked all corresponding mainboards if the devicetree configuration
matches the EnableLan setting.

Change-Id: I36347e8e0f0ddba47aec52aeb6bc047e3c8bfaa4
Signed-off-by: Felix Singer <felixsinger@posteo.net>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43844
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Michael Niewöhner
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
Felix Singer 2020-07-25 07:50:51 +02:00 committed by Michael Niewöhner
parent 0901d03085
commit 57c8143350
21 changed files with 3 additions and 25 deletions

View File

@ -36,7 +36,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "1" register "SataSalpSupport" = "1"
register "SataMode" = "0" register "SataMode" = "0"

View File

@ -125,8 +125,6 @@ chip soc/intel/skylake
.voltage_limit = 1520 \ .voltage_limit = 1520 \
}" }"
register "EnableLan" = "0"
# USB # USB
register "usb2_ports[0]" = "USB2_PORT_MID(OC0)" register "usb2_ports[0]" = "USB2_PORT_MID(OC0)"
register "usb2_ports[1]" = "USB2_PORT_MID(OC0)" register "usb2_ports[1]" = "USB2_PORT_MID(OC0)"

View File

@ -47,7 +47,6 @@ chip soc/intel/skylake
register "Cio2Enable" = "0" register "Cio2Enable" = "0"
register "PmTimerDisabled" = "1" register "PmTimerDisabled" = "1"
register "HeciEnabled" = "0" register "HeciEnabled" = "0"
register "EnableLan" = "1"
register "SataSalpSupport" = "1" register "SataSalpSupport" = "1"
register "SataPortsEnable" = "{ \ register "SataPortsEnable" = "{ \

View File

@ -36,7 +36,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "0" register "SataPortsEnable[0]" = "0"

View File

@ -65,7 +65,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "1" register "SataPortsEnable[0]" = "1"

View File

@ -38,7 +38,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "0" register "SataPortsEnable[0]" = "0"

View File

@ -42,7 +42,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "0" register "SataPortsEnable[0]" = "0"

View File

@ -32,7 +32,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "0" register "SataPortsEnable[0]" = "0"

View File

@ -32,7 +32,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "EnableAzalia" = "1" register "EnableAzalia" = "1"

View File

@ -32,7 +32,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "0" register "SataPortsEnable[0]" = "0"

View File

@ -37,7 +37,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "0" register "SataPortsEnable[0]" = "0"

View File

@ -42,7 +42,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "0" register "SataPortsEnable[0]" = "0"

View File

@ -32,7 +32,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "0" register "SataPortsEnable[0]" = "0"

View File

@ -50,8 +50,6 @@ chip soc/intel/skylake
# RP17, uses uses CLK SRC 7 # RP17, uses uses CLK SRC 7
register "PcieRpClkSrcNumber[16]" = "7" register "PcieRpClkSrcNumber[16]" = "7"
register EnableLan = "1"
# USB related # USB related
register "SsicPortEnable" = "1" register "SsicPortEnable" = "1"

View File

@ -140,8 +140,6 @@ chip soc/intel/skylake
register "PcieRpClkReqNumber[5]" = "0" register "PcieRpClkReqNumber[5]" = "0"
register "PcieRpClkReqNumber[12]" = "1" register "PcieRpClkReqNumber[12]" = "1"
register "EnableLan" = "1"
# USB related # USB related
register "SsicPortEnable" = "1" register "SsicPortEnable" = "1"

View File

@ -38,7 +38,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "1" register "SataPortsEnable[0]" = "1"

View File

@ -34,7 +34,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "EnableAzalia" = "0" register "EnableAzalia" = "0"

View File

@ -45,7 +45,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "1" register "SataPortsEnable[0]" = "1"

View File

@ -27,7 +27,6 @@ chip soc/intel/skylake
# FSP Configuration # FSP Configuration
register "ProbelessTrace" = "0" register "ProbelessTrace" = "0"
register "EnableLan" = "0"
register "SataSalpSupport" = "0" register "SataSalpSupport" = "0"
register "SataMode" = "0" register "SataMode" = "0"
register "SataPortsEnable[0]" = "0" register "SataPortsEnable[0]" = "0"

View File

@ -242,8 +242,9 @@ void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd)
params->PchPmWoWlanDeepSxEnable = config->PchPmWoWlanDeepSxEnable; params->PchPmWoWlanDeepSxEnable = config->PchPmWoWlanDeepSxEnable;
params->PchPmLanWakeFromDeepSx = config->WakeConfigPcieWakeFromDeepSx; params->PchPmLanWakeFromDeepSx = config->WakeConfigPcieWakeFromDeepSx;
params->PchLanEnable = config->EnableLan; dev = pcidev_path_on_root(PCH_DEVFN_GBE);
if (config->EnableLan) { params->PchLanEnable = dev ? dev->enabled : 0;
if (params->PchLanEnable) {
params->PchLanLtrEnable = config->EnableLanLtr; params->PchLanLtrEnable = config->EnableLanLtr;
params->PchLanK1OffEnable = config->EnableLanK1Off; params->PchLanK1OffEnable = config->EnableLanK1Off;
params->PchLanClkReqSupported = config->LanClkReqSupported; params->PchLanClkReqSupported = config->LanClkReqSupported;

View File

@ -136,7 +136,6 @@ struct soc_intel_skylake_config {
u8 CmdTriStateDis; u8 CmdTriStateDis;
/* Lan */ /* Lan */
u8 EnableLan;
u8 EnableLanLtr; u8 EnableLanLtr;
u8 EnableLanK1Off; u8 EnableLanK1Off;
u8 LanClkReqSupported; u8 LanClkReqSupported;