diff options
author | Aaron Durbin <adurbin@chromium.org> | 2015-09-30 09:12:57 -0500 |
---|---|---|
committer | Aaron Durbin <adurbin@gmail.com> | 2015-10-11 23:55:41 +0000 |
commit | cc5ac17fab97bd16f3122bb492fbdc28644c8567 (patch) | |
tree | 3c1b0a9c0d8155b06e0556567905d52d9562a800 /src/soc/intel/common/romstage.c | |
parent | 3c4053fa59a8654b2f10cf175915914c37da9daf (diff) | |
download | coreboot-cc5ac17fab97bd16f3122bb492fbdc28644c8567.tar.xz |
soc/intel/common: remove chipset specific calls
The report_platform_info() and set_max_freq() are not being
used similarly on skylake and braswell. With the addition
of other SoCs I suspect a similar pattern will emerge. Instead
of having weak functions to ensure things link with the hardcoded
policy push these calls into their respective SoC homes.
For parity, both skylake and braswell were updated to be consistent
with the same calls prior to this patch.
BUG=chrome-os-partner:44827
BRANCH=None
TEST=Built and booted glados. Built braswell.
Original-Change-Id: I3371d09aff0629503254296955fef28d35754a38
Original-Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Original-Reviewed-on: https://chromium-review.googlesource.com/303334
Original-Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Change-Id: I2de33632ed127cac52d7075cbad95cd6387a1b46
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/11815
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'src/soc/intel/common/romstage.c')
-rw-r--r-- | src/soc/intel/common/romstage.c | 18 |
1 files changed, 0 insertions, 18 deletions
diff --git a/src/soc/intel/common/romstage.c b/src/soc/intel/common/romstage.c index e1095b2f09..42e624cab9 100644 --- a/src/soc/intel/common/romstage.c +++ b/src/soc/intel/common/romstage.c @@ -103,12 +103,6 @@ asmlinkage void *romstage_main(struct cache_as_ram_params *car_params) /* Get power state */ params.power_state = fill_power_state(); - /* Print useful platform information */ - report_platform_info(); - - /* Set CPU frequency to maximum */ - set_max_freq(); - /* Perform SOC specific initialization. */ soc_romstage_init(¶ms); @@ -435,12 +429,6 @@ __attribute__((weak)) void report_memory_config(void) printk(BIOS_DEBUG, "WEAK: %s/%s called\n", __FILE__, __func__); } -/* Display the platform configuration */ -__attribute__((weak)) void report_platform_info(void) -{ - printk(BIOS_DEBUG, "WEAK: %s/%s called\n", __FILE__, __func__); -} - /* Choose top of stack and setup MTRRs */ __attribute__((weak)) void *setup_stack_and_mtrrs(void) { @@ -449,12 +437,6 @@ __attribute__((weak)) void *setup_stack_and_mtrrs(void) return NULL; } -/* Speed up the CPU to the maximum frequency */ -__attribute__((weak)) void set_max_freq(void) -{ - printk(BIOS_DEBUG, "WEAK: %s/%s called\n", __FILE__, __func__); -} - /* SOC initialization after RAM is enabled */ __attribute__((weak)) void soc_after_ram_init(struct romstage_params *params) { |