diff options
| author | Vincent Stehlé <vincent.stehle@laposte.net> | 2020-12-13 19:26:22 +0100 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2020-12-15 12:50:12 +0100 |
| commit | d0edaa28a1f7830997131cbce87b6c52472825d1 (patch) | |
| tree | 4dbd378e0e0d9df54885a95f1179e1da7fc7aa46 /drivers/ps3/ps3stor_lib.c | |
| parent | powerpc: force inlining of csum_partial() to avoid multiple csum_partial() wi... (diff) | |
| download | linux-d0edaa28a1f7830997131cbce87b6c52472825d1.tar.xz linux-d0edaa28a1f7830997131cbce87b6c52472825d1.zip | |
powerpc/ps3: use dma_mapping_error()
The DMA address returned by dma_map_single() should be checked with
dma_mapping_error(). Fix the ps3stor_setup() function accordingly.
Fixes: 80071802cb9c ("[POWERPC] PS3: Storage Driver Core")
Signed-off-by: Vincent Stehlé <vincent.stehle@laposte.net>
Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20201213182622.23047-1-vincent.stehle@laposte.net
Diffstat (limited to 'drivers/ps3/ps3stor_lib.c')
| -rw-r--r-- | drivers/ps3/ps3stor_lib.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/ps3/ps3stor_lib.c b/drivers/ps3/ps3stor_lib.c index 333ba83006e4..a12a1ad9b5fe 100644 --- a/drivers/ps3/ps3stor_lib.c +++ b/drivers/ps3/ps3stor_lib.c @@ -189,7 +189,7 @@ int ps3stor_setup(struct ps3_storage_device *dev, irq_handler_t handler) dev->bounce_lpar = ps3_mm_phys_to_lpar(__pa(dev->bounce_buf)); dev->bounce_dma = dma_map_single(&dev->sbd.core, dev->bounce_buf, dev->bounce_size, DMA_BIDIRECTIONAL); - if (!dev->bounce_dma) { + if (dma_mapping_error(&dev->sbd.core, dev->bounce_dma)) { dev_err(&dev->sbd.core, "%s:%u: map DMA region failed\n", __func__, __LINE__); error = -ENODEV; |