commit 9911d64b9edb8c055b37b0635e97b4c585aa4ae1
author Felix Held <felix-coreboot@felixheld.de> Fri Jul 06 20:55:53 2018 +0200
committer Felix Held <felix-coreboot@felixheld.de> Wed Jul 18 11:40:20 2018 +0000
tree d823093f50b3945e11deba929f543a616113e57e
parent ad417c9c7b20529d20a72c81b225c29953439acc

superio/nuvoton: remove LDN-specific ops overrides

The pnp ops struct is already passed to the pnp_enable_devices function and it
is used if no override is supplied in the elements of the pnp_info struct array

Change-Id: I18345d7cc50a7d46cb15584dfb54df28e8534f81
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/27386
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Patrick Rudolph <siro@das-labor.org>

src/superio/nuvoton/nct6791d/superio.c

diff --git a/src/superio/nuvoton/nct6791d/superio.c b/src/superio/nuvoton/nct6791d/superio.c
index f66689e..c2d5f2e 100644
--- a/src/superio/nuvoton/nct6791d/superio.c
+++ b/src/superio/nuvoton/nct6791d/superio.c
@@ -50,44 +50,44 @@
 };
 
 static struct pnp_info pnp_dev_info[] = {
-	{ &ops, NCT6791D_PP, PNP_IO0 | PNP_IRQ0 | PNP_DRQ0,
+	{ NULL, NCT6791D_PP, PNP_IO0 | PNP_IRQ0 | PNP_DRQ0,
 		0x0ff8, },
-	{ &ops, NCT6791D_SP1, PNP_IO0 | PNP_IRQ0,
+	{ NULL, NCT6791D_SP1, PNP_IO0 | PNP_IRQ0,
 		0x0ff8, },
-	{ &ops, NCT6791D_SP2, PNP_IO0 | PNP_IRQ0,
+	{ NULL, NCT6791D_SP2, PNP_IO0 | PNP_IRQ0,
 		0x0ff8, },
-	{ &ops, NCT6791D_KBC, PNP_IO0 | PNP_IO1 | PNP_IRQ0 | PNP_IRQ1,
+	{ NULL, NCT6791D_KBC, PNP_IO0 | PNP_IO1 | PNP_IRQ0 | PNP_IRQ1,
 		0x0fff, 0x0fff, },
-	{ &ops, NCT6791D_CIR, PNP_IO0 | PNP_IRQ0,
+	{ NULL, NCT6791D_CIR, PNP_IO0 | PNP_IRQ0,
 		0x0ff8, },
-	{ &ops, NCT6791D_ACPI},
-	{ &ops, NCT6791D_HWM_FPLED, PNP_IO0 | PNP_IO1 | PNP_IRQ0,
+	{ NULL, NCT6791D_ACPI},
+	{ NULL, NCT6791D_HWM_FPLED, PNP_IO0 | PNP_IO1 | PNP_IRQ0,
 		0x0ffe, 0x0ffe, },
-	{ &ops, NCT6791D_BCLK_WDT2_WDTMEM},
-	{ &ops, NCT6791D_CIRWUP, PNP_IO0 | PNP_IRQ0,
+	{ NULL, NCT6791D_BCLK_WDT2_WDTMEM},
+	{ NULL, NCT6791D_CIRWUP, PNP_IO0 | PNP_IRQ0,
 		0x0ff8, },
-	{ &ops, NCT6791D_GPIO_PP_OD},
-	{ &ops, NCT6791D_PORT80},
-	{ &ops, NCT6791D_WDT1},
-	{ &ops, NCT6791D_WDTMEM},
-	{ &ops, NCT6791D_GPIOBASE, PNP_IO0,
+	{ NULL, NCT6791D_GPIO_PP_OD},
+	{ NULL, NCT6791D_PORT80},
+	{ NULL, NCT6791D_WDT1},
+	{ NULL, NCT6791D_WDTMEM},
+	{ NULL, NCT6791D_GPIOBASE, PNP_IO0,
 		0x0ff8, },
-	{ &ops, NCT6791D_GPIO0},
-	{ &ops, NCT6791D_GPIO1},
-	{ &ops, NCT6791D_GPIO2},
-	{ &ops, NCT6791D_GPIO3},
-	{ &ops, NCT6791D_GPIO4},
-	{ &ops, NCT6791D_GPIO5},
-	{ &ops, NCT6791D_GPIO6},
-	{ &ops, NCT6791D_GPIO7},
-	{ &ops, NCT6791D_GPIO8},
-	{ &ops, NCT6791D_DS5},
-	{ &ops, NCT6791D_DS3},
-	{ &ops, NCT6791D_PCHDSW},
-	{ &ops, NCT6791D_DSWWOPT},
-	{ &ops, NCT6791D_DS3OPT},
-	{ &ops, NCT6791D_DSDSS},
-	{ &ops, NCT6791D_DSPU},
+	{ NULL, NCT6791D_GPIO0},
+	{ NULL, NCT6791D_GPIO1},
+	{ NULL, NCT6791D_GPIO2},
+	{ NULL, NCT6791D_GPIO3},
+	{ NULL, NCT6791D_GPIO4},
+	{ NULL, NCT6791D_GPIO5},
+	{ NULL, NCT6791D_GPIO6},
+	{ NULL, NCT6791D_GPIO7},
+	{ NULL, NCT6791D_GPIO8},
+	{ NULL, NCT6791D_DS5},
+	{ NULL, NCT6791D_DS3},
+	{ NULL, NCT6791D_PCHDSW},
+	{ NULL, NCT6791D_DSWWOPT},
+	{ NULL, NCT6791D_DS3OPT},
+	{ NULL, NCT6791D_DSDSS},
+	{ NULL, NCT6791D_DSPU},
 };
 
 static void enable_dev(struct device *dev)