From 516a31551ef3b4da9840ffa6354aef17bff12535 Mon Sep 17 00:00:00 2001 From: Shon Date: Fri, 19 Jul 2024 13:12:56 +0800 Subject: mb/google/brask/var/bujia: Add PSYS setting According to the Intel OPS spec, the DC power from display is 12~19V@8A max. It can't set PsysPmax by unknown voltage, so get voltage by ec command "ectool adcread 4" then calculate PsysPmax value. The OPS display can supply 90W power, configure psys_pl2 to limit the system power to 90W. BUG=b:329037849 BRANCH=firmware-brya-14505.B TEST= USE="fw_debug" LOCALES="en" emerge-brask chromeos-bmpblk intel-rplfsp intel-adlfsp coreboot chromeos-bootimage Check adcread value by ectool adcread 4. If get 19540, PsysPmax should be 19540 * 8000 ~= 156 W. Check FSP debug log have the following message. PsysPmax = 156W Change-Id: Ic6e9c6ce9f3179c7d63c1169695fbc23188456dd Signed-off-by: Shon Reviewed-on: https://review.coreboot.org/c/coreboot/+/83593 Tested-by: build bot (Jenkins) Reviewed-by: Derek Huang --- .../google/brya/variants/bujia/Makefile.mk | 1 + .../google/brya/variants/bujia/ramstage.c | 84 ++++++++++++++++++++++ 2 files changed, 85 insertions(+) create mode 100644 src/mainboard/google/brya/variants/bujia/ramstage.c (limited to 'src') diff --git a/src/mainboard/google/brya/variants/bujia/Makefile.mk b/src/mainboard/google/brya/variants/bujia/Makefile.mk index d38141ca24..bc39984d6c 100644 --- a/src/mainboard/google/brya/variants/bujia/Makefile.mk +++ b/src/mainboard/google/brya/variants/bujia/Makefile.mk @@ -4,3 +4,4 @@ bootblock-y += gpio.c romstage-y += gpio.c ramstage-y += gpio.c +ramstage-y += ramstage.c diff --git a/src/mainboard/google/brya/variants/bujia/ramstage.c b/src/mainboard/google/brya/variants/bujia/ramstage.c new file mode 100644 index 0000000000..1aaa54b590 --- /dev/null +++ b/src/mainboard/google/brya/variants/bujia/ramstage.c @@ -0,0 +1,84 @@ +/* SPDX-License-Identifier: GPL-2.0-or-later */ + +#include +#include +#include +#include +#include + +const struct cpu_power_limits limits[] = { + /* SKU_ID, TDP (Watts), pl1_min, pl1_max, pl2_min, pl2_max, pl4 */ + { PCI_DID_INTEL_ADL_P_ID_10, 15, 15000, 15000, 55000, 55000, 123000 }, + { PCI_DID_INTEL_ADL_P_ID_7, 15, 15000, 15000, 55000, 55000, 123000 }, + { PCI_DID_INTEL_ADL_P_ID_6, 15, 15000, 15000, 55000, 55000, 123000 }, + { PCI_DID_INTEL_ADL_P_ID_5, 28, 28000, 28000, 64000, 64000, 140000 }, + { PCI_DID_INTEL_ADL_P_ID_3, 28, 28000, 28000, 64000, 64000, 140000 }, + { PCI_DID_INTEL_RPL_P_ID_5, 15, 15000, 15000, 55000, 55000, 100000 }, + { PCI_DID_INTEL_RPL_P_ID_4, 15, 15000, 15000, 55000, 55000, 100000 }, + { PCI_DID_INTEL_RPL_P_ID_3, 15, 15000, 15000, 55000, 55000, 100000 }, +}; + +const struct system_power_limits sys_limits[] = { + /* SKU_ID, TDP (Watts), psys_pl2 (Watts) */ + { PCI_DID_INTEL_ADL_P_ID_10, 15, 90 }, + { PCI_DID_INTEL_ADL_P_ID_7, 15, 90 }, + { PCI_DID_INTEL_ADL_P_ID_6, 15, 90 }, + { PCI_DID_INTEL_ADL_P_ID_5, 28, 90 }, + { PCI_DID_INTEL_ADL_P_ID_3, 28, 90 }, + { PCI_DID_INTEL_RPL_P_ID_5, 15, 90 }, + { PCI_DID_INTEL_RPL_P_ID_4, 15, 90 }, + { PCI_DID_INTEL_RPL_P_ID_3, 15, 90 }, +}; + +/* + * Psys_pmax considerations. + * + * Given the hardware design in bujia, the serial shunt resistor is 0.005ohm. + * The full scale of hardware PSYS signal 1.6v maps to system current 8A + * instead of real system power. The equation is shown below: + * PSYS = 1.6v = (0.005ohm x 8A) x 50 (INA213, gain 50V/V) x R501/(R501 + R510) + * R501/(R501 + R510) = 0.8 = 45.3K / (45.3K + 11.5K) + * + * The Psys_pmax is a SW setting which tells IMVP9.1 the mapping b/w system input + * current and the actual system power. Since there is no voltage information + * from PSYS, different voltage input would map to different Psys_pmax settings: + * For Type-C 15V, the Psys_pmax should be 15v x 8A = 120W + * For Type-C 20V, the Psys_pmax should be 20v x 8A = 160W + * For a barrel jack, the Psys_pmax should be 20v x 8A = 160W + * + * Imagine that there is a Type-c 100W (20V/5A) connected to DUT w/ full loading, + * and the Psys_pmax setting is 160W. Then IMVP9.1 can calculate the current system + * power = 160W * 5A / 8A = 100W, which is the actual system power. + */ +struct psys_config psys_config = { + .efficiency = 97, + .psys_imax_ma = 8000, + .bj_volts_mv = 12000 +}; + +static int32_t get_input_power_voltage(void) +{ + struct ec_params_adc_read req = { + .adc_channel = 4, + }; + struct ec_response_adc_read info; + struct chromeec_command cmd = { + .cmd_code = EC_CMD_ADC_READ, + .cmd_size_in = sizeof(req), + .cmd_data_in = &req, + .cmd_size_out = sizeof(info), + .cmd_data_out = &info, + }; + if (!google_chromeec_command(&cmd)) + return psys_config.bj_volts_mv; + + return info.adc_value; +} + +void variant_devtree_update(void) +{ + psys_config.bj_volts_mv = get_input_power_voltage(); + size_t total_entries = ARRAY_SIZE(limits); + variant_update_psys_power_limits(limits, sys_limits, total_entries, &psys_config); + variant_update_power_limits(limits, total_entries); +} -- cgit v1.2.3