On Sun May 3 20:30:26 2026 +0530, Shyam Sunder Reddy Padira wrote:
> Remove dev_err() call after dma_alloc_coherent() failure.
>
> checkpatch.pl reports this as an unnecessary out-of-memory message because
> failure is already conveyed by returning -ENOMEM, and the current message
> does not provide additional useful debugging information.
>
> Signed-off-by: Shyam Sunder Reddy Padira <[email protected]>
> Signed-off-by: Frank Li <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>
Patch committed.
Thanks,
Hans Verkuil
drivers/staging/media/imx/imx-media-utils.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
---
diff --git a/drivers/staging/media/imx/imx-media-utils.c
b/drivers/staging/media/imx/imx-media-utils.c
index 1a2e5e40c99c..f119477cac6b 100644
--- a/drivers/staging/media/imx/imx-media-utils.c
+++ b/drivers/staging/media/imx/imx-media-utils.c
@@ -589,10 +589,8 @@ int imx_media_alloc_dma_buf(struct device *dev,
buf->len = PAGE_ALIGN(size);
buf->virt = dma_alloc_coherent(dev, buf->len, &buf->phys,
GFP_DMA | GFP_KERNEL);
- if (!buf->virt) {
- dev_err(dev, "%s: failed\n", __func__);
+ if (!buf->virt)
return -ENOMEM;
- }
return 0;
}
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]
