Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak, Patrick Rudolph.
EricR Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49338 )
Change subject: spd: Create new folder for SPD files
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
woops,link not allowed
Report any symbolic links (lint-stable-018-symlinks): src/spd/ddr4/tgl
src/spd/lp4x/adl
src/spd/lp4x/jsl
src/spd/lp4x/tgl
test failed
--
To view, visit https://review.coreboot.org/c/coreboot/+/49338
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3a871981642485020adaeaafd062f718c636a708
Gerrit-Change-Number: 49338
Gerrit-PatchSet: 1
Gerrit-Owner: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Furquan Shaikh <furquan(a)google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Tue, 12 Jan 2021 08:01:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: David Wu, Nick Vaccaro.
Jim Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49302 )
Change subject: mb/google/volteer/var/voema: Add camera ACPI configuration
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://review.coreboot.org/c/coreboot/+/49302
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I903e5e0b5f85718c7c9cbb6d5cafb8fc9ad5814e
Gerrit-Change-Number: 49302
Gerrit-PatchSet: 1
Gerrit-Owner: David Wu <david_wu(a)quanta.corp-partner.google.com>
Gerrit-Reviewer: Jim Lai <jim.lai(a)intel.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Andy Yeh <andy.yeh(a)intel.com>
Gerrit-CC: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-CC: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: David Wu <david_wu(a)quanta.corp-partner.google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Comment-Date: Tue, 12 Jan 2021 08:00:44 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Furquan Shaikh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/48968 )
Change subject: soc/intel/common/pcie: Add helper function for getting mask of enabled ports
......................................................................
soc/intel/common/pcie: Add helper function for getting mask of enabled ports
This change adds a helper function `pcie_rp_enable_mask()` that
returns a 32-bit mask indicating the status (enabled/disabled) of PCIe
root ports (in the groups table) as configured by the mainboard in the
device tree.
With this helper function, SoC chip config does not need to add
another `PcieRpEnable[]` config to identify what root ports are
enabled.
Change-Id: I7ce5fca1c662064fd21f0961dac13cda1fa2ca44
Signed-off-by: Furquan Shaikh <furquan(a)google.com>
---
M src/soc/intel/common/block/include/intelblocks/pcie_rp.h
M src/soc/intel/common/block/pcie/Makefile.inc
A src/soc/intel/common/block/pcie/pcie_helpers.c
3 files changed, 58 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/68/48968/1
diff --git a/src/soc/intel/common/block/include/intelblocks/pcie_rp.h b/src/soc/intel/common/block/include/intelblocks/pcie_rp.h
index ca50b13..264c43f 100644
--- a/src/soc/intel/common/block/include/intelblocks/pcie_rp.h
+++ b/src/soc/intel/common/block/include/intelblocks/pcie_rp.h
@@ -3,6 +3,8 @@
#ifndef SOC_INTEL_COMMON_BLOCK_PCIE_RP_H
#define SOC_INTEL_COMMON_BLOCK_PCIE_RP_H
+#include <stdint.h>
+
/*
* The PCIe Root Ports usually come in groups of up to 8 PCI-device
* functions.
@@ -34,4 +36,17 @@
*/
void pcie_rp_update_devicetree(const struct pcie_rp_group *groups);
+/*
+ * Return mask of PCIe root ports that are enabled by mainboard. Mask is set in the same order
+ * as the root ports in pcie_rp_group groups table.
+ *
+ * Thus, the status of first root port in the groups table is indicated by bit 0 in the returned
+ * mask, second root port by bit 1 and so on.
+
+ * 1 in the bit position indicates root port is enabled, whereas 0 indicates root port is
+ * disabled. This function assumes that the maximum count of root ports in the groups table is
+ * <= 32.
+ */
+uint32_t pcie_rp_enable_mask(const struct pcie_rp_group *groups);
+
#endif /* SOC_INTEL_COMMON_BLOCK_PCIE_RP_H */
diff --git a/src/soc/intel/common/block/pcie/Makefile.inc b/src/soc/intel/common/block/pcie/Makefile.inc
index e2ad685..521ca6b 100644
--- a/src/soc/intel/common/block/pcie/Makefile.inc
+++ b/src/soc/intel/common/block/pcie/Makefile.inc
@@ -1,4 +1,7 @@
subdirs-y += ./*
+romstage-$(CONFIG_SOC_INTEL_COMMON_BLOCK_PCIE) += pcie_helpers.c
+
ramstage-$(CONFIG_SOC_INTEL_COMMON_BLOCK_PCIE) += pcie.c
+ramstage-$(CONFIG_SOC_INTEL_COMMON_BLOCK_PCIE) += pcie_helpers.c
ramstage-$(CONFIG_SOC_INTEL_COMMON_BLOCK_PCIE) += pcie_rp.c
diff --git a/src/soc/intel/common/block/pcie/pcie_helpers.c b/src/soc/intel/common/block/pcie/pcie_helpers.c
new file mode 100644
index 0000000..dec786e
--- /dev/null
+++ b/src/soc/intel/common/block/pcie/pcie_helpers.c
@@ -0,0 +1,40 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#include <console/console.h>
+#include <device/device.h>
+#include <intelblocks/pcie_rp.h>
+#include <stdint.h>
+
+static uint32_t pcie_slot_enable_mask(int slot, int count)
+{
+ uint32_t mask = 0;
+ unsigned int i;
+ const struct device *dev;
+
+ for (i = 0; i < count; i++) {
+ dev = pcidev_on_root(slot, i);
+ if (is_dev_enabled(dev))
+ mask |= BIT(i);
+ }
+
+ return mask;
+}
+
+uint32_t pcie_rp_enable_mask(const struct pcie_rp_group *const groups)
+{
+ uint32_t mask = 0;
+ uint32_t offset = 0;
+ const struct pcie_rp_group *group;
+
+ for (group = groups; group->count; ++group) {
+ if (group->count + offset >= sizeof(mask) * 8) {
+ printk(BIOS_ERR, "ERROR: %s: Root port count greater than mask size!\n",
+ __func__);
+ break;
+ }
+ mask |= pcie_slot_enable_mask(group->slot, group->count) << offset;
+ offset += group->count;
+ }
+
+ return mask;
+}
--
To view, visit https://review.coreboot.org/c/coreboot/+/48968
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7ce5fca1c662064fd21f0961dac13cda1fa2ca44
Gerrit-Change-Number: 48968
Gerrit-PatchSet: 1
Gerrit-Owner: Furquan Shaikh <furquan(a)google.com>
Gerrit-MessageType: newchange
Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak, Patrick Rudolph, EricR Lai.
EricR Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49338 )
Change subject: spd: Create new folder for SPD files
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
@Furquan, any name can replace set1/set2? Can we use the main attribute
--
To view, visit https://review.coreboot.org/c/coreboot/+/49338
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3a871981642485020adaeaafd062f718c636a708
Gerrit-Change-Number: 49338
Gerrit-PatchSet: 1
Gerrit-Owner: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Furquan Shaikh <furquan(a)google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Attention: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-Comment-Date: Tue, 12 Jan 2021 07:59:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
EricR Lai has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/49338 )
Change subject: spd: Create new folder for SPD files
......................................................................
spd: Create new folder for SPD files
SPD file can share with different platforms if have the same MRC
calculating. TGL and ADL can share the same SPD file generated by
spd_tools.
lp4x
|_ set1/
|_ set2/
|_ jsl --> set1
|_ tgl --> set2
|_ adl --> set2
ddr4
|_ set1/
|_ tgl --> set1
Signed-off-by: Eric Lai <ericr_lai(a)compal.corp-partner.google.com>
Change-Id: I3a871981642485020adaeaafd062f718c636a708
---
R src/spd/ddr4/set1/ddr4-spd-1.hex
R src/spd/ddr4/set1/ddr4-spd-2.hex
R src/spd/ddr4/set1/ddr4-spd-3.hex
R src/spd/ddr4/set1/ddr4-spd-4.hex
R src/spd/ddr4/set1/ddr4-spd-5.hex
R src/spd/ddr4/set1/ddr4-spd-6.hex
R src/spd/ddr4/set1/ddr4-spd-7.hex
R src/spd/ddr4/set1/ddr4-spd-8.hex
R src/spd/ddr4/set1/ddr4-spd-9.hex
R src/spd/ddr4/set1/ddr4-spd-empty.hex
R src/spd/ddr4/set1/ddr4_spd_manifest.generated.txt
R src/spd/ddr4/set1/placeholder.spd.hex
A src/spd/ddr4/tgl
A src/spd/lp4x/adl
A src/spd/lp4x/jsl
R src/spd/lp4x/set1/lp4x-spd-1.hex
R src/spd/lp4x/set1/lp4x-spd-2.hex
R src/spd/lp4x/set1/lp4x-spd-3.hex
R src/spd/lp4x/set1/lp4x-spd-4.hex
R src/spd/lp4x/set1/lp4x-spd-5.hex
R src/spd/lp4x/set1/lp4x-spd-6.hex
R src/spd/lp4x/set1/lp4x-spd-7.hex
R src/spd/lp4x/set1/lp4x_spd_manifest.generated.txt
R src/spd/lp4x/set1/placeholder.spd.hex
R src/spd/lp4x/set2/lp4x-spd-1.hex
R src/spd/lp4x/set2/lp4x-spd-2.hex
R src/spd/lp4x/set2/lp4x-spd-3.hex
R src/spd/lp4x/set2/lp4x-spd-4.hex
R src/spd/lp4x/set2/lp4x-spd-5.hex
R src/spd/lp4x/set2/lp4x_spd_manifest.generated.txt
C src/spd/lp4x/set2/placeholder.spd.hex
A src/spd/lp4x/tgl
32 files changed, 4 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/49338/1
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-1.hex b/src/spd/ddr4/set1/ddr4-spd-1.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-1.hex
rename to src/spd/ddr4/set1/ddr4-spd-1.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-2.hex b/src/spd/ddr4/set1/ddr4-spd-2.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-2.hex
rename to src/spd/ddr4/set1/ddr4-spd-2.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-3.hex b/src/spd/ddr4/set1/ddr4-spd-3.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-3.hex
rename to src/spd/ddr4/set1/ddr4-spd-3.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-4.hex b/src/spd/ddr4/set1/ddr4-spd-4.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-4.hex
rename to src/spd/ddr4/set1/ddr4-spd-4.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-5.hex b/src/spd/ddr4/set1/ddr4-spd-5.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-5.hex
rename to src/spd/ddr4/set1/ddr4-spd-5.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-6.hex b/src/spd/ddr4/set1/ddr4-spd-6.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-6.hex
rename to src/spd/ddr4/set1/ddr4-spd-6.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-7.hex b/src/spd/ddr4/set1/ddr4-spd-7.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-7.hex
rename to src/spd/ddr4/set1/ddr4-spd-7.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-8.hex b/src/spd/ddr4/set1/ddr4-spd-8.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-8.hex
rename to src/spd/ddr4/set1/ddr4-spd-8.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-9.hex b/src/spd/ddr4/set1/ddr4-spd-9.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-9.hex
rename to src/spd/ddr4/set1/ddr4-spd-9.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4-spd-empty.hex b/src/spd/ddr4/set1/ddr4-spd-empty.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4-spd-empty.hex
rename to src/spd/ddr4/set1/ddr4-spd-empty.hex
diff --git a/src/soc/intel/tigerlake/spd/ddr4_spd_manifest.generated.txt b/src/spd/ddr4/set1/ddr4_spd_manifest.generated.txt
similarity index 100%
rename from src/soc/intel/tigerlake/spd/ddr4_spd_manifest.generated.txt
rename to src/spd/ddr4/set1/ddr4_spd_manifest.generated.txt
diff --git a/src/soc/intel/tigerlake/spd/placeholder.spd.hex b/src/spd/ddr4/set1/placeholder.spd.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/placeholder.spd.hex
rename to src/spd/ddr4/set1/placeholder.spd.hex
diff --git a/src/spd/ddr4/tgl b/src/spd/ddr4/tgl
new file mode 120000
index 0000000..c9d22dd
--- /dev/null
+++ b/src/spd/ddr4/tgl
@@ -0,0 +1 @@
+set1
\ No newline at end of file
diff --git a/src/spd/lp4x/adl b/src/spd/lp4x/adl
new file mode 120000
index 0000000..6dc17fc
--- /dev/null
+++ b/src/spd/lp4x/adl
@@ -0,0 +1 @@
+set2
\ No newline at end of file
diff --git a/src/spd/lp4x/jsl b/src/spd/lp4x/jsl
new file mode 120000
index 0000000..c9d22dd
--- /dev/null
+++ b/src/spd/lp4x/jsl
@@ -0,0 +1 @@
+set1
\ No newline at end of file
diff --git a/src/soc/intel/jasperlake/spd/lp4x-spd-1.hex b/src/spd/lp4x/set1/lp4x-spd-1.hex
similarity index 100%
rename from src/soc/intel/jasperlake/spd/lp4x-spd-1.hex
rename to src/spd/lp4x/set1/lp4x-spd-1.hex
diff --git a/src/soc/intel/jasperlake/spd/lp4x-spd-2.hex b/src/spd/lp4x/set1/lp4x-spd-2.hex
similarity index 100%
rename from src/soc/intel/jasperlake/spd/lp4x-spd-2.hex
rename to src/spd/lp4x/set1/lp4x-spd-2.hex
diff --git a/src/soc/intel/jasperlake/spd/lp4x-spd-3.hex b/src/spd/lp4x/set1/lp4x-spd-3.hex
similarity index 100%
rename from src/soc/intel/jasperlake/spd/lp4x-spd-3.hex
rename to src/spd/lp4x/set1/lp4x-spd-3.hex
diff --git a/src/soc/intel/jasperlake/spd/lp4x-spd-4.hex b/src/spd/lp4x/set1/lp4x-spd-4.hex
similarity index 100%
rename from src/soc/intel/jasperlake/spd/lp4x-spd-4.hex
rename to src/spd/lp4x/set1/lp4x-spd-4.hex
diff --git a/src/soc/intel/jasperlake/spd/lp4x-spd-5.hex b/src/spd/lp4x/set1/lp4x-spd-5.hex
similarity index 100%
rename from src/soc/intel/jasperlake/spd/lp4x-spd-5.hex
rename to src/spd/lp4x/set1/lp4x-spd-5.hex
diff --git a/src/soc/intel/jasperlake/spd/lp4x-spd-6.hex b/src/spd/lp4x/set1/lp4x-spd-6.hex
similarity index 100%
rename from src/soc/intel/jasperlake/spd/lp4x-spd-6.hex
rename to src/spd/lp4x/set1/lp4x-spd-6.hex
diff --git a/src/soc/intel/jasperlake/spd/lp4x-spd-7.hex b/src/spd/lp4x/set1/lp4x-spd-7.hex
similarity index 100%
rename from src/soc/intel/jasperlake/spd/lp4x-spd-7.hex
rename to src/spd/lp4x/set1/lp4x-spd-7.hex
diff --git a/src/soc/intel/jasperlake/spd/lp4x_spd_manifest.generated.txt b/src/spd/lp4x/set1/lp4x_spd_manifest.generated.txt
similarity index 100%
rename from src/soc/intel/jasperlake/spd/lp4x_spd_manifest.generated.txt
rename to src/spd/lp4x/set1/lp4x_spd_manifest.generated.txt
diff --git a/src/soc/intel/jasperlake/spd/placeholder.spd.hex b/src/spd/lp4x/set1/placeholder.spd.hex
similarity index 100%
rename from src/soc/intel/jasperlake/spd/placeholder.spd.hex
rename to src/spd/lp4x/set1/placeholder.spd.hex
diff --git a/src/soc/intel/tigerlake/spd/lp4x-spd-1.hex b/src/spd/lp4x/set2/lp4x-spd-1.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/lp4x-spd-1.hex
rename to src/spd/lp4x/set2/lp4x-spd-1.hex
diff --git a/src/soc/intel/tigerlake/spd/lp4x-spd-2.hex b/src/spd/lp4x/set2/lp4x-spd-2.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/lp4x-spd-2.hex
rename to src/spd/lp4x/set2/lp4x-spd-2.hex
diff --git a/src/soc/intel/tigerlake/spd/lp4x-spd-3.hex b/src/spd/lp4x/set2/lp4x-spd-3.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/lp4x-spd-3.hex
rename to src/spd/lp4x/set2/lp4x-spd-3.hex
diff --git a/src/soc/intel/tigerlake/spd/lp4x-spd-4.hex b/src/spd/lp4x/set2/lp4x-spd-4.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/lp4x-spd-4.hex
rename to src/spd/lp4x/set2/lp4x-spd-4.hex
diff --git a/src/soc/intel/tigerlake/spd/lp4x-spd-5.hex b/src/spd/lp4x/set2/lp4x-spd-5.hex
similarity index 100%
rename from src/soc/intel/tigerlake/spd/lp4x-spd-5.hex
rename to src/spd/lp4x/set2/lp4x-spd-5.hex
diff --git a/src/soc/intel/tigerlake/spd/lp4x_spd_manifest.generated.txt b/src/spd/lp4x/set2/lp4x_spd_manifest.generated.txt
similarity index 100%
rename from src/soc/intel/tigerlake/spd/lp4x_spd_manifest.generated.txt
rename to src/spd/lp4x/set2/lp4x_spd_manifest.generated.txt
diff --git a/src/soc/intel/tigerlake/spd/placeholder.spd.hex b/src/spd/lp4x/set2/placeholder.spd.hex
similarity index 100%
copy from src/soc/intel/tigerlake/spd/placeholder.spd.hex
copy to src/spd/lp4x/set2/placeholder.spd.hex
diff --git a/src/spd/lp4x/tgl b/src/spd/lp4x/tgl
new file mode 120000
index 0000000..6dc17fc
--- /dev/null
+++ b/src/spd/lp4x/tgl
@@ -0,0 +1 @@
+set2
\ No newline at end of file
--
To view, visit https://review.coreboot.org/c/coreboot/+/49338
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3a871981642485020adaeaafd062f718c636a708
Gerrit-Change-Number: 49338
Gerrit-PatchSet: 1
Gerrit-Owner: EricR Lai <ericr_lai(a)compal.corp-partner.google.com>
Gerrit-MessageType: newchange
Attention is currently required from: Nico Huber, Tim Wawrzynczak, Subrata Banik, Angel Pons, Patrick Rudolph.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49291 )
Change subject: soc/intel/adl: Add new Kconfig ENABLE_DISPLAY_OVER_DGPU for ADLRVP with DG
......................................................................
Patch Set 2:
(1 comment)
File src/mainboard/intel/adlrvp/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/49291/comment/c7abb017_8e8b0517
PS2, Line 192: #if CONFIG(ENABLE_DISPLAY_OVER_DGPU)
I don't think this works as you intend it to. Are you really seeing device .enabled being set differently in static.c depending upon the CONFIG() value?
--
To view, visit https://review.coreboot.org/c/coreboot/+/49291
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ibbe9c8c4d77018de83815d7d203284b1fbc0da58
Gerrit-Change-Number: 49291
Gerrit-PatchSet: 2
Gerrit-Owner: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Nico Huber <nico.h(a)gmx.de>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Subrata Banik <subrata.banik(a)intel.com>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Tue, 12 Jan 2021 07:13:39 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: John Zhao, Tim Wawrzynczak, Nick Vaccaro, Srinidhi N Kaushik, Raj Astekar, Patrick Rudolph.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49244 )
Change subject: soc/intel/tigerlake: Disable TC cold support
......................................................................
Patch Set 7:
(1 comment)
File src/soc/intel/tigerlake/fsp_params.c:
https://review.coreboot.org/c/coreboot/+/49244/comment/2aee334e_d200a370
PS6, Line 138: 1
> This implementation would be for the whole Tigerlake platform.
Right. But some mainboards for TGL might choose to not disable TCCold. Not volteer boards, but maybe other mainboards.
--
To view, visit https://review.coreboot.org/c/coreboot/+/49244
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0ad0322693b4f8fbf1000b24eb21dddcebec686b
Gerrit-Change-Number: 49244
Gerrit-PatchSet: 7
Gerrit-Owner: Srinidhi N Kaushik <srinidhi.n.kaushik(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: John Zhao <john.zhao(a)intel.com>
Gerrit-Reviewer: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Raj Astekar <raj.astekar(a)intel.com>
Gerrit-Reviewer: Srinidhi N Kaushik <srinidhi.n.kaushik(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-Attention: John Zhao <john.zhao(a)intel.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Attention: Srinidhi N Kaushik <srinidhi.n.kaushik(a)intel.com>
Gerrit-Attention: Raj Astekar <raj.astekar(a)intel.com>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Tue, 12 Jan 2021 06:57:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Furquan Shaikh <furquan(a)google.com>
Comment-In-Reply-To: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-MessageType: comment
Attention is currently required from: John Zhao, Tim Wawrzynczak, Nick Vaccaro, Srinidhi N Kaushik, Raj Astekar, Patrick Rudolph.
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49244 )
Change subject: soc/intel/tigerlake: Disable TC cold support
......................................................................
Patch Set 7: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/49244
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0ad0322693b4f8fbf1000b24eb21dddcebec686b
Gerrit-Change-Number: 49244
Gerrit-PatchSet: 7
Gerrit-Owner: Srinidhi N Kaushik <srinidhi.n.kaushik(a)intel.com>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: John Zhao <john.zhao(a)intel.com>
Gerrit-Reviewer: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Raj Astekar <raj.astekar(a)intel.com>
Gerrit-Reviewer: Srinidhi N Kaushik <srinidhi.n.kaushik(a)intel.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: John Zhao <john.zhao(a)intel.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Attention: Srinidhi N Kaushik <srinidhi.n.kaushik(a)intel.com>
Gerrit-Attention: Raj Astekar <raj.astekar(a)intel.com>
Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Tue, 12 Jan 2021 06:56:43 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment