New patch to review for coreboot: e4d4373 superio/nct5104d: Handle shared GPIO/UART pins

[Kyösti Mälkki]

Kyösti Mälkki ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10177

-gerrit

commit e4d4373d50d90261cfc8915af9041e1ec699f5bd
Author: Kyösti Mälkki 
Date:   Mon May 11 20:58:18 2015 +0300

superio/nct5104d: Handle shared GPIO/UART pins

Routing is decided based on enabled logical/virtual devices.
    For a valid devicetree, one should have only one of SP3 and GPIO0,
    and only one of SP4 and GPIO1, enabled at a time in configuration.

Change-Id: I02017786aba9dd22d12403aaa71d7641f5bbf997
    Signed-off-by: Kyösti Mälkki 
---
 src/superio/nuvoton/nct5104d/superio.c | 37 ++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/src/superio/nuvoton/nct5104d/superio.c b/src/superio/nuvoton/nct5104d/superio.c
index f382c2c..01c1c27 100644
--- a/src/superio/nuvoton/nct5104d/superio.c
+++ b/src/superio/nuvoton/nct5104d/superio.c
@@ -81,6 +81,36 @@ static void set_irq_trigger_type(struct device *dev, bool trig_level)
    pnp_write_config(dev, GLOBAL_OPTION_CR26, reg26);
 }

+static void route_pins_to_uart(struct device *dev, bool to_uart)
+{
+	u8 reg;
+
+	reg = pnp_read_config(dev, 0x1c);
+
+	switch (dev->path.pnp.device) {
+	case NCT5104D_SP3:
+	case NCT5104D_GPIO0:
+		/* Route pins 33 - 40. */
+		if (to_uart)
+			reg |= (1 << 3);
+		else
+			reg &= ~(1 << 3);
+		break;
+	case NCT5104D_SP4:
+	case NCT5104D_GPIO1:
+		/* Route pins 41 - 48. */
+		if (to_uart)
+			reg |= (1 << 2);
+		else
+			reg &= ~(1 << 2);
+		break;
+	default:
+		break;
+	}
+
+	pnp_write_config(dev, 0x1c, reg);
+}
+
 static void nct5104d_init(struct device *dev)
 {
    struct superio_nuvoton_nct5104d_config *conf = dev->chip_info;
@@ -93,10 +123,17 @@ static void nct5104d_init(struct device *dev)
    switch(dev->path.pnp.device) {
    case NCT5104D_SP1:
    case NCT5104D_SP2:
+		set_irq_trigger_type(dev, conf->irq_trigger_type != 0);
+		break;
    case NCT5104D_SP3:
    case NCT5104D_SP4:
+		route_pins_to_uart(dev, true);
    	set_irq_trigger_type(dev, conf->irq_trigger_type != 0);
    	break;
+	case NCT5104D_GPIO0:
+	case NCT5104D_GPIO1:
+		route_pins_to_uart(dev, false);
+		break;
    default:
    	break;
    }