summaryrefslogtreecommitdiffstats
path: root/arch/x86
diff options
context:
space:
mode:
authorBin Meng <bmeng.cn@gmail.com>2015-12-10 22:02:57 -0800
committerBin Meng <bmeng.cn@gmail.com>2016-01-13 12:20:13 +0800
commitf0285fbe4e868311f925aeae480796d4626e1728 (patch)
tree0205e923598b31e8b13753f0ff29506bd2512ca9 /arch/x86
parentaa02446406af194b7c79c2fae06073690e4fbba3 (diff)
downloadtalos-obmc-uboot-f0285fbe4e868311f925aeae480796d4626e1728.tar.gz
talos-obmc-uboot-f0285fbe4e868311f925aeae480796d4626e1728.zip
x86: fsp: Avoid cast stack_top in struct shared_data
Declare stack_top as u32 in struct shared_data and struct common_buf so that we can avoid casting in fsp_init(). Signed-off-by: Bin Meng <bmeng.cn@gmail.com> Acked-by: Simon Glass <sjg@chromium.org> Tested-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'arch/x86')
-rw-r--r--arch/x86/include/asm/fsp/fsp_api.h2
-rw-r--r--arch/x86/include/asm/fsp/fsp_support.h2
-rw-r--r--arch/x86/lib/fsp/fsp_support.c4
3 files changed, 4 insertions, 4 deletions
diff --git a/arch/x86/include/asm/fsp/fsp_api.h b/arch/x86/include/asm/fsp/fsp_api.h
index 2d34d138ab..db83e35e1d 100644
--- a/arch/x86/include/asm/fsp/fsp_api.h
+++ b/arch/x86/include/asm/fsp/fsp_api.h
@@ -30,7 +30,7 @@ struct common_buf {
* Stack top pointer used by the bootloader. The new stack frame will be
* set up at this location after FspInit API call.
*/
- u32 *stack_top;
+ u32 stack_top;
u32 boot_mode; /* Current system boot mode */
void *upd_data; /* User platform configuraiton data region */
u32 reserved[7]; /* Reserved */
diff --git a/arch/x86/include/asm/fsp/fsp_support.h b/arch/x86/include/asm/fsp/fsp_support.h
index f30d7b4854..685778e11e 100644
--- a/arch/x86/include/asm/fsp/fsp_support.h
+++ b/arch/x86/include/asm/fsp/fsp_support.h
@@ -20,7 +20,7 @@
struct shared_data {
struct fsp_header *fsp_hdr;
- u32 *stack_top;
+ u32 stack_top;
struct upd_region fsp_upd;
};
diff --git a/arch/x86/lib/fsp/fsp_support.c b/arch/x86/lib/fsp/fsp_support.c
index 0408b5d5cd..df62ba8097 100644
--- a/arch/x86/lib/fsp/fsp_support.c
+++ b/arch/x86/lib/fsp/fsp_support.c
@@ -122,7 +122,7 @@ void fsp_init(u32 stack_top, u32 boot_mode, void *nvs_buf)
memset(&rt_buf, 0, sizeof(struct fspinit_rtbuf));
/* Reserve a gap in stack top */
- rt_buf.common.stack_top = (u32 *)stack_top - 32;
+ rt_buf.common.stack_top = stack_top - 32;
rt_buf.common.boot_mode = boot_mode;
rt_buf.common.upd_data = fsp_upd;
@@ -152,7 +152,7 @@ void fsp_init(u32 stack_top, u32 boot_mode, void *nvs_buf)
params_ptr = &params;
shared_data.fsp_hdr = fsp_hdr;
- shared_data.stack_top = (u32 *)stack_top;
+ shared_data.stack_top = stack_top;
post_code(POST_PRE_MRC);
OpenPOWER on IntegriCloud