bcma: add support for 1 and 2 byte extended config space access
authorNathan Hintz <nlhintz@hotmail.com>
Sat, 12 Jan 2013 10:46:16 +0000 (02:46 -0800)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 14 Jan 2013 20:10:41 +0000 (15:10 -0500)
The sanity checks allow 1 and 2 byte reads/writes of the extended
PCI config space to proceed; however, the code only supports 4
byte reads/writes.  This patch adds support for 1 and 2 byte
reads/writes of the extended PCI config space.

Signed-off-by: Nathan Hintz <nlhintz@hotmail.com>
Acked-by: Hauke Mehrtens <hauke@hauke-m.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/bcma/driver_pci_host.c

index c0cb0620ab62ab9865ff8bc745faae47094175a5..f92124c299983df66d61bf0d7adccf78220d1367 100644 (file)
@@ -101,7 +101,7 @@ static int bcma_extpci_read_config(struct bcma_drv_pci *pc, unsigned int dev,
                 */
                if (off >= PCI_CONFIG_SPACE_SIZE) {
                        addr = (func << 12);
-                       addr |= (off & 0x0FFF);
+                       addr |= (off & 0x0FFC);
                        val = bcma_pcie_read_config(pc, addr);
                } else {
                        addr = BCMA_CORE_PCI_PCICFG0;
@@ -164,7 +164,11 @@ static int bcma_extpci_write_config(struct bcma_drv_pci *pc, unsigned int dev,
                /* accesses to config registers with offsets >= 256
                 * requires indirect access.
                 */
-               if (off < PCI_CONFIG_SPACE_SIZE) {
+               if (off >= PCI_CONFIG_SPACE_SIZE) {
+                       addr = (func << 12);
+                       addr |= (off & 0x0FFC);
+                       val = bcma_pcie_read_config(pc, addr);
+               } else {
                        addr = BCMA_CORE_PCI_PCICFG0;
                        addr |= (func << 8);
                        addr |= (off & 0xfc);
@@ -202,13 +206,10 @@ static int bcma_extpci_write_config(struct bcma_drv_pci *pc, unsigned int dev,
                /* accesses to config registers with offsets >= 256
                 * requires indirect access.
                 */
-               if (off >= PCI_CONFIG_SPACE_SIZE) {
-                       addr = (func << 12);
-                       addr |= (off & 0x0FFF);
+               if (off >= PCI_CONFIG_SPACE_SIZE)
                        bcma_pcie_write_config(pc, addr, val);
-               } else {
+               else
                        pcicore_write32(pc, addr, val);
-               }
        } else {
                writel(val, mmio);