sfc: Remove locking from implementation of efx_writeo_paged()
authorBen Hutchings <bhutchings@solarflare.com>
Mon, 6 Dec 2010 22:58:41 +0000 (22:58 +0000)
committerBen Hutchings <bhutchings@solarflare.com>
Mon, 6 Dec 2010 22:58:41 +0000 (22:58 +0000)
It is not necessary to serialise writes to the paged 128-bit
registers.  However, if we don't then we must always write the last
dword separately, not as part of a qword write.

Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
drivers/net/sfc/io.h

index 7f3adf29d6f648853f91a1eae223e88506f4a7d7..6da4ae20a039df235f135d39a51e8e7213ba374f 100644 (file)
@@ -229,7 +229,20 @@ static inline void efx_readd_table(struct efx_nic *efx, efx_dword_t *value,
 static inline void _efx_writeo_page(struct efx_nic *efx, efx_oword_t *value,
                                    unsigned int reg, unsigned int page)
 {
-       efx_writeo(efx, value, EFX_PAGED_REG(page, reg));
+       reg = EFX_PAGED_REG(page, reg);
+
+       netif_vdbg(efx, hw, efx->net_dev,
+                  "writing register %x with " EFX_OWORD_FMT "\n", reg,
+                  EFX_OWORD_VAL(*value));
+
+#ifdef EFX_USE_QWORD_IO
+       _efx_writeq(efx, value->u64[0], reg + 0);
+#else
+       _efx_writed(efx, value->u32[0], reg + 0);
+       _efx_writed(efx, value->u32[1], reg + 4);
+#endif
+       _efx_writed(efx, value->u32[2], reg + 8);
+       _efx_writed(efx, value->u32[3], reg + 12);
 }
 #define efx_writeo_page(efx, value, reg, page)                         \
        _efx_writeo_page(efx, value,                                    \