pci: remove PIO pci_{read,write}{8,16,32} declarations
authorBarret Rhoden <brho@cs.berkeley.edu>
Tue, 13 Aug 2019 21:35:27 +0000 (17:35 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Wed, 14 Aug 2019 15:36:09 +0000 (11:36 -0400)
The port-mapped IO interface to PCI config space is only used in pci.c.
We want to allow MMIO PCI config space too, and we don't want other
files using the PIO internal functions.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/arch/x86/pci.c
kern/arch/x86/pci.h

index 3da1b86..34c79d1 100644 (file)
@@ -25,6 +25,9 @@ static char STD_PCI_DEV[] = "Standard PCI Device";
 static char PCI2PCI[] = "PCI-to-PCI Bridge";
 static char PCI2CARDBUS[] = "PCI-Cardbus Bridge";
 
+static uint32_t pci_read32(uint8_t bus, uint8_t dev, uint8_t func,
+                          uint32_t offset);
+
 /* Gets any old raw bar, with some catches based on type. */
 static uint32_t pci_getbar(struct pci_device *pcidev, unsigned int bar)
 {
@@ -306,7 +309,8 @@ uint32_t pci_config_addr(uint8_t bus, uint8_t dev, uint8_t func, uint32_t reg)
 
 /* Helper to read 32 bits from the config space of B:D:F.  'Offset' is how far
  * into the config space we offset before reading, aka: where we are reading. */
-uint32_t pci_read32(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset)
+static uint32_t pci_read32(uint8_t bus, uint8_t dev, uint8_t func,
+                          uint32_t offset)
 {
        uint32_t ret;
 
@@ -319,8 +323,8 @@ uint32_t pci_read32(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset)
 
 /* Same, but writes (doing 32bit at a time).  Never actually tested (not sure if
  * PCI lets you write back). */
-void pci_write32(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
-                 uint32_t value)
+static void pci_write32(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
+                       uint32_t value)
 {
        spin_lock_irqsave(&pci_lock);
        outl(PCI_CONFIG_ADDR, pci_config_addr(bus, dev, func, offset));
@@ -328,7 +332,8 @@ void pci_write32(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
        spin_unlock_irqsave(&pci_lock);
 }
 
-uint16_t pci_read16(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset)
+static uint16_t pci_read16(uint8_t bus, uint8_t dev, uint8_t func,
+                          uint32_t offset)
 {
        uint16_t ret;
 
@@ -339,8 +344,8 @@ uint16_t pci_read16(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset)
        return ret;
 }
 
-void pci_write16(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
-                 uint16_t value)
+static void pci_write16(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
+                       uint16_t value)
 {
        spin_lock_irqsave(&pci_lock);
        outl(PCI_CONFIG_ADDR, pci_config_addr(bus, dev, func, offset));
@@ -348,7 +353,8 @@ void pci_write16(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
        spin_unlock_irqsave(&pci_lock);
 }
 
-uint8_t pci_read8(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset)
+static uint8_t pci_read8(uint8_t bus, uint8_t dev, uint8_t func,
+                        uint32_t offset)
 {
        uint8_t ret;
 
@@ -359,8 +365,8 @@ uint8_t pci_read8(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset)
        return ret;
 }
 
-void pci_write8(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
-                uint8_t value)
+static void pci_write8(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
+                      uint8_t value)
 {
        spin_lock_irqsave(&pci_lock);
        outl(PCI_CONFIG_ADDR, pci_config_addr(bus, dev, func, offset));
index ba0ad9f..89c355a 100644 (file)
@@ -235,17 +235,6 @@ void pci_init(void);
 void pcidev_print_info(struct pci_device *pcidev, int verbosity);
 uint32_t pci_config_addr(uint8_t bus, uint8_t dev, uint8_t func, uint32_t reg);
 
-/* Read and write helpers (Eventually, we should have these be statics, since no
- * device should touch PCI config space). */
-uint32_t pci_read32(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset);
-void pci_write32(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
-                 uint32_t value);
-uint16_t pci_read16(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset);
-void pci_write16(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
-                 uint16_t value);
-uint8_t pci_read8(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset);
-void pci_write8(uint8_t bus, uint8_t dev, uint8_t func, uint32_t offset,
-                uint8_t value);
 uint32_t pcidev_read32(struct pci_device *pcidev, uint32_t offset);
 void pcidev_write32(struct pci_device *pcidev, uint32_t offset, uint32_t value);
 uint16_t pcidev_read16(struct pci_device *pcidev, uint32_t offset);