GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/amd/stoneyridge: Get rid of void pointer math 

Pointer math with void pointers is illegal in many compilers, though it
works with GCC because it assumes size of void to be 1. Change the pointers
or add parenthesis to force a proper order that will not cause compile
errors if compiled with a different compiler, and more importantly, don't
have unsuspected side effects.

BUG=b:118484178
TEST=Build and boot grunt.

Change-Id: Ibfeb83893f09cb897d459856aff2a4ab2a74e6e5
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/29337
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
This commit is contained in:
Richard Spiegel 2018-10-29 08:01:53 -07:00 committed by Patrick Georgi
parent 7ca21c1024
commit 9f25e9d480
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/soc/amd/stoneyridge/southbridge.c
View File

@ -429,27 +429,27 @@ static uintptr_t sb_spibase(void)
void sb_set_spi100(u16 norm, u16 fast, u16 alt, u16 tpm) void sb_set_spi100(u16 norm, u16 fast, u16 alt, u16 tpm)
{ {
uintptr_t base = sb_spibase(); uintptr_t base = sb_spibase();
write16((void *)base + SPI100_SPEED_CONFIG, write16((void *)(base + SPI100_SPEED_CONFIG),
(norm << SPI_NORM_SPEED_NEW_SH) | (norm << SPI_NORM_SPEED_NEW_SH) |
(fast << SPI_FAST_SPEED_NEW_SH) | (fast << SPI_FAST_SPEED_NEW_SH) |
(alt << SPI_ALT_SPEED_NEW_SH) | (alt << SPI_ALT_SPEED_NEW_SH) |
(tpm << SPI_TPM_SPEED_NEW_SH)); (tpm << SPI_TPM_SPEED_NEW_SH));
write16((void *)base + SPI100_ENABLE, SPI_USE_SPI100); write16((void *)(base + SPI100_ENABLE), SPI_USE_SPI100);
} }
void sb_disable_4dw_burst(void) void sb_disable_4dw_burst(void)
{ {
uintptr_t base = sb_spibase(); uintptr_t base = sb_spibase();
write16((void *)base + SPI100_HOST_PREF_CONFIG, write16((void *)(base + SPI100_HOST_PREF_CONFIG),
read16((void *)base + SPI100_HOST_PREF_CONFIG) read16((void *)(base + SPI100_HOST_PREF_CONFIG))
& ~SPI_RD4DW_EN_HOST); & ~SPI_RD4DW_EN_HOST);
} }
void sb_read_mode(u32 mode) void sb_read_mode(u32 mode)
{ {
uintptr_t base = sb_spibase(); uintptr_t base = sb_spibase();
write32((void *)base + SPI_CNTRL0, write32((void *)(base + SPI_CNTRL0),
(read32((void *)base + SPI_CNTRL0) (read32((void *)(base + SPI_CNTRL0))
& ~SPI_READ_MODE_MASK) | mode); & ~SPI_READ_MODE_MASK) | mode);
} }