diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2018-07-06 23:39:00 +0200 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-07-18 18:51:52 +0000 |
| commit | b0d868e8fe3f9a13f9be109fc3b73dfa949bec15 (patch) | |
| tree | 79a9f71eb93778d8989707b3379fa56e5c9bc417 /src/superio/smsc/lpc47n217 | |
| parent | c40275bce0efb8b79792c7a6f846204b4c317124 (diff) | |
superio/smsc: pass the chip-specific ops struct to pnp_enable_devices
Pass the address of the chip-specific ops struct instead of the one of the
generic pnp_ops struct to the PNP device enable function.
This allows the removal of the LDN-specific ops overrides which is also done in
this patch.
Change-Id: I16e485494e448ae02e0a7b9e21b90ddbb1a53a4b
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/23007
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/superio/smsc/lpc47n217')
| -rw-r--r-- | src/superio/smsc/lpc47n217/superio.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/superio/smsc/lpc47n217/superio.c b/src/superio/smsc/lpc47n217/superio.c index 43b56dc549..e63645e522 100644 --- a/src/superio/smsc/lpc47n217/superio.c +++ b/src/superio/smsc/lpc47n217/superio.c @@ -57,9 +57,9 @@ static struct device_operations ops = { }; static struct pnp_info pnp_dev_info[] = { - { &ops, LPC47N217_PP, PNP_IO0 | PNP_IRQ0 | PNP_DRQ0, 0x07f8, }, - { &ops, LPC47N217_SP1, PNP_IO0 | PNP_IRQ0, 0x07f8, }, - { &ops, LPC47N217_SP2, PNP_IO0 | PNP_IRQ0, 0x07f8, } + { NULL, LPC47N217_PP, PNP_IO0 | PNP_IRQ0 | PNP_DRQ0, 0x07f8, }, + { NULL, LPC47N217_SP1, PNP_IO0 | PNP_IRQ0, 0x07f8, }, + { NULL, LPC47N217_SP2, PNP_IO0 | PNP_IRQ0, 0x07f8, } }; /** @@ -70,8 +70,7 @@ static struct pnp_info pnp_dev_info[] = { */ static void enable_dev(struct device *dev) { - pnp_enable_devices(dev, &pnp_ops, ARRAY_SIZE(pnp_dev_info), - pnp_dev_info); + pnp_enable_devices(dev, &ops, ARRAY_SIZE(pnp_dev_info), pnp_dev_info); } /** |