| commit | bd48b235fce91c53267063d00b912f621052ce6a | |
| author | Richard Spiegel <richard.spiegel@amd.corp-partner.google.com> | |
| Fri, 2 Nov 2018 15:25:00 +0000 (2 08:25 -0700) | ||
| committer | Patrick Georgi <pgeorgi@google.com> | |
| Wed, 7 Nov 2018 16:43:15 +0000 (7 16:43 +0000) | ||
| tree | 923aa46894bb32453eef04cf606730d4b7ee5884 | treesnapshot (tar.gz zip) |
| parent | 69b18f0b6826ecfd794594b4ae5ebf299e9378ba | commitdiff |
southbridge/amd/pi/hudson: 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 AMD Bettong.
Change-Id: I4167c7eeb9339937b064e81e615ceb65f4689082
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/29438
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
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 AMD Bettong.
Change-Id: I4167c7eeb9339937b064e81e615ceb65f4689082
Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com>
Reviewed-on: https://review.coreboot.org/29438
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
| src/southbridge/amd/pi/hudson/early_setup.c | diffblobblamehistory |