diff options
author | Angel Pons <th3fanbus@gmail.com> | 2021-06-23 14:29:12 +0200 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2021-08-27 16:04:33 +0000 |
commit | 2d45322639b4bc2796040c7b7417b7a2a45d700b (patch) | |
tree | ac7fae631651b377e4061dc53609a17022fa7fd6 | |
parent | 2bc7a6b3d2c68fd820a9390efedc44a3d632ab26 (diff) |
broadwell: Drop weak `mainboard_fill_spd_data` definition
Make `mainboard_fill_spd_data` mandatory and adapt mainboards to define
this function accordingly.
Change-Id: Ic18c4c574e8c963bbb41c980f43bdbacc57735af
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55806
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
4 files changed, 16 insertions, 10 deletions
diff --git a/src/mainboard/intel/wtm2/pei_data.c b/src/mainboard/intel/wtm2/pei_data.c index 5591c4ca4d..9df5081f23 100644 --- a/src/mainboard/intel/wtm2/pei_data.c +++ b/src/mainboard/intel/wtm2/pei_data.c @@ -3,7 +3,7 @@ #include <soc/pei_data.h> #include <soc/pei_wrapper.h> -void mainboard_fill_pei_data(struct pei_data *pei_data) +void mainboard_fill_spd_data(struct pei_data *pei_data) { /* One installed DIMM per channel */ pei_data->dimm_channel0_disabled = 2; @@ -11,7 +11,10 @@ void mainboard_fill_pei_data(struct pei_data *pei_data) pei_data->spd_addresses[0] = 0xa2; pei_data->spd_addresses[2] = 0xa2; +} +void mainboard_fill_pei_data(struct pei_data *pei_data) +{ pei_data_usb2_port(pei_data, 0, 0x40, 1, USB_OC_PIN_SKIP, USB_PORT_FRONT_PANEL); pei_data_usb2_port(pei_data, 1, 0x40, 1, USB_OC_PIN_SKIP, diff --git a/src/mainboard/purism/librem_bdw/variants/librem13v1/pei_data.c b/src/mainboard/purism/librem_bdw/variants/librem13v1/pei_data.c index 1e5d9d9a1d..93ad4dc34e 100644 --- a/src/mainboard/purism/librem_bdw/variants/librem13v1/pei_data.c +++ b/src/mainboard/purism/librem_bdw/variants/librem13v1/pei_data.c @@ -3,13 +3,16 @@ #include <soc/pei_data.h> #include <soc/pei_wrapper.h> -void mainboard_fill_pei_data(struct pei_data *pei_data) +void mainboard_fill_spd_data(struct pei_data *pei_data) { - pei_data->ec_present = 1; - /* One DIMM slot */ pei_data->dimm_channel1_disabled = 3; pei_data->spd_addresses[0] = 0xa0; +} + +void mainboard_fill_pei_data(struct pei_data *pei_data) +{ + pei_data->ec_present = 1; /* P1: Left Side Port (USB2 only) */ pei_data_usb2_port(pei_data, 0, 0x0080, 1, USB_OC_PIN_SKIP, diff --git a/src/mainboard/purism/librem_bdw/variants/librem15v2/pei_data.c b/src/mainboard/purism/librem_bdw/variants/librem15v2/pei_data.c index c3580b3f8a..025591ab69 100644 --- a/src/mainboard/purism/librem_bdw/variants/librem15v2/pei_data.c +++ b/src/mainboard/purism/librem_bdw/variants/librem15v2/pei_data.c @@ -3,9 +3,8 @@ #include <soc/pei_data.h> #include <soc/pei_wrapper.h> -void mainboard_fill_pei_data(struct pei_data *pei_data) +void mainboard_fill_spd_data(struct pei_data *pei_data) { - pei_data->ec_present = 1; pei_data->dq_pins_interleaved = 1; /* One DIMM slot */ @@ -14,6 +13,11 @@ void mainboard_fill_pei_data(struct pei_data *pei_data) pei_data->spd_addresses[0] = 0xa0; pei_data->spd_addresses[2] = 0xa4; +} + +void mainboard_fill_pei_data(struct pei_data *pei_data) +{ + pei_data->ec_present = 1; /* P1: Right Side Port (USB2) */ pei_data_usb2_port(pei_data, 0, 0x0080, 1, USB_OC_PIN_SKIP, diff --git a/src/soc/intel/broadwell/raminit.c b/src/soc/intel/broadwell/raminit.c index 6254071ac8..28b3f37ed9 100644 --- a/src/soc/intel/broadwell/raminit.c +++ b/src/soc/intel/broadwell/raminit.c @@ -177,10 +177,6 @@ static void setup_sdram_meminfo(struct pei_data *pei_data) } } -__weak void mainboard_fill_spd_data(struct pei_data *pei_data) -{ -} - void perform_raminit(const struct chipset_power_state *const power_state) { const int s3resume = power_state->prev_sleep_state == ACPI_S3; |