media: aspeed: fix an incorrect return code on buffer allocation failure
authorJae Hyun Yoo <jae.hyun.yoo@linux.intel.com>
Fri, 23 Aug 2019 21:29:57 +0000 (18:29 -0300)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Mon, 26 Aug 2019 13:46:29 +0000 (10:46 -0300)
It returns '0' even when a failure happens on jpeg buffer allocation
so this commit fixes the issue.

Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com>
Reviewed-by: Andrew Jeffery <andrew@aj.id.au>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/platform/aspeed-video.c

index f899ac3..94f97d9 100644 (file)
@@ -1624,6 +1624,7 @@ static int aspeed_video_init(struct aspeed_video *video)
        if (!aspeed_video_alloc_buf(video, &video->jpeg,
                                    VE_JPEG_HEADER_SIZE)) {
                dev_err(dev, "Failed to allocate DMA for JPEG header\n");
+               rc = -ENOMEM;
                goto err_release_reserved_mem;
        }