Paul Menzel has uploaded this change for review.

View Change

superio/nuvoton/common: Collapse two if statements

There are more devices requiring this code, so avoid duplicating the if
block over and over.

Change-Id: Ib4f787e3c883b1fec941de77bc8e19ccf0d5224c
Signed-off-by: Paul Menzel <pmenzel@molgen.mpg.de>
---
M src/superio/nuvoton/common/early_serial.c
1 file changed, 2 insertions(+), 5 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/70/45970/1

diff --git a/src/superio/nuvoton/common/early_serial.c b/src/superio/nuvoton/common/early_serial.c
index 22d882e..cc21f40 100644
--- a/src/superio/nuvoton/common/early_serial.c
+++ b/src/superio/nuvoton/common/early_serial.c
@@ -52,11 +52,8 @@
 
 	nuvoton_pnp_enter_conf_state(dev);
 
-	if (CONFIG(SUPERIO_NUVOTON_NCT5539D_COM_A))
-		/* Route COM A to GPIO8 pin group */
-		pnp_write_config(dev, 0x2a, 0x40);
-
-	if (CONFIG(SUPERIO_NUVOTON_NCT6776_COM_A))
+	if (CONFIG(SUPERIO_NUVOTON_NCT5539D_COM_A) ||
+	    CONFIG(SUPERIO_NUVOTON_NCT6776_COM_A))
 		/* Route COM A to GPIO8 pin group */
 		pnp_write_config(dev, 0x2a, 0x40);
 

To view, visit change 45970. To unsubscribe, or for help writing mail filters, visit settings.