[coreboot-gerrit] New patch to review for coreboot: superio/nuvoton/nct5572d: Add missing logical devices

[Kyösti Mälkki (kyosti.malkki@gmail.com)]

Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/12213

-gerrit

commit 6f6f94a8b554848c07aff60e3ccaaaaf2832fde0
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Tue Oct 27 18:00:57 2015 +0200

    superio/nuvoton/nct5572d: Add missing logical devices
    
    While the actual pins behind these devices are not exposed on the chip,
    the enable registers are implemented in hardware. Allow to turn these LDNs
    off, like the vendor bios for asrock/e350m1 does.
    
    Change-Id: I4d6d5a8de12b09095138cacbad62b2dfbbe54028
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/superio/nuvoton/nct5572d/nct5572d.h | 4 ++++
 src/superio/nuvoton/nct5572d/superio.c  | 4 ++++
 2 files changed, 8 insertions(+)

diff --git a/src/superio/nuvoton/nct5572d/nct5572d.h b/src/superio/nuvoton/nct5572d/nct5572d.h
index 94e7988..4cc28ba 100644
--- a/src/superio/nuvoton/nct5572d/nct5572d.h
+++ b/src/superio/nuvoton/nct5572d/nct5572d.h
@@ -22,6 +22,8 @@
 #define SUPERIO_NUVOTON_NCT5572D
 
 /* Logical Device Numbers (LDN). */
+#define NCT5572D_FDC		0x00
+#define NCT5572D_PP			0x01
 #define NCT5572D_SP1		0x02 /* Com1 */
 #define NCT5572D_IR		0x03
 #define NCT5572D_KBC		0x05
@@ -39,8 +41,10 @@
 /* virtual LDN for GPIO */
 #define NCT5572D_GPIO2		((0 << 8) | NCT5572D_GPIO235_V)
 #define NCT5572D_GPIO3		((1 << 8) | NCT5572D_GPIO235_V)
+#define NCT5572D_GPIO4		((2 << 8) | NCT5572D_GPIO235_V)
 #define NCT5572D_GPIO5		((3 << 8) | NCT5572D_GPIO235_V)
 #define NCT5572D_GPIO6		((1 << 8) | NCT5572D_GPIO689_V)
+#define NCT5572D_GPIO7		((2 << 8) | NCT5572D_GPIO689_V)
 #define NCT5572D_GPIO8		((3 << 8) | NCT5572D_GPIO689_V)
 #define NCT5572D_GPIO9		((4 << 8) | NCT5572D_GPIO689_V)
 
diff --git a/src/superio/nuvoton/nct5572d/superio.c b/src/superio/nuvoton/nct5572d/superio.c
index 3b71c6b..14c7d1d 100644
--- a/src/superio/nuvoton/nct5572d/superio.c
+++ b/src/superio/nuvoton/nct5572d/superio.c
@@ -80,6 +80,8 @@ static struct device_operations ops = {
 };
 
 static struct pnp_info pnp_dev_info[] = {
+	{ &ops, NCT5572D_FDC, PNP_IO0 | PNP_IRQ0, {0x0FF8, 0}, },
+	{ &ops, NCT5572D_PP, PNP_IO0 | PNP_IRQ0, {0x0FF8, 0}, },
 	{ &ops, NCT5572D_SP1, PNP_IO0 | PNP_IRQ0, {0x0FF8, 0}, },
 	{ &ops, NCT5572D_IR, PNP_IO0 | PNP_IRQ0, {0x0FF8, 0}, },
 	{ &ops, NCT5572D_KBC, PNP_IO0 | PNP_IO1 | PNP_IRQ0 | PNP_IRQ1, {0x0FFF, 0}, {0x0FFF, 4}, },
@@ -93,8 +95,10 @@ static struct pnp_info pnp_dev_info[] = {
 	{ &ops, NCT5572D_GPIO_PP_OD},
 	{ &ops, NCT5572D_GPIO2},
 	{ &ops, NCT5572D_GPIO3},
+	{ &ops, NCT5572D_GPIO4},
 	{ &ops, NCT5572D_GPIO5},
 	{ &ops, NCT5572D_GPIO6},
+	{ &ops, NCT5572D_GPIO7},
 	{ &ops, NCT5572D_GPIO8},
 	{ &ops, NCT5572D_GPIO9},
 };