[Gstreamer-openmax] Support for hardware codecs

[Felipe Contreras]

On Wed, Feb 24, 2010 at 4:10 PM, Rob Clark <rob at ti.com> wrote:
> On Feb 24, 2010, at 6:41 AM, Felipe Contreras wrote:
>> I think rank should also be optional.
>
> ok, should default rank=GST_RANK_NONE?

That was my idea, yeah.

>> Anyway, the only important comment I have for now is that I think the
>> type should also be configurable (you can use g_type_from_name())
>>
>> I'm thinking something like:
>>
>> ---
>> omx_mp3dec_ti:
>>       type: GstOmxMp3Dec
>>       library-name: libOMX_Core.so.0
>>       component-name: OMX.TI.AUDIO.DECODE
>>       component-role: audio_decode.dsp.mp3
>>       rank: 256
>>
>> omx_mp3dec_nokia:
>>       type: GstOmxMp3Dec
>>       library-name: libomxil_bellagio.so.0
>>       component-name: OMX.nokia.audio_decode.mp3
>>       rank: 128
>> ---
>
> ok, that seems reasonable..  are you ok w/ basically same thing in GstStructure syntax:
>
> omx_mp3dec_ti,
>        type=GstOmxMp3Dec,
>        library-name=libOMX_Core.so.0,
>        component-name=OMX.TI.AUDIO.DECODE,
>        component-role=audio_decode.dsp.mp3,
>        rank=256;
>
> omx_mp3dec_nokia,
>        type=GstOmxMp3Dec,
>        library-name=libomxil_bellagio.so.0,
>        component-name=OMX.nokia.audio_decode.mp3,
>        rank=128;

Yeah, whatever is easier.

>> Oh, and I think the default configuration should be in a separate file.
>
> ok, I'll do that..  expect an updated patch, rebased on latest gst-openmax master so you can more easily apply the patch and try it out

Cool :)

-- 
Felipe Contreras