diff options
| author | Alexandru Gagniuc <alexandrux.gagniuc@intel.com> | 2016-05-02 13:40:15 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2016-05-04 20:08:03 +0200 |
| commit | 15f755bd01d6bf9692ed0395d1f813dca9fa2013 (patch) | |
| tree | 69177ac4f2271c5bb4e03820ecb0af0783f94d3b /src/soc/intel | |
| parent | 1116fa86e3f4f1cf50773ba0959d9d4f7a5ebb92 (diff) | |
| download | coreboot-15f755bd01d6bf9692ed0395d1f813dca9fa2013.tar.xz | |
soc/apollolake/romstage: Do not cast const to non-const pointers
That was a workaround for the MRC cache API, which has since been
reworked. The workaround is no longer needed.
Change-Id: I1c1883f3ea37245615248459cd993ed774bf92de
Signed-off-by: Alexandru Gagniuc <alexandrux.gagniuc@intel.com>
Reviewed-on: https://review.coreboot.org/14574
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins)
Diffstat (limited to 'src/soc/intel')
| -rw-r--r-- | src/soc/intel/apollolake/romstage.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/soc/intel/apollolake/romstage.c b/src/soc/intel/apollolake/romstage.c index 9dcb26a759..42af6c08f8 100644 --- a/src/soc/intel/apollolake/romstage.c +++ b/src/soc/intel/apollolake/romstage.c @@ -79,7 +79,8 @@ static void disable_watchdog(void) asmlinkage void car_stage_entry(void) { - void *hob_list_ptr, *mrc_data; + void *hob_list_ptr; + const void *mrc_data; struct range_entry fsp_mem, reg_car; struct postcar_frame pcf; size_t mrc_data_size; @@ -116,8 +117,7 @@ asmlinkage void car_stage_entry(void) /* Save MRC Data to CBMEM */ if (IS_ENABLED(CONFIG_CACHE_MRC_SETTINGS)) { - /* TODO: treat MRC data as const */ - mrc_data = (void*) fsp_find_nv_storage_data(&mrc_data_size); + mrc_data = fsp_find_nv_storage_data(&mrc_data_size); if (mrc_data && mrc_cache_stash_data(mrc_data, mrc_data_size) < 0) printk(BIOS_ERR, "Failed to stash MRC data\n"); } |