Felix Singer has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/48143 )
Change subject: mb/kontron/mal10: Use mainboard_ops driver for GPIO configuration ......................................................................
mb/kontron/mal10: Use mainboard_ops driver for GPIO configuration
`mainboard_silicon_init_params()` should *only* be used for configuring FSP options which can not be configured anywhere else. Therefore, use the init phase from the mainboard_ops driver for configuring the GPIOs.
Signed-off-by: Felix Singer felixsinger@posteo.net Change-Id: Ia01091938ac113cb5cf95f046609a1ebf3620806 --- M src/mainboard/kontron/mal10/ramstage.c 1 file changed, 9 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/43/48143/1
diff --git a/src/mainboard/kontron/mal10/ramstage.c b/src/mainboard/kontron/mal10/ramstage.c index 48194e6..87ead15 100644 --- a/src/mainboard/kontron/mal10/ramstage.c +++ b/src/mainboard/kontron/mal10/ramstage.c @@ -1,13 +1,17 @@ /* SPDX-License-Identifier: GPL-2.0-only */
+#include <device/device.h> #include <soc/ramstage.h> #include <carrier/gpio.h> #include <stddef.h>
-void mainboard_silicon_init_params(FSP_S_CONFIG *silconfig) +static void init_mainboard(void *chip_info) { carrier_gpio_configure(); +}
+void mainboard_silicon_init_params(FSP_S_CONFIG *silconfig) +{ /* * CPU Power Management Configuration correspond to the BIOS Setup menu settings * in the AMI UEFI v112. @@ -45,3 +49,7 @@ silconfig->IoApicDeviceNumber = 0x1F; silconfig->IoApicFunctionNumber = 0; } + +struct chip_operations mainboard_ops = { + .init = init_mainboard, +};