Am 01.02.2016 um 19:04 schrieb Leo Liu:
On 02/01/2016 12:59 PM, Ilia Mirkin wrote:
On Mon, Feb 1, 2016 at 12:35 PM, Leo Liu <leo....@amd.com> wrote:
The scaling list should be filled out with zig zag scan
v2: integrate zig zag scan for list 4x4 to vl(Christian)
Cc: "11.0 11.1" <mesa-sta...@lists.freedesktop.org>
Signed-off-by: Leo Liu <leo....@amd.com>
---
src/gallium/state_trackers/omx/vid_dec_h264.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_trackers/omx/vid_dec_h264.c
b/src/gallium/state_trackers/omx/vid_dec_h264.c
index f66ed89..f17be91 100644
--- a/src/gallium/state_trackers/omx/vid_dec_h264.c
+++ b/src/gallium/state_trackers/omx/vid_dec_h264.c
@@ -35,6 +35,7 @@
#include "util/u_memory.h"
#include "util/u_video.h"
#include "vl/vl_rbsp.h"
+#include "vl/vl_zscan.h"
#include "entrypoint.h"
#include "vid_dec.h"
@@ -215,6 +216,7 @@ static void scaling_list(struct vl_rbsp *rbsp,
uint8_t *scalingList, unsigned si
}
for (i = 0; i < sizeOfScalingList; ++i ) {
+ unsigned scan = (sizeOfScalingList == 16) ?
vl_zscan_normal_16[i] : vl_zscan_normal[i];
Perhaps determine the list just once rather than on each loop iter?
Also (and more importantly) it appears like you forgot to add
vl_zscan_normal_16?
patch 1 is sent, not sure why it haven't arrived.
Yeah, patch #1 arrived a bit later for me as well. No idea why.
I think we can leave optimizing the away which list to use to the
compiler, patches are Reviewed-by: Christian König
<christian.koe...@amd.com>
Regards,
Christian.
if (nextScale != 0) {
signed delta_scale = vl_rbsp_se(rbsp);
@@ -224,8 +226,8 @@ static void scaling_list(struct vl_rbsp *rbsp,
uint8_t *scalingList, unsigned si
return;
}
}
- scalingList[i] = nextScale == 0 ? lastScale : nextScale;
- lastScale = scalingList[i];
+ scalingList[scan] = nextScale == 0 ? lastScale : nextScale;
+ lastScale = scalingList[scan];
}
}
--
1.9.1
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
