diff options
author | Aaron Durbin <adurbin@chromium.org> | 2016-07-16 17:29:47 -0500 |
---|---|---|
committer | Aaron Durbin <adurbin@chromium.org> | 2016-07-19 20:14:10 +0200 |
commit | a3a06aeac5879f36f35bb462616e29fbe4a177f9 (patch) | |
tree | b8584ba34d401bb5ee0d6df500564e630301e2eb /src/soc/imgtec/pistachio/cbmem.c | |
parent | ddbdcc3898f031cc323a31eddd48671c3dbe5f11 (diff) |
drivers/intel/fsp2_0: fix hand-off-block types and size
The gcc compiler treats sizeof(void) == 1. Therefore requesting
a 1 byte reservation in cbmem and writing a pointer into the
buffer returned is wrong. Fix the size of the request to be
32-bits because FSP 2.0 is in 32-bit space by definition. Also,
since the access to the field happens across stage boundaries
it's important to ensure fixed widths are used in case a later
stage has a different pointer bit width.
BUG=chrome-os-partner:52679
Change-Id: Ib4efc7d5369d44a995318aac6c4a7cfdc73e4a8c
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/15737
Reviewed-by: Furquan Shaikh <furquan@google.com>
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/soc/imgtec/pistachio/cbmem.c')
0 files changed, 0 insertions, 0 deletions