diff options
author | Martin Roth <martinroth@google.com> | 2017-11-17 09:25:02 -0700 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2017-11-19 01:43:04 +0000 |
commit | 1cdb6f2a1d6ddb1814719c77f539a061735f705a (patch) | |
tree | daf00f4b73f0fbbdab8b399d54a3f6dbaf20ad16 /src/mainboard/google/kahlee/variants/baseboard/memory.c | |
parent | c09c2a4883a1278af0f758d73f4705a939197a68 (diff) |
mainboard/google/kahlee: Update memory.c
This fixes some issues with the initial implementation that was copied
from reef.
- The board ID value shouldn't be size_t - it's not a size.
- Kahlee doesn't even need the memory.c file - it uses an SoDIMM.
BUG=b:68293392
TEST=build stoney platforms, boot kahleebo
Change-Id: Ife5660d36912e887edfd0365a9f16c5a172c9c86
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/22515
Reviewed-by: Marc Jones <marc@marcjonesconsulting.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/google/kahlee/variants/baseboard/memory.c')
-rw-r--r-- | src/mainboard/google/kahlee/variants/baseboard/memory.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/mainboard/google/kahlee/variants/baseboard/memory.c b/src/mainboard/google/kahlee/variants/baseboard/memory.c index 511241f603..ae8734e9d9 100644 --- a/src/mainboard/google/kahlee/variants/baseboard/memory.c +++ b/src/mainboard/google/kahlee/variants/baseboard/memory.c @@ -17,7 +17,7 @@ #include <baseboard/variants.h> #include <variant/gpio.h> -size_t __attribute__((weak)) variant_board_id(void) +uint8_t __attribute__((weak)) variant_board_id(void) { gpio_t pads[] = { [3] = MEM_CONFIG3, @@ -26,5 +26,5 @@ size_t __attribute__((weak)) variant_board_id(void) [0] = MEM_CONFIG0, }; - return gpio_pullup_base2_value(pads, ARRAY_SIZE(pads)); + return gpio_base2_value(pads, ARRAY_SIZE(pads)); } |