From 1cdb6f2a1d6ddb1814719c77f539a061735f705a Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Fri, 17 Nov 2017 09:25:02 -0700 Subject: [PATCH] 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 Reviewed-on: https://review.coreboot.org/22515 Reviewed-by: Marc Jones Tested-by: build bot (Jenkins) --- .../baseboard/include/baseboard/variants.h | 2 +- .../google/kahlee/variants/baseboard/memory.c | 4 +-- .../kahlee/variants/kahlee/Makefile.inc | 2 -- .../google/kahlee/variants/kahlee/memory.c | 29 ------------------- 4 files changed, 3 insertions(+), 34 deletions(-) delete mode 100644 src/mainboard/google/kahlee/variants/kahlee/memory.c diff --git a/src/mainboard/google/kahlee/variants/baseboard/include/baseboard/variants.h b/src/mainboard/google/kahlee/variants/baseboard/include/baseboard/variants.h index b4e0891063..cf995fc59c 100644 --- a/src/mainboard/google/kahlee/variants/baseboard/include/baseboard/variants.h +++ b/src/mainboard/google/kahlee/variants/baseboard/include/baseboard/variants.h @@ -23,6 +23,6 @@ const GPIO_CONTROL *get_gpio_table(void); const struct sci_source *get_gpe_table(size_t *num); -size_t variant_board_id(void); +uint8_t variant_board_id(void); #endif /* __BASEBOARD_VARIANTS_H__ */ 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 #include -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)); } diff --git a/src/mainboard/google/kahlee/variants/kahlee/Makefile.inc b/src/mainboard/google/kahlee/variants/kahlee/Makefile.inc index 94f388912c..c5aab8ea27 100644 --- a/src/mainboard/google/kahlee/variants/kahlee/Makefile.inc +++ b/src/mainboard/google/kahlee/variants/kahlee/Makefile.inc @@ -16,7 +16,5 @@ bootblock-y += gpio.c romstage-y += gpio.c -romstage-y += memory.c ramstage-y += gpio.c -ramstage-y += memory.c diff --git a/src/mainboard/google/kahlee/variants/kahlee/memory.c b/src/mainboard/google/kahlee/variants/kahlee/memory.c deleted file mode 100644 index 4341b824b5..0000000000 --- a/src/mainboard/google/kahlee/variants/kahlee/memory.c +++ /dev/null @@ -1,29 +0,0 @@ -/* - * This file is part of the coreboot project. - * - * Copyright 2017 Google Inc. - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; version 2 of the License. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ - -#include /* src/include/gpio.h */ -#include -#include - -size_t variant_board_id(void) -{ - gpio_t pads[] = { - [2] = MEM_CONFIG2, - [1] = MEM_CONFIG1, - [0] = MEM_CONFIG0, - }; - - return gpio_pullup_base2_value(pads, ARRAY_SIZE(pads)); -}