[S] Change in coreboot[master]: soc/amd/mendocino: Reinterpret smu_power_and_thm_limit

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/74126 )

Change subject: soc/amd/mendocino: Reinterpret smu_power_and_thm_limit
......................................................................

soc/amd/mendocino: Reinterpret smu_power_and_thm_limit

The FSP will return the TDP in the format 0xX0000, where 'X' is the
value we're interested in. For example: 0xF0000 (15W), 0x60000 (6W).
Re-interpret the value so the caller just sees the TDP directly, without
needing to re-interpret things themselves.

BUG=b:253301653
TEST=Manually verify value is correct

Change-Id: I632e702d986a4ac85605040e09c1afab2bbdc59d
Signed-off-by: Tim Van Patten 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74126
Reviewed-by: Felix Held 
Tested-by: build bot (Jenkins) 
---
M src/soc/amd/mendocino/fsp_misc_data_hob.c
1 file changed, 29 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Felix Held: Looks good to me, approved

diff --git a/src/soc/amd/mendocino/fsp_misc_data_hob.c b/src/soc/amd/mendocino/fsp_misc_data_hob.c
index 103f821..3b51fd8 100644
--- a/src/soc/amd/mendocino/fsp_misc_data_hob.c
+++ b/src/soc/amd/mendocino/fsp_misc_data_hob.c
@@ -43,7 +43,14 @@
    if (get_amd_misc_data_hob(&fsp_misc_data, AMD_MISC_DATA_VERSION) != CB_SUCCESS)
    	return CB_ERR;

-	*tdp = fsp_misc_data->smu_power_and_thm_limit;
+	/*
+	 * The FSP will return the TDP in the format 0xX0000, where 'X' is the value
+	 * we're interested in. For example: 0xF0000 (15W), 0x60000 (6W). Re-interpret
+	 * the value so the caller just sees the TDP.
+	 */
+	printk(BIOS_INFO, "fsp_misc_data->smu_power_and_thm_limit = 0x%08X\n",
+		fsp_misc_data->smu_power_and_thm_limit);
+	*tdp = fsp_misc_data->smu_power_and_thm_limit >> 16;

return CB_SUCCESS;
 }