diff options
author | Gwenole Beauchesne <gb.devel@gmail.com> | 2014-04-08 06:56:03 -0600 |
---|---|---|
committer | Zhao Yakui <yakui.zhao@intel.com> | 2014-04-09 11:25:17 +0800 |
commit | 36ccd9c3e47766edc70ecbdf82acc89ed67e26c4 (patch) | |
tree | 6aacd66dc0d64099f328df1f1289aa98de9172b0 | |
parent | 150f67c67bd92cd201b75a92388fe3a63b00cd8a (diff) |
vp8: fix loop filter for bitexact reconstruction.
Each loop filter delta update value shall be encoded within 7 bits,
including the sign bit and 6-bit magnitude in 2's complement. So,
don't propagate the sign bit while packing the filter level values.
Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>
-rw-r--r-- | src/gen8_mfd.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/gen8_mfd.c b/src/gen8_mfd.c index 276247e..c04c887 100644 --- a/src/gen8_mfd.c +++ b/src/gen8_mfd.c @@ -2930,16 +2930,16 @@ gen8_mfd_vp8_pic_state(VADriverContextP ctx, } OUT_BCS_BATCH(batch, - pic_param->loop_filter_deltas_ref_frame[3] << 24 | - pic_param->loop_filter_deltas_ref_frame[2] << 16 | - pic_param->loop_filter_deltas_ref_frame[1] << 8 | - pic_param->loop_filter_deltas_ref_frame[0] << 0); + (pic_param->loop_filter_deltas_ref_frame[3] & 0x7f) << 24 | + (pic_param->loop_filter_deltas_ref_frame[2] & 0x7f) << 16 | + (pic_param->loop_filter_deltas_ref_frame[1] & 0x7f) << 8 | + (pic_param->loop_filter_deltas_ref_frame[0] & 0x7f) << 0); OUT_BCS_BATCH(batch, - pic_param->loop_filter_deltas_mode[3] << 24 | - pic_param->loop_filter_deltas_mode[2] << 16 | - pic_param->loop_filter_deltas_mode[1] << 8 | - pic_param->loop_filter_deltas_mode[0] << 0); + (pic_param->loop_filter_deltas_mode[3] & 0x7f) << 24 | + (pic_param->loop_filter_deltas_mode[2] & 0x7f) << 16 | + (pic_param->loop_filter_deltas_mode[1] & 0x7f) << 8 | + (pic_param->loop_filter_deltas_mode[0] & 0x7f) << 0); /* segmentation id stream base address, DW35-DW37 */ OUT_BCS_BATCH(batch, 0); |