[XS] Change in coreboot[main]: mb/google/brya: Make get_soc_power_limit_config() to public function

[SH Kim (Code Review)]

Attention is currently required from: Dinesh Gehlot, Eric Lai, Kapil Porwal, Nick Vaccaro, Subrata Banik.

SH Kim has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/81616?usp=email )


Change subject: mb/google/brya: Make get_soc_power_limit_config() to public function
......................................................................

mb/google/brya: Make get_soc_power_limit_config() to public function

Make get_soc_power_limit_config() function as a public function to use
on brya variants. Add prefix 'varant_' for it.

BUG=None
BRANCH=brya
TEST=emerge-brya coreboot

Change-Id: I31f938938e7c9da49c2aa7b52dd4b5f46f793495
Signed-off-by: Seunghwan Kim <[e-mail]>
---
M src/mainboard/google/brya/variants/baseboard/brya/ramstage.c
M src/mainboard/google/brya/variants/baseboard/include/baseboard/variants.h
2 files changed, 6 insertions(+), 2 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/16/81616/1

diff --git a/src/mainboard/google/brya/variants/baseboard/brya/ramstage.c b/src/mainboard/google/brya/variants/baseboard/brya/ramstage.c
index a7e5e8a..9ebe0d7 100644
--- a/src/mainboard/google/brya/variants/baseboard/brya/ramstage.c
+++ b/src/mainboard/google/brya/variants/baseboard/brya/ramstage.c
@@ -11,7 +11,7 @@
 
 WEAK_DEV_PTR(dptf_policy);
 
-static struct soc_power_limits_config *get_soc_power_limit_config(void)
+struct soc_power_limits_config *variant_get_soc_power_limit_config(void)
 {
 	config_t *config = config_of_soc();
 	size_t i;
@@ -44,7 +44,7 @@
 	if (!num_entries)
 		return;
 
-	struct soc_power_limits_config *soc_config = get_soc_power_limit_config();
+	struct soc_power_limits_config *soc_config = variant_get_soc_power_limit_config();
 	if (!soc_config)
 		return;
 
diff --git a/src/mainboard/google/brya/variants/baseboard/include/baseboard/variants.h b/src/mainboard/google/brya/variants/baseboard/include/baseboard/variants.h
index ad8eb05..583af59 100644
--- a/src/mainboard/google/brya/variants/baseboard/include/baseboard/variants.h
+++ b/src/mainboard/google/brya/variants/baseboard/include/baseboard/variants.h
@@ -4,6 +4,7 @@
 #define __BASEBOARD_VARIANTS_H__
 
 #include <chip.h>
+#include <intelblocks/power_limit.h>
 #include <soc/gpio.h>
 #include <soc/meminit.h>
 #include <stdint.h>
@@ -70,6 +71,9 @@
 	unsigned int bj_volts_mv;
 };
 
+/* Get soc power limit config struct for current CPU sku */
+struct soc_power_limits_config *variant_get_soc_power_limit_config(void);
+
 /* Modify Power Limit devictree settings during ramstage */
 void variant_update_power_limits(const struct cpu_power_limits *limits,
 				 size_t num_entries);

-- 
To view, visit https://review.coreboot.org/c/coreboot/+/81616?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I31f938938e7c9da49c2aa7b52dd4b5f46f793495
Gerrit-Change-Number: 81616
Gerrit-PatchSet: 1
Gerrit-Owner: SH Kim <[e-mail]>
Gerrit-Reviewer: Dinesh Gehlot <[e-mail]>
Gerrit-Reviewer: Eric Lai <[e-mail]>
Gerrit-Reviewer: Kapil Porwal <[e-mail]>
Gerrit-Reviewer: Nick Vaccaro <[e-mail]>
Gerrit-Reviewer: Subrata Banik <[e-mail]>
Gerrit-Attention: Eric Lai <[e-mail]>
Gerrit-Attention: Subrata Banik <[e-mail]>
Gerrit-Attention: Kapil Porwal <[e-mail]>
Gerrit-Attention: Dinesh Gehlot <[e-mail]>
Gerrit-Attention: Nick Vaccaro <[e-mail]>
Gerrit-MessageType: newchange