diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2017-12-27 22:11:30 +0100 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2018-07-18 18:49:40 +0000 |
| commit | c40275bce0efb8b79792c7a6f846204b4c317124 (patch) | |
| tree | e8f81f5970721ddc93b759d5374708de58ba9867 /src/superio/ite/it8623e/superio.c | |
| parent | 3a59174cf02c8c0afd8eb547f5325788cd3537ea (diff) | |
superio/ite: 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: I5f03a4064778c419f4b9c50e70db1296addf6c9e
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Reviewed-on: https://review.coreboot.org/23006
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/superio/ite/it8623e/superio.c')
| -rw-r--r-- | src/superio/ite/it8623e/superio.c | 21 |
1 files changed, 11 insertions, 10 deletions
diff --git a/src/superio/ite/it8623e/superio.c b/src/superio/ite/it8623e/superio.c index a9cd4e55ae..d735a5948f 100644 --- a/src/superio/ite/it8623e/superio.c +++ b/src/superio/ite/it8623e/superio.c @@ -60,20 +60,21 @@ static struct device_operations ops = { }; static struct pnp_info pnp_dev_info[] = { - { &ops, IT8623E_FDC, PNP_IO0 | PNP_IRQ0 | PNP_DRQ0, 0x0ff8, }, - { &ops, IT8623E_SP1, PNP_IO0 | PNP_IRQ0, 0x0ff8, }, - { &ops, IT8623E_SP2, PNP_IO0 | PNP_IRQ0, 0x0ff8, }, - { &ops, IT8623E_PP, PNP_IO0 | PNP_IO1 | PNP_IRQ0 | PNP_DRQ0, 0x0ff8, 0x0ff8, }, - { &ops, IT8623E_EC, PNP_IO0 | PNP_IO1 | PNP_IRQ0, 0x0ff0, 0x0ff0, }, - { &ops, IT8623E_KBCK, PNP_IO0 | PNP_IO1 | PNP_IRQ0, 0x0fff, 0x0fff, }, - { &ops, IT8623E_KBCM, PNP_IRQ0, }, - { &ops, IT8623E_GPIO, PNP_IO0 | PNP_IO1 | PNP_IO2 | PNP_IRQ0, 0x0fff, 0x0fe0, 0x0fff, }, + { NULL, IT8623E_FDC, PNP_IO0 | PNP_IRQ0 | PNP_DRQ0, 0x0ff8, }, + { NULL, IT8623E_SP1, PNP_IO0 | PNP_IRQ0, 0x0ff8, }, + { NULL, IT8623E_SP2, PNP_IO0 | PNP_IRQ0, 0x0ff8, }, + { NULL, IT8623E_PP, PNP_IO0 | PNP_IO1 | PNP_IRQ0 | PNP_DRQ0, + 0x0ff8, 0x0ff8, }, + { NULL, IT8623E_EC, PNP_IO0 | PNP_IO1 | PNP_IRQ0, 0x0ff0, 0x0ff0, }, + { NULL, IT8623E_KBCK, PNP_IO0 | PNP_IO1 | PNP_IRQ0, 0x0fff, 0x0fff, }, + { NULL, IT8623E_KBCM, PNP_IRQ0, }, + { NULL, IT8623E_GPIO, PNP_IO0 | PNP_IO1 | PNP_IO2 | PNP_IRQ0, + 0x0fff, 0x0fe0, 0x0fff, }, }; static void enable_dev(struct device *dev) { - pnp_enable_devices(dev, &ops, - ARRAY_SIZE(pnp_dev_info), pnp_dev_info); + pnp_enable_devices(dev, &ops, ARRAY_SIZE(pnp_dev_info), pnp_dev_info); } struct chip_operations superio_ite_it8623e_ops = { |