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

Commit 0eb2f24b authored by Keith Mok's avatar Keith Mok
Browse files

stagefright: Fix cts issue with divx test

8909 is using soft-ip divx decoder using component
name starts with "OMX.ittiam." instead of "OMX.qcom."
leading to FFMPEGSoftCodec override with wrong compression
format using OMX_VIDEO_CodingDIVX (0x7F000003) defined in
OMX_FFMPEG_Extn.h. But qcom omx component actually use
QOMX_VIDEO_CodingDivx (0x7FA30C02) which defined in OMX_QCOMExtns.h.

Add checking for OMX.ittiam in additional to OMX.qcom solved the
problem.

FEIJ-700

Change-Id: I4071eb3978974d1072373f0c88ea83653c0c53a9
parent 8436846a
Loading
Loading
Loading
Loading
+2 −1
Original line number Diff line number Diff line
@@ -319,7 +319,8 @@ status_t FFMPEGSoftCodec::setVideoFormat(
    // from the CAF L release. It was unfortunately moved to a proprietary
    // blob and an architecture which is hellish for OEMs who wish to
    // customize the platform.
    if (err == OK && (!strncmp(componentName, "OMX.qcom.", 9))) {
    if (err == OK && (!strncmp(componentName, "OMX.qcom.", 9)
        || !strncmp(componentName, "OMX.ittiam.", 11))) {
        status_t xerr = OK;