Commit: 6c2765a53496384cd7f27af3935d7a2fee7e8bba - hardware-intel-libva (git)

hardware/intel/libva

Commit MetaInfo

• Revision: 6c2765a53496384cd7f27af3935d7a2fee7e8bba (tree)
• Time: 2012-04-08 14:14:50
• Author: Gwenole Beauchesne <gwenole.beauchesne@inte...>
• Commiter: Gwenole Beauchesne

Log Message

API: h264 decode: fix the definition of slice_data_bit_offset.

Align with the existing practice in various media stacks available
in Linux today. i.e. codec layer submits slice data that represents
the original bitstream, thus including any emulation prevent bytes.
However, the slice_data_bit_offset represents an offset relative
to the buffer that contains the NAL unit byte and the rest of the
slice with any emulation prevention bytes removed.

Signed-off-by: Gwenole Beauchesne <[e-mail]>

Change Summary

• modified: va/va.h

Incremental Difference

--- a/va/va.h
+++ b/va/va.h
@@ -1227,12 +1227,12 @@ typedef struct _VASliceParameterBufferH264
     /**
      * \brief Bit offset from NAL Header Unit to the begining of slice_data().
      *
-     * This bit offset is relative from the NAL unit byte to the first
-     * bit of slice_data(), thus including any emulation prevention
-     * bytes in slice_header().
-     *
-     * Basically, this field represents the number of bits parsed in
-     * the slice_header() + 8 for the initial NAL unit byte.
+     * This bit offset is relative to and includes the NAL unit byte
+     * and represents the number of bits parsed in the slice_header()
+     * after the removal of any emulation prevention bytes in
+     * there. However, the slice data buffer passed to the hardware is
+     * the original bitstream, thus including any emulation prevention
+     * bytes.
      */
     unsigned short slice_data_bit_offset;
     unsigned short first_mb_in_slice;