soc/intel/common/block/uart: Update the UART PCI device reference

This implementation revises the UART PCI device reference in common
UART driver. The SOC functions have been aligned to provide the UART
PCI device reference using pcidev_path_on_root.

The uart_get_device() return type is changed, and files in which
it gets used are updated.

Change-Id: Ie0fe5991f3b0b9c596c3de9472e98e4091d7dd87
Signed-off-by: Aamir Bohra <aamir.bohra@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34582
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>

1 files changed, 18 insertions, 11 deletions

diff --git a/src/soc/intel/common/block/uart/uart.c b/src/soc/intel/common/block/uart/uart.c
index 9d820ffd7ef2..f556aed3d6a8 100644
--- a/src/soc/intel/common/block/uart/uart.c
+++ b/src/soc/intel/common/block/uart/uart.c
@@ -65,15 +65,13 @@ static int uart_get_valid_index(void)
 	return UART_CONSOLE_INVALID_INDEX;
 }
 
-void uart_common_init(struct device *device, uintptr_t baseaddr)
+void uart_common_init(const struct device *device, uintptr_t baseaddr)
 {
 #if defined(__SIMPLE_DEVICE__)
-	pci_devfn_t dev = (pci_devfn_t)(uintptr_t)device;
+	pci_devfn_t dev = PCI_BDF(device);
 #else
-	struct device *dev = device;
+	const struct device *dev = device;
 #endif
-	if (!dev)
-		return;
 
 	/* Set UART base address */
 	pci_write_config32(dev, PCI_BASE_ADDRESS_0, baseaddr);
@@ -84,7 +82,7 @@ void uart_common_init(struct device *device, uintptr_t baseaddr)
 	uart_lpss_init(baseaddr);
 }
 
-struct device *uart_get_device(void)
+const struct device *uart_get_device(void)
 {
 	/*
 	 * This function will get called even if INTEL_LPSS_UART_FOR_CONSOLE
@@ -105,14 +103,16 @@ struct device *uart_get_device(void)
 bool uart_is_controller_initialized(void)
 {
 	uintptr_t base;
+	const struct device *dev_uart = uart_get_device();
+
+	if (!dev_uart)
+		return false;
 
 #if defined(__SIMPLE_DEVICE__)
-	pci_devfn_t dev = (pci_devfn_t)(uintptr_t)uart_get_device();
+	pci_devfn_t dev = PCI_BDF(dev_uart);
 #else
-	struct device *dev = uart_get_device();
+	const struct device *dev = dev_uart;
 #endif
-	if (!dev)
-		return false;
 
 	base = pci_read_config32(dev, PCI_BASE_ADDRESS_0) & ~0xFFF;
 	if (!base)
@@ -136,8 +136,15 @@ static void uart_configure_gpio_pads(void)
 
 void uart_bootblock_init(void)
 {
+	const struct device *dev_uart;
+
+	dev_uart = uart_get_device();
+
+	if (!dev_uart)
+		return;
+
 	/* Program UART BAR0, command, reset and clock register */
-	uart_common_init(uart_get_device(), CONFIG_CONSOLE_UART_BASE_ADDRESS);
+	uart_common_init(dev_uart, CONFIG_CONSOLE_UART_BASE_ADDRESS);
 
 	/* Configure the 2 pads per UART. */
 	uart_configure_gpio_pads();