diff options
author | James Dong <jdong@google.com> | 2012-05-07 12:29:52 -0700 |
---|---|---|
committer | James Dong <jdong@google.com> | 2012-05-07 12:29:52 -0700 |
commit | 7c25df82dfc8bbedb58608242f0d923a4594bb14 (patch) | |
tree | 465490eaf86b8e8716671f9bbb32d576aef23ca0 | |
parent | f53c36c6463d899895719b8b17ba45bd2d552cee (diff) |
Get rid of vendor specific video encoder component names from ACodec and OMXCodec
o related-to-bug: 6446245
Change-Id: I1fa07ad8a39337e3b19ac51c10533a2de8c11bb5
-rw-r--r-- | media/libstagefright/ACodec.cpp | 8 | ||||
-rwxr-xr-x | media/libstagefright/OMXCodec.cpp | 11 |
2 files changed, 7 insertions, 12 deletions
diff --git a/media/libstagefright/ACodec.cpp b/media/libstagefright/ACodec.cpp index 7d990640..1e00c5df 100644 --- a/media/libstagefright/ACodec.cpp +++ b/media/libstagefright/ACodec.cpp @@ -1715,7 +1715,9 @@ status_t ACodec::setupAVCEncoderParameters(const sp<AMessage> &msg) { } // XXX - if (!strncmp(mComponentName.c_str(), "OMX.TI.DUCATI1", 14)) { + if (h264type.eProfile != OMX_VIDEO_AVCProfileBaseline) { + ALOGW("Use baseline profile instead of %d for AVC recording", + h264type.eProfile); h264type.eProfile = OMX_VIDEO_AVCProfileBaseline; } @@ -1750,10 +1752,6 @@ status_t ACodec::setupAVCEncoderParameters(const sp<AMessage> &msg) { h264type.bMBAFF = OMX_FALSE; h264type.eLoopFilterMode = OMX_VIDEO_AVCLoopFilterEnable; - if (!strcasecmp("OMX.Nvidia.h264.encoder", mComponentName.c_str())) { - h264type.eLevel = OMX_VIDEO_AVCLevelMax; - } - err = mOMX->setParameter( mNode, OMX_IndexParamVideoAvc, &h264type, sizeof(h264type)); diff --git a/media/libstagefright/OMXCodec.cpp b/media/libstagefright/OMXCodec.cpp index 56016a8b..604d0e04 100755 --- a/media/libstagefright/OMXCodec.cpp +++ b/media/libstagefright/OMXCodec.cpp @@ -1123,9 +1123,10 @@ status_t OMXCodec::setupAVCEncoderParameters(const sp<MetaData>& meta) { h264type.eProfile = static_cast<OMX_VIDEO_AVCPROFILETYPE>(profileLevel.mProfile); h264type.eLevel = static_cast<OMX_VIDEO_AVCLEVELTYPE>(profileLevel.mLevel); - // FIXME: - // Remove the workaround after the work in done. - if (!strncmp(mComponentName, "OMX.TI.DUCATI1", 14)) { + // XXX + if (h264type.eProfile != OMX_VIDEO_AVCProfileBaseline) { + ALOGW("Use baseline profile instead of %d for AVC recording", + h264type.eProfile); h264type.eProfile = OMX_VIDEO_AVCProfileBaseline; } @@ -1160,10 +1161,6 @@ status_t OMXCodec::setupAVCEncoderParameters(const sp<MetaData>& meta) { h264type.bMBAFF = OMX_FALSE; h264type.eLoopFilterMode = OMX_VIDEO_AVCLoopFilterEnable; - if (!strcasecmp("OMX.Nvidia.h264.encoder", mComponentName)) { - h264type.eLevel = OMX_VIDEO_AVCLevelMax; - } - err = mOMX->setParameter( mNode, OMX_IndexParamVideoAvc, &h264type, sizeof(h264type)); CHECK_EQ(err, (status_t)OK); |