Skip to content

Commit 4ace7a6

Browse files
Dan Carpenterkuba-moo
authored andcommitted
net: ipa: pass correct dma_handle to dma_free_coherent()
The "ring->addr = addr;" assignment is done a few lines later so we can't use "ring->addr" yet. The correct dma_handle is "addr". Fixes: 650d160 ("soc: qcom: ipa: the generic software interface") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Alex Elder <elder@linaro.org> Link: https://lore.kernel.org/r/YBjpTU2oejkNIULT@mwanda Signed-off-by: Jakub Kicinski <kuba@kernel.org>
1 parent cc9f07a commit 4ace7a6

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/net/ipa/gsi.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1373,7 +1373,7 @@ static int gsi_ring_alloc(struct gsi *gsi, struct gsi_ring *ring, u32 count)
13731373
/* Hardware requires a 2^n ring size, with alignment equal to size */
13741374
ring->virt = dma_alloc_coherent(dev, size, &addr, GFP_KERNEL);
13751375
if (ring->virt && addr % size) {
1376-
dma_free_coherent(dev, size, ring->virt, ring->addr);
1376+
dma_free_coherent(dev, size, ring->virt, addr);
13771377
dev_err(dev, "unable to alloc 0x%zx-aligned ring buffer\n",
13781378
size);
13791379
return -EINVAL; /* Not a good error value, but distinct */

0 commit comments

Comments
 (0)