soc/intel/*: drop UART pad configuration from common code
UART pad configuration should not be done in common code, because that
may cause short circuits, when the user sets a wrong UART index.
Since all boards do pad setup on their own now, finally drop the pad
configuration from SoC common code.
Change-Id: Id03719eb8bd0414083148471ed05dea62a895126
Signed-off-by: Michael Niewöhner <foss@mniewoehner.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/48829
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Lance Zhao
diff --git a/src/soc/intel/common/block/include/intelblocks/uart.h b/src/soc/intel/common/block/include/intelblocks/uart.h
index c3a22bc..f140fc8 100644
--- a/src/soc/intel/common/block/include/intelblocks/uart.h
+++ b/src/soc/intel/common/block/include/intelblocks/uart.h
@@ -5,24 +5,13 @@
#include <console/uart.h>
#include <device/device.h>
-#include <intelblocks/gpio.h>
#include <stdint.h>
-#define MAX_GPIO_PAD_PER_UART 2
-
-struct uart_controller_config {
- int console_index;
- /* devfn in PCI_DEVFN() format */
- unsigned int devfn;
- struct pad_config gpios[MAX_GPIO_PAD_PER_UART];
-};
-
/*
* While using this common UART block for any SOC following is expected from soc
* 1. SOC will define proper UART_BASE which is base address for UART console.
* 2. SOC will return correct device pointer based on console index
- * 3. SOC will provide appropriate GPIO pad configuration for UART console
- * 4. SOC will allow common code to set UART into legacy mode if supported.
+ * 3. SOC will allow common code to set UART into legacy mode if supported.
*/
/*
diff --git a/src/soc/intel/common/block/uart/Kconfig b/src/soc/intel/common/block/uart/Kconfig
index 3437ec7..7ba22b5 100644
--- a/src/soc/intel/common/block/uart/Kconfig
+++ b/src/soc/intel/common/block/uart/Kconfig
@@ -13,7 +13,5 @@
help
Selected by mainboards that use one of the SoC's LPSS UARTS
for the coreboot console.
- WARNING: UART_FOR_CONSOLE has to be set to a correct value,
- otherwise wrong pad configurations might be selected.
endif
diff --git a/src/soc/intel/common/block/uart/uart.c b/src/soc/intel/common/block/uart/uart.c
index 4a84935..9b34031 100644
--- a/src/soc/intel/common/block/uart/uart.c
+++ b/src/soc/intel/common/block/uart/uart.c
@@ -18,10 +18,9 @@
#include "chip.h"
#define UART_PCI_ENABLE (PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER)
-#define UART_CONSOLE_INVALID_INDEX 0xFF
-extern const struct uart_controller_config uart_ctrlr_config[];
-extern const int uart_ctrlr_config_size;
+extern const unsigned int uart_devices[];
+extern const int uart_devices_size;
static void uart_lpss_init(pci_devfn_t dev, uintptr_t baseaddr)
{
@@ -45,23 +44,9 @@
}
#endif
-static int uart_get_valid_index(void)
-{
- int index;
-
- for (index = 0; index < uart_ctrlr_config_size; index++) {
- if (uart_ctrlr_config[index].console_index ==
- CONFIG_UART_FOR_CONSOLE)
- return index;
- }
- /* For valid index, code should not reach here */
- return UART_CONSOLE_INVALID_INDEX;
-}
-
static pci_devfn_t uart_console_get_pci_bdf(void)
{
int devfn;
- int index;
/*
* This function will get called even if INTEL_LPSS_UART_FOR_CONSOLE
@@ -71,11 +56,13 @@
if (!CONFIG(INTEL_LPSS_UART_FOR_CONSOLE))
return PCI_DEV_INVALID;
- index = uart_get_valid_index();
- if (index == UART_CONSOLE_INVALID_INDEX)
+ if (CONFIG_UART_FOR_CONSOLE > uart_devices_size)
return PCI_DEV_INVALID;
- devfn = uart_ctrlr_config[index].devfn;
+ devfn = uart_devices[CONFIG_UART_FOR_CONSOLE];
+ if (devfn == PCI_DEVFN_INVALID)
+ return PCI_DEV_INVALID;
+
return PCI_DEV(0, PCI_SLOT(devfn), PCI_FUNC(devfn));
}
@@ -107,15 +94,6 @@
return !lpss_is_controller_in_reset(base);
}
-static void uart_configure_gpio_pads(void)
-{
- int index = uart_get_valid_index();
-
- if (index != UART_CONSOLE_INVALID_INDEX)
- gpio_configure_pads(uart_ctrlr_config[index].gpios,
- MAX_GPIO_PAD_PER_UART);
-}
-
void uart_bootblock_init(void)
{
const uint32_t baseaddr = CONFIG_CONSOLE_UART_BASE_ADDRESS;
@@ -131,9 +109,6 @@
pci_s_write_config16(dev, PCI_COMMAND, UART_PCI_ENABLE);
uart_lpss_init(dev, baseaddr);
-
- /* Configure the 2 pads per UART. */
- uart_configure_gpio_pads();
}
#if ENV_RAMSTAGE