soc/intel/uart: Drop SoC callback `soc_uart_console_to_device`

This change renames `struct uart_gpio_pad_config` to `struct
uart_controller_config` and adds a new parameter devfn (which expects
devfn for the UART controller corresponding to the index in
PCI_DEVFN() format). This gets rid of the SoC callback to get `struct
device` pointer to the UART controller device.

Change-Id: Id0712a0038f2cc1a61b8b5a58fa155f14e7949a5
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49212
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

2 files changed, 7 insertions, 22 deletions

diff --git a/src/soc/intel/common/block/include/intelblocks/uart.h b/src/soc/intel/common/block/include/intelblocks/uart.h
index fd65f4c85e..c3a22bc96b 100644
--- a/src/soc/intel/common/block/include/intelblocks/uart.h
+++ b/src/soc/intel/common/block/include/intelblocks/uart.h
@@ -10,8 +10,10 @@
 
 #define MAX_GPIO_PAD_PER_UART		2
 
-struct uart_gpio_pad_config {
+struct uart_controller_config {
 	int console_index;
+	/* devfn in PCI_DEVFN() format */
+	unsigned int devfn;
 	struct pad_config gpios[MAX_GPIO_PAD_PER_UART];
 };
 
@@ -55,21 +57,4 @@ void uart_bootblock_init(void);
  */
 const struct device *uart_get_device(void);
 
-/**************************** SoC callbacks ***********************************/
-
-/*
- * SoC should implement soc_uart_console_to_device() function to
- * get UART debug controller device structure based on console number
- * Caller needs to check proper UART console index supported by SoC.
- * If wrong UART console index is passed to function, it'll return NULL.
- *
- * Input:
- * UART console index selected in config
- *
- * Returns:
- * Pointer to device structure = If device has a UART debug controller.
- * NULL = otherwise
- */
-DEVTREE_CONST struct device *soc_uart_console_to_device(int uart_console);
-
 #endif	/* SOC_INTEL_COMMON_BLOCK_UART_H */
diff --git a/src/soc/intel/common/block/uart/uart.c b/src/soc/intel/common/block/uart/uart.c
index 1e205ef07f..559ba6da23 100644
--- a/src/soc/intel/common/block/uart/uart.c
+++ b/src/soc/intel/common/block/uart/uart.c
@@ -20,7 +20,7 @@
 #define UART_PCI_ENABLE	(PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER)
 #define UART_CONSOLE_INVALID_INDEX	0xFF
 
-extern const struct uart_gpio_pad_config uart_gpio_pads[];
+extern const struct uart_controller_config uart_ctrlr_config[];
 extern const int uart_max_index;
 
 static void uart_lpss_init(const struct device *dev, uintptr_t baseaddr)
@@ -50,7 +50,7 @@ static int uart_get_valid_index(void)
 	int index;
 
 	for (index = 0; index < uart_max_index; index++) {
-		if (uart_gpio_pads[index].console_index ==
+		if (uart_ctrlr_config[index].console_index ==
 				CONFIG_UART_FOR_CONSOLE)
 			return index;
 	}
@@ -88,7 +88,7 @@ const struct device *uart_get_device(void)
 	int console_index = uart_get_valid_index();
 
 	if (console_index != UART_CONSOLE_INVALID_INDEX)
-		return soc_uart_console_to_device(CONFIG_UART_FOR_CONSOLE);
+		return pcidev_path_on_root(uart_ctrlr_config[console_index].devfn);
 	else
 		return NULL;
 }
@@ -123,7 +123,7 @@ static void uart_configure_gpio_pads(void)
 	int index = uart_get_valid_index();
 
 	if (index != UART_CONSOLE_INVALID_INDEX)
-		gpio_configure_pads(uart_gpio_pads[index].gpios,
+		gpio_configure_pads(uart_ctrlr_config[index].gpios,
 				MAX_GPIO_PAD_PER_UART);
 }