diff options
author | Russell King <rmk+kernel@arm.linux.org.uk> | 2013-06-26 18:49:11 -0400 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2013-09-21 16:02:19 -0400 |
commit | 9663dedafa302bb7fd511f9ae75d9ce33865477e (patch) | |
tree | 842de392a3da79dc294064c778edd4777787eb77 | |
parent | 9931a26ea7fa25b02547c2a15f4aa80f538197e2 (diff) |
DMA-API: net: sfc/efx.c: replace dma_set_mask()+dma_set_coherent_mask() with new helper
Replace the following sequence:
dma_set_mask(dev, mask);
dma_set_coherent_mask(dev, mask);
with a call to the new helper dma_set_mask_and_coherent().
Acked-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
-rw-r--r-- | drivers/net/ethernet/sfc/efx.c | 12 |
1 files changed, 1 insertions, 11 deletions
diff --git a/drivers/net/ethernet/sfc/efx.c b/drivers/net/ethernet/sfc/efx.c index 07c9bc4c61bc..2e27837ce6a2 100644 --- a/drivers/net/ethernet/sfc/efx.c +++ b/drivers/net/ethernet/sfc/efx.c | |||
@@ -1121,7 +1121,7 @@ static int efx_init_io(struct efx_nic *efx) | |||
1121 | */ | 1121 | */ |
1122 | while (dma_mask > 0x7fffffffUL) { | 1122 | while (dma_mask > 0x7fffffffUL) { |
1123 | if (dma_supported(&pci_dev->dev, dma_mask)) { | 1123 | if (dma_supported(&pci_dev->dev, dma_mask)) { |
1124 | rc = dma_set_mask(&pci_dev->dev, dma_mask); | 1124 | rc = dma_set_mask_and_coherent(&pci_dev->dev, dma_mask); |
1125 | if (rc == 0) | 1125 | if (rc == 0) |
1126 | break; | 1126 | break; |
1127 | } | 1127 | } |
@@ -1134,16 +1134,6 @@ static int efx_init_io(struct efx_nic *efx) | |||
1134 | } | 1134 | } |
1135 | netif_dbg(efx, probe, efx->net_dev, | 1135 | netif_dbg(efx, probe, efx->net_dev, |
1136 | "using DMA mask %llx\n", (unsigned long long) dma_mask); | 1136 | "using DMA mask %llx\n", (unsigned long long) dma_mask); |
1137 | rc = dma_set_coherent_mask(&pci_dev->dev, dma_mask); | ||
1138 | if (rc) { | ||
1139 | /* dma_set_coherent_mask() is not *allowed* to | ||
1140 | * fail with a mask that dma_set_mask() accepted, | ||
1141 | * but just in case... | ||
1142 | */ | ||
1143 | netif_err(efx, probe, efx->net_dev, | ||
1144 | "failed to set consistent DMA mask\n"); | ||
1145 | goto fail2; | ||
1146 | } | ||
1147 | 1137 | ||
1148 | efx->membase_phys = pci_resource_start(efx->pci_dev, EFX_MEM_BAR); | 1138 | efx->membase_phys = pci_resource_start(efx->pci_dev, EFX_MEM_BAR); |
1149 | rc = pci_request_region(pci_dev, EFX_MEM_BAR, "sfc"); | 1139 | rc = pci_request_region(pci_dev, EFX_MEM_BAR, "sfc"); |