mb/google/volteer: Fix USB4 enabling for volteer family

volteer baseboard was currently enabling TBT(USB4) devices in
baseboard devicetree and also selecting the Kconfigs required for
resource allocation above 4G for the USB4 controllers. However, not
all volteer devices have USB4 support. This change fixes USB4 enabling
for volteer family by making the following udpates:
1. TBT devices are moved from baseboard devicetree to individual
override trees for the variants that actually support USB4.
2. When moving TBT devices to override tree, tbt_pcie_rp0 is marked as
on instead of hidden for all variants other than volteer
reference. This is because volteer reference is the only device that
has an asymmetric support for USB4 (i.e. does not support USB4 on C0
port).
3. Kconfig selection for PCIEXP_HOTPLUG is moved to Kconfig.name for
these variants.

Change-Id: If380dcb1ea1633b3a1d6932e769cb6ed0a2761c7
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57112
Reviewed-by: Weimin Wu <wuweimin@huaqin.corp-partner.google.com>
Reviewed-by: Nick Vaccaro <nvaccaro@google.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Furquan Shaikh 2021-08-24 00:38:39 -07:00 committed by Tim Wawrzynczak
parent 9c348a7b7e
commit ae20d4c78f
9 changed files with 75 additions and 48 deletions

View File

@ -106,8 +106,7 @@ config MAINBOARD_PART_NUMBER
default "Volet" if BOARD_GOOGLE_VOLET default "Volet" if BOARD_GOOGLE_VOLET
default "Chronicler" if BOARD_GOOGLE_CHRONICLER default "Chronicler" if BOARD_GOOGLE_CHRONICLER
config PCIEXP_HOTPLUG if PCIEXP_HOTPLUG
default y
# Reserving resources for PCIe Hotplug as per TGL BIOS Spec (doc #611569) # Reserving resources for PCIe Hotplug as per TGL BIOS Spec (doc #611569)
# Revision 0.7.6 Section 7.2.5.1.5 # Revision 0.7.6 Section 7.2.5.1.5
@ -123,6 +122,8 @@ config PCIEXP_HOTPLUG_PREFETCH_MEM
hex hex
default 0x1c000000 # 448 MiB default 0x1c000000 # 448 MiB
endif # PCIEXP_HOTPLUG
config TPM_TIS_ACPI_INTERRUPT config TPM_TIS_ACPI_INTERRUPT
int int
default 21 # GPE0_DW0_21 (GPP_C21) default 21 # GPE0_DW0_21 (GPP_C21)

View File

@ -46,6 +46,7 @@ config BOARD_GOOGLE_VOLTEER
select BOARD_GOOGLE_BASEBOARD_VOLTEER select BOARD_GOOGLE_BASEBOARD_VOLTEER
select VARIANT_HAS_MIPI_CAMERA select VARIANT_HAS_MIPI_CAMERA
select INTEL_CAR_NEM select INTEL_CAR_NEM
select PCIEXP_HOTPLUG
config BOARD_GOOGLE_VOLTEER2 config BOARD_GOOGLE_VOLTEER2
bool "-> Volteer2" bool "-> Volteer2"
@ -53,6 +54,7 @@ config BOARD_GOOGLE_VOLTEER2
select VARIANT_HAS_MIPI_CAMERA select VARIANT_HAS_MIPI_CAMERA
select DRIVERS_GENESYSLOGIC_GL9755 select DRIVERS_GENESYSLOGIC_GL9755
select DRIVER_I2C_TPM_ACPI select DRIVER_I2C_TPM_ACPI
select PCIEXP_HOTPLUG
# Reworked Volteer2 prototype, Haven chip replaced with Dauntless demo board # Reworked Volteer2 prototype, Haven chip replaced with Dauntless demo board
config BOARD_GOOGLE_VOLTEER2_TI50 config BOARD_GOOGLE_VOLTEER2_TI50
@ -61,11 +63,13 @@ config BOARD_GOOGLE_VOLTEER2_TI50
select VARIANT_HAS_MIPI_CAMERA select VARIANT_HAS_MIPI_CAMERA
select DRIVERS_GENESYSLOGIC_GL9755 select DRIVERS_GENESYSLOGIC_GL9755
select DRIVER_I2C_TPM_ACPI select DRIVER_I2C_TPM_ACPI
select PCIEXP_HOTPLUG
config BOARD_GOOGLE_VOXEL config BOARD_GOOGLE_VOXEL
bool "-> Voxel" bool "-> Voxel"
select BOARD_GOOGLE_BASEBOARD_VOLTEER select BOARD_GOOGLE_BASEBOARD_VOLTEER
select GEO_SAR_ENABLE if CHROMEOS_WIFI_SAR select GEO_SAR_ENABLE if CHROMEOS_WIFI_SAR
select PCIEXP_HOTPLUG
config BOARD_GOOGLE_ELEMI config BOARD_GOOGLE_ELEMI
bool "-> Elemi" bool "-> Elemi"
@ -80,10 +84,12 @@ config BOARD_GOOGLE_DROBIT
bool "-> Drobit" bool "-> Drobit"
select BOARD_GOOGLE_BASEBOARD_VOLTEER select BOARD_GOOGLE_BASEBOARD_VOLTEER
select DRIVERS_GENESYSLOGIC_GL9755 select DRIVERS_GENESYSLOGIC_GL9755
select PCIEXP_HOTPLUG
config BOARD_GOOGLE_COPANO config BOARD_GOOGLE_COPANO
bool "-> Copano" bool "-> Copano"
select BOARD_GOOGLE_BASEBOARD_VOLTEER select BOARD_GOOGLE_BASEBOARD_VOLTEER
select PCIEXP_HOTPLUG
config BOARD_GOOGLE_COLLIS config BOARD_GOOGLE_COLLIS
bool "-> Collis" bool "-> Collis"

