From: Moti Haimovski <mhaimov...@habana.ai>

This commit minimizes the "chain of errors" displayed when memory
mapping fails.

Signed-off-by: Moti Haimovski <mhaimov...@habana.ai>
Reviewed-by: Oded Gabbay <ogab...@kernel.org>
Signed-off-by: Oded Gabbay <ogab...@kernel.org>
---
 drivers/accel/habanalabs/common/memory.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/drivers/accel/habanalabs/common/memory.c 
b/drivers/accel/habanalabs/common/memory.c
index 1d318d7240b7..be0cba3b61ab 100644
--- a/drivers/accel/habanalabs/common/memory.c
+++ b/drivers/accel/habanalabs/common/memory.c
@@ -235,10 +235,8 @@ static int dma_map_host_va(struct hl_device *hdev, u64 
addr, u64 size,
        }
 
        rc = hl_pin_host_memory(hdev, addr, size, userptr);
-       if (rc) {
-               dev_err(hdev->dev, "Failed to pin host memory\n");
+       if (rc)
                goto pin_err;
-       }
 
        userptr->dma_mapped = true;
        userptr->dir = DMA_BIDIRECTIONAL;
@@ -1097,10 +1095,8 @@ static int map_device_va(struct hl_ctx *ctx, struct 
hl_mem_in *args, u64 *device
                        huge_page_size = hdev->asic_prop.pmmu_huge.page_size;
 
                rc = dma_map_host_va(hdev, addr, size, &userptr);
-               if (rc) {
-                       dev_err(hdev->dev, "failed to get userptr from va\n");
+               if (rc)
                        return rc;
-               }
 
                rc = init_phys_pg_pack_from_userptr(ctx, userptr,
                                &phys_pg_pack, false);
-- 
2.25.1

Reply via email to