diff options
author | Colin Ian King | 2017-10-29 14:43:39 +0100 |
---|---|---|
committer | Mauro Carvalho Chehab | 2017-10-31 12:27:58 +0100 |
commit | bf3881707b6f6350c3f8f8ff15524b6d719847bc (patch) | |
tree | e341d9c068529650a16f86b2c7376af166ea6ba3 | |
parent | media: gspca: remove redundant assignment to variable j (diff) | |
download | kernel-qcow2-linux-bf3881707b6f6350c3f8f8ff15524b6d719847bc.tar.gz kernel-qcow2-linux-bf3881707b6f6350c3f8f8ff15524b6d719847bc.tar.xz kernel-qcow2-linux-bf3881707b6f6350c3f8f8ff15524b6d719847bc.zip |
media: bdisp: remove redundant assignment to pix
Pointer pix is being initialized to a value and a little later
being assigned the same value again. Remove the initial assignment to
avoid a duplicate assignment. Cleans up the clang warning:
drivers/media/platform/sti/bdisp/bdisp-v4l2.c:726:26: warning: Value
stored to 'pix' during its initialization is never read
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
-rw-r--r-- | drivers/media/platform/sti/bdisp/bdisp-v4l2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/platform/sti/bdisp/bdisp-v4l2.c b/drivers/media/platform/sti/bdisp/bdisp-v4l2.c index 939da6da7644..7e9ed9c7b3e1 100644 --- a/drivers/media/platform/sti/bdisp/bdisp-v4l2.c +++ b/drivers/media/platform/sti/bdisp/bdisp-v4l2.c @@ -723,7 +723,7 @@ static int bdisp_enum_fmt(struct file *file, void *fh, struct v4l2_fmtdesc *f) static int bdisp_g_fmt(struct file *file, void *fh, struct v4l2_format *f) { struct bdisp_ctx *ctx = fh_to_ctx(fh); - struct v4l2_pix_format *pix = &f->fmt.pix; + struct v4l2_pix_format *pix; struct bdisp_frame *frame = ctx_get_frame(ctx, f->type); if (IS_ERR(frame)) { |