[vlc-devel] [PATCH 4/4] omxil: Look for the role video_decoder.wmv for VC1

[Martin Storsjö]

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.
---
 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 7e850aa..d652859 100644
--- a/modules/codec/omxil/utils.c
+++ b/modules/codec/omxil/utils.c
@@ -285,7 +285,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"    },
-- 
1.7.10