[vlc-commits] omxil: Look for the role video_decoder.wmv for VC1

Martin Storsjö git at videolan.org
Wed Sep 19 14:49:01 CEST 2012


vlc | branch: master | Martin Storsjö <martin at martin.st> | Wed Sep 19 12:54:45 2012 +0300| [631093e249a8c5e97163dbd8822f2bf66d909c74] | committer: Jean-Baptiste Kempf

omxil: Look for the role video_decoder.wmv for VC1

On the Galaxy S3 at least, the component OMX.SEC.vc1.dec only
exposes this role, not video_decoder.vc1.

The role name video_decoder.vc1 is pretty recent (from OMX IL 1.2),
and in that spec version the coding field should be OMX_VIDEO_CodingVC1
instead of OMX_VIDEO_CodingWMV as it currently is. The omxil module
currently only uses OMX IL 1.1 headers, and so does Android.

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=631093e249a8c5e97163dbd8822f2bf66d909c74
---

 modules/codec/omxil/utils.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/codec/omxil/utils.c b/modules/codec/omxil/utils.c
index 15db2da..c1e5fba 100644
--- a/modules/codec/omxil/utils.c
+++ b/modules/codec/omxil/utils.c
@@ -281,7 +281,7 @@ static const struct
     { VLC_CODEC_WMV1, OMX_VIDEO_CodingWMV,   "video_decoder.wmv"   },
     { VLC_CODEC_WMV2, OMX_VIDEO_CodingWMV,   "video_decoder.wmv"   },
     { VLC_CODEC_WMV3, OMX_VIDEO_CodingWMV,   "video_decoder.wmv"   },
-    { VLC_CODEC_VC1,  OMX_VIDEO_CodingWMV,   "video_decoder.vc1"   },
+    { VLC_CODEC_VC1,  OMX_VIDEO_CodingWMV,   "video_decoder.wmv"   },
     { VLC_CODEC_MJPG, OMX_VIDEO_CodingMJPEG, "video_decoder.jpeg"  },
     { VLC_CODEC_MJPG, OMX_VIDEO_CodingMJPEG, "video_decoder.mjpeg" },
     { VLC_CODEC_RV10, OMX_VIDEO_CodingRV,    "video_decoder.rv"    },



More information about the vlc-commits mailing list