diff options
author | Aaron Durbin <adurbin@chromium.org> | 2018-01-24 11:59:09 -0700 |
---|---|---|
committer | Aaron Durbin <adurbin@chromium.org> | 2018-01-24 22:30:03 +0000 |
commit | 9cf2f707ab4f4aff0812949222fc41a57ae00dd3 (patch) | |
tree | 5b705558895b4eb1d9fb1d6b937df37dbb012f14 | |
parent | 8aa0792c0fe8bad058ce9b20fc690f223837b661 (diff) |
soc/amd/stoneyridge/spi: do not open code existing CAR APIs
The CAR APIs already exist to deal with proper type useage.
Don't open code things that already exist.
BUG=b:65485690
Change-Id: I09593401513f6060a30cf5c02c94d14afbe8f4fd
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/23412
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Justin TerAvest <teravest@chromium.org>
-rw-r--r-- | src/soc/amd/stoneyridge/spi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/amd/stoneyridge/spi.c b/src/soc/amd/stoneyridge/spi.c index cf8157cc42..311cd37483 100644 --- a/src/soc/amd/stoneyridge/spi.c +++ b/src/soc/amd/stoneyridge/spi.c @@ -34,12 +34,12 @@ static uintptr_t spibar CAR_GLOBAL; static uintptr_t get_spibase(void) { - return *(uintptr_t *)car_get_var_ptr(&spibar); + return car_get_var(spibar); } static void set_spibar(uintptr_t base) { - *(uintptr_t *)car_get_var_ptr(&spibar) = base; + car_set_var(spibar, base); } static inline uint8_t spi_read8(uint8_t reg) |