Commit 631093e2 authored by Martin Storsjö's avatar Martin Storsjö Committed by Jean-Baptiste Kempf
Browse files

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's avatarJean-Baptiste Kempf <>
parent c73e88ce
......@@ -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" },
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment