Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/84333?usp=email )
Change subject: mb/google/brask/var/bujia: Fix PSYS voltage setting ......................................................................
mb/google/brask/var/bujia: Fix PSYS voltage setting
It return 0 when google_chromeec_command() on success, so get_input_power_voltage() should return adaptor voltage instead of psys_config default value.
BUG=b:329037849 BRANCH=firmware-brya-14505.B TEST= cbmem -c | grep -i PsysPmax
Change-Id: I848c92752b7a7b53f47c6296aad0bdda20e9b0bd Signed-off-by: Shon shon.wang@quanta.corp-partner.google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/84333 Reviewed-by: Derek Huang derekhuang@google.com Reviewed-by: Eric Lai ericllai@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/google/brya/variants/bujia/ramstage.c 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: Derek Huang: Looks good to me, approved build bot (Jenkins): Verified Eric Lai: Looks good to me, approved
diff --git a/src/mainboard/google/brya/variants/bujia/ramstage.c b/src/mainboard/google/brya/variants/bujia/ramstage.c index 1aaa54b..51f5ecf 100644 --- a/src/mainboard/google/brya/variants/bujia/ramstage.c +++ b/src/mainboard/google/brya/variants/bujia/ramstage.c @@ -70,9 +70,9 @@ .cmd_data_out = &info, }; if (!google_chromeec_command(&cmd)) - return psys_config.bj_volts_mv; + return info.adc_value;
- return info.adc_value; + return psys_config.bj_volts_mv; }
void variant_devtree_update(void)