diff options
| author | Felix Held <felix.held@amd.corp-partner.google.com> | 2020-06-16 15:35:20 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2020-06-17 18:52:07 +0000 |
| commit | 097e44901351f48972dbacd83e91ded4cb2efa33 (patch) | |
| tree | dbc4d20ef938ba9846cb18b9a8c04365df6b7793 /src/soc | |
| parent | ca481ee87f3d10d2b2ca6de8151a79850f059585 (diff) | |
soc/amd/picasso: rename PICASSO_UART Kconfig option
The PICASSO_UART Kconfig option is about using the internal MMIO UART
controllers in Picasso for console, so rename it to PICASSO_CONSOLE_UART
Change-Id: I38ac9ee96af826fe49307b4d0e055a43fcbd4334
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42429
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Diffstat (limited to 'src/soc')
| -rw-r--r-- | src/soc/amd/picasso/Kconfig | 10 | ||||
| -rw-r--r-- | src/soc/amd/picasso/aoac.c | 4 | ||||
| -rw-r--r-- | src/soc/amd/picasso/southbridge.c | 2 | ||||
| -rw-r--r-- | src/soc/amd/picasso/uart.c | 6 |
4 files changed, 11 insertions, 11 deletions
diff --git a/src/soc/amd/picasso/Kconfig b/src/soc/amd/picasso/Kconfig index e23cfd8f87..dd5731df05 100644 --- a/src/soc/amd/picasso/Kconfig +++ b/src/soc/amd/picasso/Kconfig @@ -254,8 +254,8 @@ config PICASSO_ACPI_IO_BASE help Base address for the ACPI registers. -config PICASSO_UART - bool "UART controller on Picasso" +config PICASSO_CONSOLE_UART + bool "Use Picasso UART controller for console" default n select DRIVERS_UART_8250MEM select DRIVERS_UART_8250MEM_32 @@ -270,7 +270,7 @@ config PICASSO_UART choice PICASSO_UART_CLOCK_SOURCE prompt "UART Frequency" - depends on PICASSO_UART + depends on PICASSO_CONSOLE_UART default PICASSO_UART_48MZ config PICASSO_UART_48MZ @@ -288,7 +288,7 @@ endchoice config PICASSO_UART_LEGACY bool "Decode legacy I/O range" - depends on PICASSO_UART + depends on PICASSO_CONSOLE_UART # TODO: shouldn't depend on this help Assign I/O 3F8, 2F8, etc. to a Picasso UART. Only a single UART may decode legacy addresses and this option enables the one used for the @@ -296,7 +296,7 @@ config PICASSO_UART_LEGACY of MMIO. The MMIO decode is still present when this option is used. config CONSOLE_UART_BASE_ADDRESS - depends on CONSOLE_SERIAL && PICASSO_UART + depends on CONSOLE_SERIAL && PICASSO_CONSOLE_UART hex default 0xfedc9000 if UART_FOR_CONSOLE = 0 default 0xfedca000 if UART_FOR_CONSOLE = 1 diff --git a/src/soc/amd/picasso/aoac.c b/src/soc/amd/picasso/aoac.c index 2bf99845c9..f31afbf808 100644 --- a/src/soc/amd/picasso/aoac.c +++ b/src/soc/amd/picasso/aoac.c @@ -76,13 +76,13 @@ void enable_aoac_devices(void) for (i = 0; i < ARRAY_SIZE(aoac_devs); i++) power_on_aoac_device(aoac_devs[i]); - if (CONFIG(PICASSO_UART)) + if (CONFIG(PICASSO_CONSOLE_UART)) power_on_aoac_device(FCH_AOAC_UART_FOR_CONSOLE); /* Wait for AOAC devices to indicate power and clock OK */ for (i = 0; i < ARRAY_SIZE(aoac_devs); i++) wait_for_aoac_enabled(aoac_devs[i]); - if (CONFIG(PICASSO_UART)) + if (CONFIG(PICASSO_CONSOLE_UART)) wait_for_aoac_enabled(FCH_AOAC_UART_FOR_CONSOLE); } diff --git a/src/soc/amd/picasso/southbridge.c b/src/soc/amd/picasso/southbridge.c index 54128abe5d..0ff56066e4 100644 --- a/src/soc/amd/picasso/southbridge.c +++ b/src/soc/amd/picasso/southbridge.c @@ -152,7 +152,7 @@ void fch_pre_init(void) */ clear_uart_legacy_config(); - if (CONFIG(PICASSO_UART)) + if (CONFIG(PICASSO_CONSOLE_UART)) set_uart_config(CONFIG_UART_FOR_CONSOLE); } diff --git a/src/soc/amd/picasso/uart.c b/src/soc/amd/picasso/uart.c index f523bce162..0d97fd701e 100644 --- a/src/soc/amd/picasso/uart.c +++ b/src/soc/amd/picasso/uart.c @@ -31,14 +31,14 @@ static const struct _uart_info { }; /* - * Don't provide uart_platform_base and uart_platform_refclk functions if PICASSO_UART + * Don't provide uart_platform_base and uart_platform_refclk functions if PICASSO_CONSOLE_UART * isn't selected. Those two functions are used by the console UART driver and need to be * provided exactly once and only by the UART that is used for console. * * TODO: Replace the #if block by factoring out the two functions into a different compilation * unit. */ -#if CONFIG(PICASSO_UART) +#if CONFIG(PICASSO_CONSOLE_UART) uintptr_t uart_platform_base(int idx) { @@ -53,7 +53,7 @@ unsigned int uart_platform_refclk(void) return CONFIG(PICASSO_UART_48MZ) ? 48000000 : 115200 * 16; } -#endif /* PICASSO_UART */ +#endif /* PICASSO_CONSOLE_UART */ void clear_uart_legacy_config(void) { |