media: staging: imx: remove unnecessary out-of-memory error message

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 <shyamsunderreddypadira@gmail.com>
Signed-off-by: Frank Li <Frank.Li@nxp.com>
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>

diff --git a/drivers/staging/media/imx/imx-media-utils.c b/drivers/staging/media/imx/imx-media-utils.c
index 1a2e5e40c99cf883ad8b5fb24307a48fc8e2a8f7..f119477cac6b16f1f25bb2d3f15dd65fa151fdd1 100644 (file)
--- 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;
 }