diff options
author | Tom Rix <trix@redhat.com> | 2021-08-18 21:41:05 +0200 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab+huawei@kernel.org> | 2021-09-30 10:07:55 +0200 |
commit | 749d896551df2ae6996aab51c25bbb3ecd589774 (patch) | |
tree | c775c90aca263d56b6191f0651b54726118c9750 /drivers/media/platform/mtk-vpu/mtk_vpu.c | |
parent | a9be3931188f240aa2dbdb7c9af65ecfb8c073c1 (diff) |
media: camss: vfe: simplify vfe_get_wm_sizes()
Static analysis reports this representative problem
camss-vfe-4-1.c:333: The result of the left shift is undefined because
the left operand is negative
reg |= (height - 1) << 4;
~~~~~~~~~ ^
The is a false positive. height is set in vfe_get_wm_sizes() which
has a switch statement without a default.
Reviewing the switch, the cases contain redundant assignments.
So simplify to assignments.
Signed-off-by: Tom Rix <trix@redhat.com>
Reviewed-by: Robert Foss <robert.foss@linaro.org>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Diffstat (limited to 'drivers/media/platform/mtk-vpu/mtk_vpu.c')
0 files changed, 0 insertions, 0 deletions