IB/ipath: Add flag and handling for chips with swapped register bug
authorRalph Campbell <ralph.campbell@qlogic.com>
Mon, 7 Jan 2008 05:12:38 +0000 (21:12 -0800)
committerRoland Dreier <rolandd@cisco.com>
Fri, 25 Jan 2008 22:15:39 +0000 (14:15 -0800)
The 6110 had a bug that caused some registers to be swapped; it was
fixed for the 7220 (and didn't affect the 6120 because it had fewer
registers).  This adds a flag and related code to handle that, and
includes some minor cleanups in the same area.

Signed-off-by: Ralph Campbell <ralph.campbell@qlogic.com>
drivers/infiniband/hw/ipath/ipath_driver.c
drivers/infiniband/hw/ipath/ipath_iba6110.c
drivers/infiniband/hw/ipath/ipath_init_chip.c
drivers/infiniband/hw/ipath/ipath_intr.c
drivers/infiniband/hw/ipath/ipath_kernel.h

index f130652cfaa325bb7618aa449630730318b40b12..5a9dc317f40abfa7a3cd86cb1b80a6339acc7a88 100644 (file)
@@ -1340,14 +1340,9 @@ static void ipath_update_pio_bufs(struct ipath_devdata *dd)
                /*
                 * Chip Errata: bug 6641; even and odd qwords>3 are swapped
                 */
-               if (i > 3) {
-                       if (i & 1)
-                               piov = le64_to_cpu(
-                                       dd->ipath_pioavailregs_dma[i - 1]);
-                       else
-                               piov = le64_to_cpu(
-                                       dd->ipath_pioavailregs_dma[i + 1]);
-               } else
+               if (i > 3 && (dd->ipath_flags & IPATH_SWAP_PIOBUFS))
+                       piov = le64_to_cpu(dd->ipath_pioavailregs_dma[i ^ 1]);
+               else
                        piov = le64_to_cpu(dd->ipath_pioavailregs_dma[i]);
                pchg = _IPATH_ALL_CHECKBITS &
                        ~(dd->ipath_pioavailshadow[i] ^ piov);
index 0c900c51e538c4585419503c803346d5a90ec4ce..3bfaf04a214816f1d51c9e4699119cbcdad86e14 100644 (file)
@@ -1043,6 +1043,8 @@ static int ipath_setup_ht_config(struct ipath_devdata *dd,
        } while ((pos = pci_find_next_capability(pdev, pos,
                                                 PCI_CAP_ID_HT)));
 
+       dd->ipath_flags |= IPATH_SWAP_PIOBUFS;
+
 bail:
        return ret;
 }
index 3174c315999bc2781c32e6722ce96d7bbd12180a..4471674975cd11e41fa15800ab3b8f9cfedeb104 100644 (file)
@@ -528,12 +528,8 @@ static void enable_chip(struct ipath_devdata *dd,
                /*
                 * Chip Errata bug 6641; even and odd qwords>3 are swapped.
                 */
-               if (i > 3) {
-                       if (i & 1)
-                               val = dd->ipath_pioavailregs_dma[i - 1];
-                       else
-                               val = dd->ipath_pioavailregs_dma[i + 1];
-               }
+               if (i > 3 && (dd->ipath_flags & IPATH_SWAP_PIOBUFS))
+                       val = dd->ipath_pioavailregs_dma[i ^ 1];
                else
                        val = dd->ipath_pioavailregs_dma[i];
                dd->ipath_pioavailshadow[i] = le64_to_cpu(val);
index e2ce5314504da43fc672aa78308333f512c3828e..ddc0a19c5eee637ec99c5c428040e59f1b9d4ba5 100644 (file)
@@ -831,8 +831,8 @@ void ipath_clear_freeze(struct ipath_devdata *dd)
         */
        for (i = 0; i < dd->ipath_pioavregs; i++) {
                /* deal with 6110 chip bug */
-               im = i > 3 ? ((i&1) ? i-1 : i+1) : i;
-               val = ipath_read_kreg64(dd, (0x1000/sizeof(u64))+im);
+               im = i > 3 ? i ^ 1 : i;
+               val = ipath_read_kreg64(dd, (0x1000 / sizeof(u64)) + im);
                dd->ipath_pioavailregs_dma[i] = dd->ipath_pioavailshadow[i]
                        = le64_to_cpu(val);
        }
index b9cca8403239faccbf5287dd4303be9601aa1881..e07df00683123691391a8e32b62c31d540f0fda7 100644 (file)
@@ -756,6 +756,7 @@ int ipath_set_rx_pol_inv(struct ipath_devdata *dd, u8 new_pol_inv);
 #define IPATH_DISABLED      0x80000 /* administratively disabled */
                /* Use GPIO interrupts for new counters */
 #define IPATH_GPIO_ERRINTRS 0x100000
+#define IPATH_SWAP_PIOBUFS  0x200000
 
 /* Bits in GPIO for the added interrupts */
 #define IPATH_GPIO_PORT0_BIT 2