View File

@ -457,21 +457,6 @@ chip soc/intel/tigerlake
device generic 0 on end device generic 0 on end
end end
end # DPTF 0x9A03 end # DPTF 0x9A03
# Volteer reference design does not have PCIe on Type-C port C0 so it should
# not have hotplug resources allocated. Marking the device hidden will ensure
# it is still enabled so it can participate in power management.
device ref tbt_pcie_rp0 hidden
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref tbt_pcie_rp1 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref tbt_dma0 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref gna on end device ref gna on end
device ref north_xhci on end device ref north_xhci on end
device ref south_xhci on end device ref south_xhci on end

View File

@ -102,6 +102,19 @@ chip soc/intel/tigerlake
device generic 0 on end device generic 0 on end
end end
end end
device ref tbt_pcie_rp0 on
probe DB_USB USB4_GEN3
end
device ref tbt_pcie_rp1 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref tbt_dma0 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref i2c0 on device ref i2c0 on
chip drivers/i2c/generic chip drivers/i2c/generic
register "hid" = ""10EC5682"" register "hid" = ""10EC5682""
@ -204,11 +217,6 @@ chip soc/intel/tigerlake
device generic 0 on end device generic 0 on end
end end
end end
# This variant has USB4/PCIe on both ports so RP0 must be enabled
# in order for hotplug resources to be assigned to Type-C Port C0.
device ref tbt_pcie_rp0 on
probe DB_USB USB4_GEN3
end
device ref pmc hidden device ref pmc hidden
# The pmc_mux chip driver is a placeholder for the # The pmc_mux chip driver is a placeholder for the
# PMC.MUX device in the ACPI hierarchy. # PMC.MUX device in the ACPI hierarchy.

View File

@ -175,11 +175,6 @@ chip soc/intel/tigerlake
end end
end end
end end
# Baseboard has these on, so they must be disabled here.
device ref tbt_pcie_rp0 off end
device ref tbt_pcie_rp1 off end
device ref tbt_dma0 off end
device ref i2c0 on device ref i2c0 on
chip drivers/i2c/generic chip drivers/i2c/generic
register "hid" = ""10EC5682"" register "hid" = ""10EC5682""

View File

@ -25,11 +25,6 @@ chip soc/intel/tigerlake
}" }"
device domain 0 on device domain 0 on
# Baseboard has these on, so they must be disabled here.
device ref tbt_pcie_rp0 off end
device ref tbt_pcie_rp1 off end
device ref tbt_dma0 off end
device ref i2c0 on device ref i2c0 on
chip drivers/i2c/generic chip drivers/i2c/generic
register "hid" = ""10EC5682"" register "hid" = ""10EC5682""

View File

@ -47,6 +47,22 @@ chip soc/intel/tigerlake
register "HybridStorageMode" = "1" register "HybridStorageMode" = "1"
device domain 0 on device domain 0 on
# Volteer reference design does not have PCIe on Type-C port C0 so it should
# not have hotplug resources allocated. Marking the device hidden will ensure
# it is still enabled so it can participate in power management.
device ref tbt_pcie_rp0 hidden
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref tbt_pcie_rp1 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref tbt_dma0 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref ipu on end device ref ipu on end
device ref i2c0 on device ref i2c0 on
chip drivers/i2c/generic chip drivers/i2c/generic

View File

@ -100,6 +100,23 @@ chip soc/intel/tigerlake
device generic 0 on end device generic 0 on end
end end
end end
# Volteer reference design does not have PCIe on Type-C port C0 so it should
# not have hotplug resources allocated. Marking the device hidden will ensure
# it is still enabled so it can participate in power management.
device ref tbt_pcie_rp0 hidden
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref tbt_pcie_rp1 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref tbt_dma0 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref ipu on end # IPU 0x9A19 device ref ipu on end # IPU 0x9A19
device ref i2c0 on device ref i2c0 on
chip drivers/i2c/generic chip drivers/i2c/generic

View File

@ -69,6 +69,26 @@ chip soc/intel/tigerlake
device generic 0 on end device generic 0 on end
end end
end end
device ref tbt_pcie_rp0 on
probe DB_USB USB4_GEN3
end
device ref tbt_pcie_rp1 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
end
device ref tbt_dma0 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
chip drivers/intel/usb4/retimer
register "dfp[0].power_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_H10)"
register "dfp[1].power_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_H10)"
use tcss_usb3_port3 as dfp[0].typec_port
use tcss_usb3_port2 as dfp[1].typec_port
device generic 0 on end
end
end
device ref i2c0 on device ref i2c0 on
chip drivers/i2c/generic chip drivers/i2c/generic
register "hid" = ""10EC5682"" register "hid" = ""10EC5682""
@ -211,22 +231,6 @@ chip soc/intel/tigerlake
device generic 0 on end device generic 0 on end
end end
end end
# This variant has USB4/PCIe on both ports so RP0 must be enabled
# in order for hotplug resources to be assigned to Type-C Port C0.
device ref tbt_pcie_rp0 on
probe DB_USB USB4_GEN3
end
device ref tbt_dma0 on
probe DB_USB USB4_GEN2
probe DB_USB USB4_GEN3
chip drivers/intel/usb4/retimer
register "dfp[0].power_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_H10)"
register "dfp[1].power_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_H10)"
use tcss_usb3_port3 as dfp[0].typec_port
use tcss_usb3_port2 as dfp[1].typec_port
device generic 0 on end
end
end
device ref pmc hidden device ref pmc hidden
# The pmc_mux chip driver is a placeholder for the # The pmc_mux chip driver is a placeholder for the
# PMC.MUX device in the ACPI hierarchy. # PMC.MUX device in the ACPI hierarchy.