summaryrefslogtreecommitdiff
path: root/src/superio
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2020-10-17 15:06:40 +0200
committerPatrick Georgi <pgeorgi@google.com>2020-10-19 07:06:20 +0000
commit00e58c35c62dbd6c2b20980d594b2b9d1153aa45 (patch)
treecbea4b4621ba6454971ea496aea4cdcc1b44c0dc /src/superio
parent9357ac38606a2b928692a1251b7c9f4aef289334 (diff)
superio/nuvoton: Factor out equivalent Kconfig option
There's no need to have multiple Kconfig symbols which do the same thing. Introduce `SUPERIO_NUVOTON_COMMON_COM_A` and update boards to use the new symbol. To preserve alphabetical order in mainboard Kconfig, place the new symbol above the Super I/O symbol (instead of below). Change-Id: Ic0a30b3177a1a535261525638be301ae07c59c14 Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/46522 Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Michael Niewöhner <foss@mniewoehner.de> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/superio')
-rw-r--r--src/superio/nuvoton/common/Kconfig7
-rw-r--r--src/superio/nuvoton/common/early_serial.c4
-rw-r--r--src/superio/nuvoton/nct5539d/Kconfig5
-rw-r--r--src/superio/nuvoton/nct6776/Kconfig5
-rw-r--r--src/superio/nuvoton/nct6791d/Kconfig5
5 files changed, 8 insertions, 18 deletions
diff --git a/src/superio/nuvoton/common/Kconfig b/src/superio/nuvoton/common/Kconfig
index 413b6a6882..78523c1b7e 100644
--- a/src/superio/nuvoton/common/Kconfig
+++ b/src/superio/nuvoton/common/Kconfig
@@ -5,6 +5,13 @@
config SUPERIO_NUVOTON_COMMON_PRE_RAM
bool
+config SUPERIO_NUVOTON_COMMON_COM_A
+ bool
+ depends on SUPERIO_NUVOTON_COMMON_PRE_RAM
+ help
+ When enabled, the 'nuvoton_enable_serial()' function will unset
+ global CR 0x2a bit 7 to route COM A to the GPIO8 pin group.
+
# Generic Nuvoton HWM driver
config SUPERIO_NUVOTON_COMMON_HWM
bool
diff --git a/src/superio/nuvoton/common/early_serial.c b/src/superio/nuvoton/common/early_serial.c
index 398678983a..f86e7a27d0 100644
--- a/src/superio/nuvoton/common/early_serial.c
+++ b/src/superio/nuvoton/common/early_serial.c
@@ -52,9 +52,7 @@ void nuvoton_enable_serial(pnp_devfn_t dev, u16 iobase)
nuvoton_pnp_enter_conf_state(dev);
- if (CONFIG(SUPERIO_NUVOTON_NCT5539D_COM_A) ||
- CONFIG(SUPERIO_NUVOTON_NCT6791D_COM_A) ||
- CONFIG(SUPERIO_NUVOTON_NCT6776_COM_A))
+ if (CONFIG(SUPERIO_NUVOTON_COMMON_COM_A))
/* Route COM A to GPIO8 pin group */
pnp_unset_and_set_config(dev, 0x2a, 1 << 7, 0);
diff --git a/src/superio/nuvoton/nct5539d/Kconfig b/src/superio/nuvoton/nct5539d/Kconfig
index 440775f13e..dc48246668 100644
--- a/src/superio/nuvoton/nct5539d/Kconfig
+++ b/src/superio/nuvoton/nct5539d/Kconfig
@@ -3,8 +3,3 @@
config SUPERIO_NUVOTON_NCT5539D
bool
select SUPERIO_NUVOTON_COMMON_PRE_RAM
-
-config SUPERIO_NUVOTON_NCT5539D_COM_A
- bool
- depends on SUPERIO_NUVOTON_NCT5539D
- default n
diff --git a/src/superio/nuvoton/nct6776/Kconfig b/src/superio/nuvoton/nct6776/Kconfig
index caa89b37fb..9f5e0843f5 100644
--- a/src/superio/nuvoton/nct6776/Kconfig
+++ b/src/superio/nuvoton/nct6776/Kconfig
@@ -3,8 +3,3 @@
config SUPERIO_NUVOTON_NCT6776
bool
select SUPERIO_NUVOTON_COMMON_PRE_RAM
-
-config SUPERIO_NUVOTON_NCT6776_COM_A
- bool
- depends on SUPERIO_NUVOTON_NCT6776
- default n
diff --git a/src/superio/nuvoton/nct6791d/Kconfig b/src/superio/nuvoton/nct6791d/Kconfig
index b06ee31a0f..a5364916c9 100644
--- a/src/superio/nuvoton/nct6791d/Kconfig
+++ b/src/superio/nuvoton/nct6791d/Kconfig
@@ -3,8 +3,3 @@
config SUPERIO_NUVOTON_NCT6791D
bool
select SUPERIO_NUVOTON_COMMON_PRE_RAM
-
-config SUPERIO_NUVOTON_NCT6791D_COM_A
- bool
- depends on SUPERIO_NUVOTON_NCT6791D
- default n