summaryrefslogtreecommitdiff
path: root/src/mainboard/google/veyron_mickey
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2017-12-05 13:39:10 -0800
committerJulius Werner <jwerner@chromium.org>2017-12-07 01:18:25 +0000
commite2f17f782f6a9236259f2c25ac30d35dc24d45bb (patch)
tree6c788bd8c2543c94abffa298952d974b38c2d0b2 /src/mainboard/google/veyron_mickey
parentec477346bfe8f73aac225e9a8c2c5e6438bba593 (diff)
downloadcoreboot-e2f17f782f6a9236259f2c25ac30d35dc24d45bb.tar.xz
boardid: Minor clean up and standardization
Merge the different coreboot table strapping ID structures into one because they're really just all the same, and I want to add more. Make the signature of the board_id() function return a uint32_t because that's also what goes in the coreboot table. Add a printk to the generic code handling strapping IDs in ramstage so that not every individual mainboard implementation needs its own print. (In turn, remove one such print from fsp1_1 code because it's in the way of my next patch.) Change-Id: Ib9563edf07b623a586a4dc168fe357564c5e68b5 Signed-off-by: Julius Werner <jwerner@chromium.org> Reviewed-on: https://review.coreboot.org/22741 Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/google/veyron_mickey')
-rw-r--r--src/mainboard/google/veyron_mickey/boardid.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mainboard/google/veyron_mickey/boardid.c b/src/mainboard/google/veyron_mickey/boardid.c
index 47e946e635..3833dbedfc 100644
--- a/src/mainboard/google/veyron_mickey/boardid.c
+++ b/src/mainboard/google/veyron_mickey/boardid.c
@@ -18,7 +18,7 @@
#include <gpio.h>
#include <stdlib.h>
-uint8_t board_id(void)
+uint32_t board_id(void)
{
static int id = -1;
gpio_t pins[] = {[3] = GPIO(2, A, 7), [2] = GPIO(2, A, 2),