Commit 4edfa4f5 authored by Martin Storsjö's avatar Martin Storsjö

omxil: Don't require SetAudioParameters on the decoder output port to succeed

It's mostly enough to call it on the input port, while we later get
back what the decoder itself set on the output port (based on the
input port and codec config data), so if this fails, just ignore it.

This is required for using google SW audio codecs via OMX, which is
useful mostly for testing. (Using them still requires a few more
hacks though.)
Signed-off-by: default avatarMartin Storsjö <martin@martin.st>
parent 3e91dcf9
......@@ -418,8 +418,14 @@ static OMX_ERRORTYPE SetPortDefinition(decoder_t *p_dec, OmxPort *p_port,
p_fmt->i_bitrate,
p_fmt->audio.i_bitspersample,
p_fmt->audio.i_blockalign);
CHECK_ERROR(omx_error, "SetAudioParameters failed (%x : %s)",
omx_error, ErrorToString(omx_error));
if (def->eDir == OMX_DirInput) {
CHECK_ERROR(omx_error, "SetAudioParameters failed (%x : %s)",
omx_error, ErrorToString(omx_error));
} else {
msg_Warn(p_dec, "SetAudioParameters failed (%x : %s) on output port",
omx_error, ErrorToString(omx_error));
omx_error = OMX_ErrorNone;
}
}
if (!strcmp(p_dec->p_sys->psz_component, "OMX.TI.DUCATI1.VIDEO.DECODER") &&
def->eDir == OMX_DirOutput)
......
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