omxil: Set the module priority to 0 for iomx

The iomx module is only enabled when the user has requested it
(via settings). Currently, when the iomx module has priority 80,
it is always preferred over avcodec.

For other platforms, where one would only enable omxil if one
explicitly wants to use it, this is ok, but for android, we
ship the iomx module but allow users to enable it at runtime
if they want to.

If it isn't enabled, the priorities are currently overridden by
adding "--codec avcodec,all" to the command line, but this makes
avcodec have precedence over other decoder modules that otherwise
would have higher priority (e.g. a52).

By setting the priority to 0 for the iomx version of this
module, we can more easily enable it only when asked for,
by adding "--codec iomx,all", without affecting the relative
priorities of all other codec modules.

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

diff --git a/modules/codec/omxil/omxil.c b/modules/codec/omxil/omxil.c
index 3c13a7366e78bc7e89a32377c32d241bc7b4afd6..ab811965fd4042854496d8c48757a8cfe300723c 100644 (file)
--- a/modules/codec/omxil/omxil.c
+++ b/modules/codec/omxil/omxil.c
@@ -109,7 +109,14 @@ vlc_module_begin ()
     set_category( CAT_INPUT )
     set_subcategory( SUBCAT_INPUT_VCODEC )
     set_section( N_("Decoding") , NULL )
+#if defined(USE_IOMX)
+    /* For IOMX, don't enable it automatically via priorities,
+     * enable it only via the --codec iomx command line parameter when
+     * wanted. */
+    set_capability( "decoder", 0 )
+#else
     set_capability( "decoder", 80 )
+#endif
     set_callbacks( OpenDecoder, CloseGeneric )
 
     add_submodule ()