Commit 2c2a8dc3 authored by Shaobo He's avatar Shaobo He Committed by Mauro Carvalho Chehab

media: platform/sh_veu.c: remove redundant NULL pointer checks

Function `sh_veu_find_fmt` returns an address that is an addition of a
base pointer `sh_veu_fmt` and an offset. The base pointer refers to a
global variable of which address cannot be NULL. Therefore, this commit
removes the NULL pointer checks on the return values of function
`sh_veu_find_fmt`.
Signed-off-by: default avatarShaobo He <shaobo@cs.utah.edu>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent dea37a97
......@@ -493,9 +493,6 @@ static int sh_veu_try_fmt_vid_cap(struct file *file, void *priv,
const struct sh_veu_format *fmt;
fmt = sh_veu_find_fmt(f);
if (!fmt)
/* wrong buffer type */
return -EINVAL;
return sh_veu_try_fmt(f, fmt);
}
......@@ -506,9 +503,6 @@ static int sh_veu_try_fmt_vid_out(struct file *file, void *priv,
const struct sh_veu_format *fmt;
fmt = sh_veu_find_fmt(f);
if (!fmt)
/* wrong buffer type */
return -EINVAL;
return sh_veu_try_fmt(f, fmt);
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment