mb/intel/harcuvar: Fix board_id() return type

The weak definition of board_id() in coreboot_table.c returns a
uint32_t, so update this function to match. This fixes a compiler error
when using LTO.

Change-Id: I6ad03ecedcf4a4d9f0c917cdc760f81ddde06d11
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39015
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: David Guckian
Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Jacob Garber 2020-02-18 23:29:21 -07:00 committed by Angel Pons
parent 9e3e49234d
commit 069cd67854
4 changed files with 4 additions and 4 deletions

View file

@ -5,7 +5,7 @@
#include "harcuvar_boardid.h" #include "harcuvar_boardid.h"
uint8_t board_id(void) uint32_t board_id(void)
{ {
int id = BoardIdHarcuvar; int id = BoardIdHarcuvar;

View file

@ -8,6 +8,6 @@
#define BoardIdHarcuvar 0x52 #define BoardIdHarcuvar 0x52
uint8_t board_id(void); uint32_t board_id(void);
#endif /* MAINBOARD_BOARD_H */ #endif /* MAINBOARD_BOARD_H */

View file

@ -7,7 +7,7 @@
size_t mainboard_get_hsio_config(BL_HSIO_INFORMATION **p_hsio_config) size_t mainboard_get_hsio_config(BL_HSIO_INFORMATION **p_hsio_config)
{ {
uint8_t boardid = board_id(); uint32_t boardid = board_id();
size_t num; size_t num;
switch (boardid) { switch (boardid) {
case BoardIdHarcuvar: case BoardIdHarcuvar:

View file

@ -88,7 +88,7 @@ void mainboard_config_gpios(void)
{ {
size_t num; size_t num;
const struct dnv_pad_config *table; const struct dnv_pad_config *table;
uint8_t boardid = board_id(); uint32_t boardid = board_id();
/* Configure pads prior to SiliconInit() in case there's any /* Configure pads prior to SiliconInit() in case there's any
* dependencies during hardware initialization. * dependencies during hardware initialization.