Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 714ec136 authored by Android (Google) Code Review's avatar Android (Google) Code Review
Browse files

Merge change 24742 into eclair 

* changes:
  Apparently I must not use OMX_UseBuffer on either input or output ports on any of the TI components... or else.
parents 0fdae4cc 2dc64d80

media/libstagefright/OMXCodec.cpp

+12 −2
Original line number Diff line number Diff line
@@ -232,6 +232,16 @@ sp<OMXCodec> OMXCodec::Create( 
        quirks |= kRequiresAllocateBufferOnOutputPorts;

    }



    if (!strncmp(componentName, "OMX.TI.", 7)) {

        // Apparently I must not use OMX_UseBuffer on either input or

        // output ports on any of the TI components or quote:

        // "(I) may have unexpected problem (sic) which can be timing related

        //  and hard to reproduce."



        quirks |= kRequiresAllocateBufferOnInputPorts;

        quirks |= kRequiresAllocateBufferOnOutputPorts;

    }



    sp<OMXCodec> codec = new OMXCodec(

            omx, node, quirks, createEncoder, mime, componentName,

            source);
@@ -840,8 +850,8 @@ status_t OMXCodec::allocateBuffersOnPort(OMX_U32 portIndex) { 
                    mNode, portIndex, mem, &buffer);

        } else if (portIndex == kPortIndexOutput

                && (mQuirks & kRequiresAllocateBufferOnOutputPorts)) {

            err = mOMX->allocate_buffer(

                    mNode, portIndex, def.nBufferSize, &buffer);

            err = mOMX->allocate_buffer_with_backup(

                    mNode, portIndex, mem, &buffer);

        } else {

            err = mOMX->use_buffer(mNode, portIndex, mem, &buffer);

        }