diff options
| author | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2022-09-30 16:14:47 +0200 |
|---|---|---|
| committer | Hans Verkuil <hverkuil-cisco@xs4all.nl> | 2022-10-24 19:03:53 +0200 |
| commit | 5e1a33759a57b5c075187b6f12f0f57c964ed1c4 (patch) | |
| tree | 1ecb3e6d7f3039e846eff6306166022e2419918b /drivers/staging/media/meson/vdec | |
| parent | 3984ea32e83bcad06b4b034ddd4b0a934c1b2f91 (diff) | |
media: meson/vdec: always init coef_node_start
It's hard for code analyzers to see that coef_node_start is
always initialized (and actually hard for humans as well!).
So change the last 'else if' to an 'else' so it is clear that
coef_node_startis always set.
This fixes a compile warning:
drivers/staging/media/meson/vdec/codec_vp9.c:1689:41: warning: 'coef_node_start' may be used uninitialized [-Wmaybe-uninitialized]
1689 | coef_node_start = coef_node_start + 1;
| ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
drivers/staging/media/meson/vdec/codec_vp9.c:1551:19: note: 'coef_node_start' was declared here
1551 | int node, coef_node_start, coef_count_node_start;
| ^~~~~~~~~~~~~~~
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Diffstat (limited to 'drivers/staging/media/meson/vdec')
| -rw-r--r-- | drivers/staging/media/meson/vdec/codec_vp9.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/staging/media/meson/vdec/codec_vp9.c b/drivers/staging/media/meson/vdec/codec_vp9.c index 897f5d7a6aad..cf0f58d982c8 100644 --- a/drivers/staging/media/meson/vdec/codec_vp9.c +++ b/drivers/staging/media/meson/vdec/codec_vp9.c @@ -1649,8 +1649,7 @@ static void adapt_coef_probs(int prev_kf, int cur_kf, int pre_fc, else if (coef_count_node_start == VP9_MV_BITS_1_COUNT_START) coef_node_start = VP9_MV_BITS_1_START; - else if (coef_count_node_start == - VP9_MV_CLASS0_HP_0_COUNT_START) + else /* node_start == VP9_MV_CLASS0_HP_0_COUNT_START */ coef_node_start = VP9_MV_CLASS0_HP_0_START; den = count[coef_count_node_start] + |