summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2019-09-27 18:09:52 +0300
committerKyösti Mälkki <kyosti.malkki@gmail.com>2019-09-28 21:17:33 +0000
commitf2cc3dd8eaa14f1253cdab0326d5aa575d6ac354 (patch)
treefd2a6885531db4c5729895e8859be6e6c8f25331
parent8f5138d8a510d98e13e49d84afeaa8a075563571 (diff)
downloadcoreboot-f2cc3dd8eaa14f1253cdab0326d5aa575d6ac354.tar.xz
usbdebug: Remove some __SIMPLE_DEVICE__ use
We can always PCI config accessors with pci_devfn_t. Change-Id: I6d98c2441cc870cdcadbe8fabc9f35b9ffc652d8 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/35651 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r--src/drivers/usb/pci_ehci.c33
1 files changed, 11 insertions, 22 deletions
diff --git a/src/drivers/usb/pci_ehci.c b/src/drivers/usb/pci_ehci.c
index 34684cb09a..1e755249b1 100644
--- a/src/drivers/usb/pci_ehci.c
+++ b/src/drivers/usb/pci_ehci.c
@@ -33,45 +33,39 @@ static struct device_operations ehci_dbg_ops;
int ehci_debug_hw_enable(unsigned int *base, unsigned int *dbg_offset)
{
- pci_devfn_t dbg_dev = pci_ehci_dbg_dev(CONFIG_USBDEBUG_HCD_INDEX);
+ pci_devfn_t dev = pci_ehci_dbg_dev(CONFIG_USBDEBUG_HCD_INDEX);
/* We only support controllers on bus 0. */
- if (PCI_DEV2SEGBUS(dbg_dev) != 0)
+ if (PCI_DEV2SEGBUS(dev) != 0)
return -1;
-#ifdef __SIMPLE_DEVICE__
- pci_devfn_t dev = dbg_dev;
-#else
- struct device *dev = pcidev_path_on_root(PCI_DEV2DEVFN(dbg_dev));
-#endif
-
- u32 class = pci_read_config32(dev, PCI_CLASS_REVISION) >> 8;
+ u32 class = pci_s_read_config32(dev, PCI_CLASS_REVISION) >> 8;
if (class != PCI_EHCI_CLASSCODE)
return -1;
- u8 pm_cap = pci_s_find_capability(dbg_dev, PCI_CAP_ID_PM);
+ u8 pm_cap = pci_s_find_capability(dev, PCI_CAP_ID_PM);
if (pm_cap) {
- u16 pm_ctrl = pci_read_config16(dev, pm_cap + PCI_PM_CTRL);
+ u16 pm_ctrl = pci_s_read_config16(dev, pm_cap + PCI_PM_CTRL);
/* Set to D0 and disable PM events. */
pm_ctrl &= ~PCI_PM_CTRL_PME_ENABLE;
pm_ctrl &= ~PCI_PM_CTRL_STATE_MASK;
- pci_write_config16(dev, pm_cap + PCI_PM_CTRL, pm_ctrl);
+ pci_s_write_config16(dev, pm_cap + PCI_PM_CTRL, pm_ctrl);
}
- u8 pos = pci_s_find_capability(dbg_dev, PCI_CAP_ID_EHCI_DEBUG);
+ u8 pos = pci_s_find_capability(dev, PCI_CAP_ID_EHCI_DEBUG);
if (!pos)
return -1;
- u32 cap = pci_read_config32(dev, pos);
+ u32 cap = pci_s_read_config32(dev, pos);
/* FIXME: We should remove static EHCI_BAR_INDEX. */
u8 ehci_bar = 0x10 + 4 * ((cap >> 29) - 1);
if (ehci_bar != EHCI_BAR_INDEX)
return -1;
- pci_write_config32(dev, ehci_bar, CONFIG_EHCI_BAR);
+ pci_s_write_config32(dev, ehci_bar, CONFIG_EHCI_BAR);
- pci_write_config8(dev, PCI_COMMAND, PCI_COMMAND_MEMORY |
+ pci_s_write_config8(dev, PCI_COMMAND, PCI_COMMAND_MEMORY |
PCI_COMMAND_MASTER);
*base = CONFIG_EHCI_BAR;
@@ -125,11 +119,6 @@ void pci_ehci_read_resources(struct device *dev)
u8 *pci_ehci_base_regs(pci_devfn_t sdev)
{
-#ifdef __SIMPLE_DEVICE__
- u8 *base = (u8 *)(pci_read_config32(sdev, EHCI_BAR_INDEX) & ~0x0f);
-#else
- struct device *dev = pcidev_path_on_root(PCI_DEV2DEVFN(sdev));
- u8 *base = (u8 *)(pci_read_config32(dev, EHCI_BAR_INDEX) & ~0x0f);
-#endif
+ u8 *base = (u8 *)(pci_s_read_config32(sdev, EHCI_BAR_INDEX) & ~0x0f);
return base + HC_LENGTH(read32(base));
}