GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

amd/mendocino/root_complex: Restrict DPTC to 15W boards 

Restrict DPTC to 15W boards, since we only have 15W values defined in
the devicetree. This will revert the 6W boards back to their default
values, rather than (incorrectly) configuring them with 15W values.

BUG=b:253301653
TEST=Verify DPTC values are set for 15W boards
TEST=Verify DPTC values are set not set for 6W boards

Change-Id: I94f3974fce6358e3cbb0c30c1af33eb7ecb29ad7
Signed-off-by: Tim Van Patten <timvp@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74127
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Tim Van Patten 2023-03-31 17:31:10 -06:00 committed by Felix Held
parent b9caac74a3
commit f5ae1dd1be
1 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/soc/amd/mendocino/root_complex.c
View File

@ -12,11 +12,13 @@
#include <console/console.h>
#include <device/device.h>
#include <device/pci.h>
#include <fsp/amd_misc_data.h>
#include <fsp/util.h>
#include <soc/iomap.h>
#include <stdint.h>
#include "chip.h"
#define TDP_15W 15
#define DPTC_TOTAL_UPDATE_PARAMS 13
struct dptc_input {
@ -367,7 +369,18 @@ static void acipgen_dptci(void)
static void root_complex_fill_ssdt(const struct device *device)
{
uint32_t tdp = 0;
acpi_fill_root_complex_tom(device);
if (get_amd_smu_reported_tdp(&tdp) != CB_SUCCESS) {
/* Unknown TDP, so return rather than setting invalid values. */
return;
}
/* TODO(b/249359574): Add support for 6W DPTC values. */
if (tdp != TDP_15W)
return;
if (CONFIG(SOC_AMD_COMMON_BLOCK_ACPI_DPTC))
acipgen_dptci();
}