From 70770ebd36621715234ca6c427caa039ea4cd4f7 Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Wed, 29 Nov 2023 01:05:58 +0530 Subject: mb/google/rex: Simplify power limit configuration usage This patch removes the deprecated PL_PERFORMANCE and PL_BASELINE configurations, relying instead on the refactored power limit flow. This flow allows for seamless overrides by the baseboard and/or by the variant board, if necessary. Specifically, this patch: - Removes PL_PERFORMANCE and PL_BASELINE configuration options from mainboard.c in the google/rex directory. - Relies on the baseboard_devtree_update() function, which is implemented by the respective baseboard, to handle power limit configuration. - Leverages the variant_devtree_update() function, which is a __weak implementation, to allow overrides by the variant directory. This simplification improves code readability and maintainability while maintaining the flexibility to handle power limit configurations as needed. Change-Id: I872e5cb59d7b2789ef517d4a090189785db46b85 Signed-off-by: Subrata Banik Reviewed-on: https://review.coreboot.org/c/coreboot/+/79331 Reviewed-by: Kapil Porwal Tested-by: build bot (Jenkins) --- .../google/rex/variants/baseboard/rex/ramstage.c | 45 +--------------------- 1 file changed, 1 insertion(+), 44 deletions(-) (limited to 'src/mainboard/google/rex/variants') diff --git a/src/mainboard/google/rex/variants/baseboard/rex/ramstage.c b/src/mainboard/google/rex/variants/baseboard/rex/ramstage.c index 7c09651046..14fbad2462 100644 --- a/src/mainboard/google/rex/variants/baseboard/rex/ramstage.c +++ b/src/mainboard/google/rex/variants/baseboard/rex/ramstage.c @@ -10,7 +10,7 @@ * pl2_min (milliWatts), pl2_max (milliWatts), pl4 (milliWatts) * Following values are for performance config as per document #640982 */ -#if CONFIG(PL_PERFORMANCE) + const struct cpu_tdp_power_limits performance_efficient_limits[] = { { .mch_id = PCI_DID_INTEL_MTL_P_ID_2, @@ -52,49 +52,6 @@ const struct cpu_tdp_power_limits power_optimized_limits[] = { .pl4_power = 64000 }, }; -#else -const struct cpu_tdp_power_limits performance_efficient_limits[] = { - { - .mch_id = PCI_DID_INTEL_MTL_P_ID_2, - .cpu_tdp = 15, - .pl1_min_power = 10000, - .pl1_max_power = 15000, - .pl2_min_power = 40000, - .pl2_max_power = 40000, - .pl4_power = 84000 - }, - { - .mch_id = PCI_DID_INTEL_MTL_P_ID_5, - .cpu_tdp = 15, - .pl1_min_power = 10000, - .pl1_max_power = 15000, - .pl2_min_power = 40000, - .pl2_max_power = 40000, - .pl4_power = 84000 - }, -}; - -const struct cpu_tdp_power_limits power_optimized_limits[] = { - { - .mch_id = PCI_DID_INTEL_MTL_P_ID_2, - .cpu_tdp = 15, - .pl1_min_power = 10000, - .pl1_max_power = 15000, - .pl2_min_power = 40000, - .pl2_max_power = 40000, - .pl4_power = 47000 - }, - { - .mch_id = PCI_DID_INTEL_MTL_P_ID_5, - .cpu_tdp = 15, - .pl1_min_power = 10000, - .pl1_max_power = 15000, - .pl2_min_power = 40000, - .pl2_max_power = 40000, - .pl4_power = 47000 - }, -}; -#endif void __weak variant_devtree_update(void) { -- cgit v1.2.3