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

mb/google/brya/var/kinox: Support DPTF oem_variables 

Enable DPTF oem_variables and override based on charger type.

BUG=b:230803675
TEST=1. With 90W adapter, check ACPI object ODVX and oem_variable[0]=1
Name (ODVX, Package (0x06)
{
   0x00000001,
   0x00000000,
   0x00000000,
   0x00000000,
   0x00000000,
   0x00000000
})
2. With 65W adapter, check ACPI object ODVX and oem_variable[0]=0
Name (ODVX, Package (0x06)
{
   0x00000000,
   0x00000000,
   0x00000000,
   0x00000000,
   0x00000000,
   0x00000000
})

Signed-off-by: Dtrain Hsu <dtrain_hsu@compal.corp-partner.google.com>
Change-Id: I78929ecbc9db56aa234b3f46c641d1f2f3b7cba8
Reviewed-on: https://review.coreboot.org/c/coreboot/+/65251
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com>
This commit is contained in:
Dtrain Hsu 2022-06-20 16:54:13 +08:00 committed by Felix Held
parent f136ddb9fa
commit 6aadb93355
2 changed files with 23 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/mainboard/google/brya/variants/kinox/overridetree.cb
View File

@ -177,6 +177,10 @@ chip soc/intel/alderlake
}
}"
register "oem_data.oem_variables" = "{
[0] = 0x0
}"
## Charger Performance Control (Control, mA)
register "controls.charger_perf" = "{
[0] = { 255, 1700 },

19
src/mainboard/google/brya/variants/kinox/ramstage.c
View File

@ -6,9 +6,12 @@
#include <device/device.h>
#include <device/pci_ids.h>
#include <device/pci_ops.h>
#include <drivers/intel/dptf/chip.h>
#include <ec/google/chromeec/ec.h>
#include <intelblocks/power_limit.h>
WEAK_DEV_PTR(dptf_policy);
const struct cpu_power_limits baseline_limits[] = {
/* SKU_ID, TDP (Watts), pl1_min, pl1_max, pl2_min, pl2_max, pl4 */
{ PCI_DID_INTEL_ADL_P_ID_10, 15, 12000, 25000, 39000, 39000, 72500 },
@ -79,6 +82,20 @@ const struct psys_config psys_config = {
.bj_volts_mv = 20000
};
static void update_oem_variables_perf(void)
{
const struct device *policy_dev;
struct drivers_intel_dptf_config *config;
policy_dev = DEV_PTR(dptf_policy);
if (!policy_dev)
return;
config = policy_dev->chip_info;
config->oem_data.oem_variables[0] = 1;
printk(BIOS_INFO, "PL124: Update oem_variables to Performance value.\n");
}
static const struct cpu_power_limits *get_power_limit(size_t *total_entries)
{
enum usb_chg_type type;
@ -94,6 +111,7 @@ static const struct cpu_power_limits *get_power_limit(size_t *total_entries)
if (type == USB_CHG_TYPE_PROPRIETARY) {
if (watts == CHARGER_170W) {
printk(BIOS_INFO, "PL124: Performance.\n");
update_oem_variables_perf();
*total_entries = ARRAY_SIZE(perf_limits);
return perf_limits;
} else {
@ -104,6 +122,7 @@ static const struct cpu_power_limits *get_power_limit(size_t *total_entries)
} else {
if (watts >= CHARGER_90W) {
printk(BIOS_INFO, "PL124: Performance.\n");
update_oem_variables_perf();
*total_entries = ARRAY_SIZE(perf_limits);
return perf_limits;
} else {