diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2019-03-11 20:34:26 +0200 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2019-03-16 15:19:06 +0000 |
commit | 34cf5619f929775efd819468ba6036e637cfbd85 (patch) | |
tree | e6763075a5e36710977ae51fcfd4a67b9bf3fffe /src/include/device/pci.h | |
parent | 74aa99a5435ce3a1b984a3e0e5a0b696d6f6165d (diff) |
device/pci_ops: Reuse romstage PCI config for ramstage
By changing the signatures we do not need to define
PCI config accessors separately for ramstage.
Change-Id: I9364cb34fe8127972c772516a0a0b1d281c5ed00
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/31685
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/include/device/pci.h')
-rw-r--r-- | src/include/device/pci.h | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/src/include/device/pci.h b/src/include/device/pci.h index 2cfcb60244..14c4693459 100644 --- a/src/include/device/pci.h +++ b/src/include/device/pci.h @@ -35,14 +35,17 @@ struct pci_operations { /* Common pci bus operations */ struct pci_bus_operations { - uint8_t (*read8)(const struct device *dev, uint16_t reg); - uint16_t (*read16)(const struct device *dev, uint16_t reg); - uint32_t (*read32)(const struct device *dev, uint16_t reg); - void (*write8)(const struct device *dev, uint16_t reg, uint8_t val); - void (*write16)(const struct device *dev, uint16_t reg, uint16_t val); - void (*write32)(const struct device *dev, uint16_t reg, uint32_t val); + uint8_t (*read8)(pci_devfn_t dev, uint16_t reg); + uint16_t (*read16)(pci_devfn_t dev, uint16_t reg); + uint32_t (*read32)(pci_devfn_t dev, uint16_t reg); + void (*write8)(pci_devfn_t dev, uint16_t reg, uint8_t val); + void (*write16)(pci_devfn_t dev, uint16_t reg, uint16_t val); + void (*write32)(pci_devfn_t dev, uint16_t reg, uint32_t val); }; +// FIXME: Needs complete pci_bus_operations +#include <device/pci_ops.h> + struct pci_driver { const struct device_operations *ops; unsigned short vendor; |