diff options
author | Sachin Mohan Gadag <sgadag@codeaurora.org> | 2017-10-30 19:14:52 +0530 |
---|---|---|
committer | Arne Coucheron <arco68@gmail.com> | 2019-12-04 14:20:31 +0100 |
commit | 1c7bc5c3146cd703c5ab42c56199acde208074a7 (patch) | |
tree | 4ccc4824898ab62e632cc8b5e4d65dd91d7d4fc2 | |
parent | 93ec5ef40038b1b233c4431f3b9c5dd8d8101133 (diff) | |
download | android_hardware_qcom_audio-1c7bc5c3146cd703c5ab42c56199acde208074a7.tar.gz android_hardware_qcom_audio-1c7bc5c3146cd703c5ab42c56199acde208074a7.tar.bz2 android_hardware_qcom_audio-1c7bc5c3146cd703c5ab42c56199acde208074a7.zip |
mm-audio: Make repetitive logs silent
Define NDEBUG Macro in Makefile to avoid
ALOGV logs to be printed on shell.
Move repetitive logs under ALOGV log level.
CRs-Fixed: 2135642
Change-Id: Id573300504cdb15aea285ec33a08319155cff189
-rw-r--r-- | mm-audio/aenc-aac/qdsp6/Makefile.am | 3 | ||||
-rw-r--r-- | mm-audio/aenc-aac/qdsp6/inc/omx_aac_aenc.h | 2 | ||||
-rw-r--r-- | mm-audio/aenc-amrnb/qdsp6/inc/omx_amr_aenc.h | 2 | ||||
-rw-r--r-- | mm-audio/aenc-amrnb/qdsp6/src/omx_amr_aenc.cpp | 30 |
4 files changed, 19 insertions, 18 deletions
diff --git a/mm-audio/aenc-aac/qdsp6/Makefile.am b/mm-audio/aenc-aac/qdsp6/Makefile.am index 12800a95..21428b5e 100644 --- a/mm-audio/aenc-aac/qdsp6/Makefile.am +++ b/mm-audio/aenc-aac/qdsp6/Makefile.am @@ -5,6 +5,7 @@ AM_CFLAGS += -Wundef AM_CFLAGS += -Wstrict-prototypes AM_CFLAGS += -Wno-trigraphs AM_CFLAGS += -g -O3 +AM_CFLAGS += -DNDEBUG AM_CPPFLAGS = -D__packed__= AM_CPPFLAGS += -DIMAGE_APPS_PROC @@ -13,7 +14,7 @@ AM_CPPFLAGS += -DFEATURE_Q_NO_SELF_QPTR AM_CPPFLAGS += -DFEATURE_LINUX AM_CPPFLAGS += -DFEATURE_NATIVELINUX AM_CPPFLAGS += -DFEATURE_DSM_DUP_ITEMS -AM_CPPFLAGS += -D_DEBUG +AM_CPPFLAGS += -DNDEBUG AM_CPPFLAGS += -Iinc c_sources =src/omx_aac_aenc.cpp diff --git a/mm-audio/aenc-aac/qdsp6/inc/omx_aac_aenc.h b/mm-audio/aenc-aac/qdsp6/inc/omx_aac_aenc.h index 374611a9..2fdf22ac 100644 --- a/mm-audio/aenc-aac/qdsp6/inc/omx_aac_aenc.h +++ b/mm-audio/aenc-aac/qdsp6/inc/omx_aac_aenc.h @@ -81,7 +81,7 @@ extern "C" { #define PrintFrameHdr(i,bufHdr) \ - DEBUG_PRINT("i=%d OMX bufHdr[%p]buf[%p]size[%d]TS[%lld]nFlags[0x%x]\n",\ + DEBUG_DETAIL("i=%d OMX bufHdr[%p]buf[%p]size[%d]TS[%lld]nFlags[0x%x]\n",\ i,\ bufHdr, \ ((OMX_BUFFERHEADERTYPE *)bufHdr)->pBuffer, \ diff --git a/mm-audio/aenc-amrnb/qdsp6/inc/omx_amr_aenc.h b/mm-audio/aenc-amrnb/qdsp6/inc/omx_amr_aenc.h index 8236c031..54935c7c 100644 --- a/mm-audio/aenc-amrnb/qdsp6/inc/omx_amr_aenc.h +++ b/mm-audio/aenc-amrnb/qdsp6/inc/omx_amr_aenc.h @@ -84,7 +84,7 @@ extern "C" { #define PrintFrameHdr(i,bufHdr) \ - DEBUG_PRINT("i=%d OMX bufHdr[%p]buf[%p]size[%d]TS[%lld]nFlags[0x%x]\n",\ + DEBUG_DETAIL("i=%d OMX bufHdr[%p]buf[%p]size[%d]TS[%lld]nFlags[0x%x]\n",\ i,\ bufHdr, \ ((OMX_BUFFERHEADERTYPE *)bufHdr)->pBuffer, \ diff --git a/mm-audio/aenc-amrnb/qdsp6/src/omx_amr_aenc.cpp b/mm-audio/aenc-amrnb/qdsp6/src/omx_amr_aenc.cpp index 5532515f..47b984c2 100644 --- a/mm-audio/aenc-amrnb/qdsp6/src/omx_amr_aenc.cpp +++ b/mm-audio/aenc-amrnb/qdsp6/src/omx_amr_aenc.cpp @@ -512,13 +512,13 @@ void omx_amr_aenc::frame_done_cb(OMX_BUFFERHEADERTYPE *bufHdr) m_amr_pb_stats.fbd_cnt++; pthread_mutex_lock(&out_buf_count_lock); nNumOutputBuf--; - DEBUG_PRINT("FBD CB:: nNumOutputBuf=%d out_buf_len=%u fbd_cnt=%u\n",\ + DEBUG_DETAIL("FBD CB:: nNumOutputBuf=%d out_buf_len=%u fbd_cnt=%u\n",\ nNumOutputBuf, m_amr_pb_stats.tot_out_buf_len, m_amr_pb_stats.fbd_cnt); m_amr_pb_stats.tot_out_buf_len += bufHdr->nFilledLen; m_amr_pb_stats.tot_pb_time = bufHdr->nTimeStamp; - DEBUG_PRINT("FBD:in_buf_len=%u out_buf_len=%u\n", + DEBUG_DETAIL("FBD:in_buf_len=%u out_buf_len=%u\n", m_amr_pb_stats.tot_in_buf_len, m_amr_pb_stats.tot_out_buf_len); @@ -3233,7 +3233,7 @@ OMX_ERRORTYPE omx_amr_aenc::get_state(OMX_IN OMX_HANDLETYPE hComp, return OMX_ErrorBadParameter; } *state = m_state; - DEBUG_PRINT("Returning the state %d\n",*state); + DEBUG_DETAIL("Returning the state %d\n",*state); return OMX_ErrorNone; } @@ -3975,12 +3975,12 @@ OMX_ERRORTYPE omx_amr_aenc::free_buffer(OMX_IN OMX_HANDLETYPE hComp, @return error status */ OMX_ERRORTYPE omx_amr_aenc::empty_this_buffer( - OMX_IN OMX_HANDLETYPE hComp, - OMX_IN OMX_BUFFERHEADERTYPE* buffer) + OMX_IN OMX_HANDLETYPE hComp, + OMX_IN OMX_BUFFERHEADERTYPE* buffer) { OMX_ERRORTYPE eRet = OMX_ErrorNone; - DEBUG_PRINT("ETB:Buf:%p Len %u TS %lld numInBuf=%d\n", \ + DEBUG_DETAIL("ETB:Buf:%p Len %u TS %lld numInBuf=%d\n", \ buffer, buffer->nFilledLen, buffer->nTimeStamp, (nNumInputBuf)); if (m_state == OMX_StateInvalid) { @@ -4076,7 +4076,7 @@ OMX_ERRORTYPE omx_amr_aenc::empty_this_buffer_proxy meta_in.nFlags |= OMX_BUFFERFLAG_EOS; } memcpy(data,&meta_in, meta_in.offsetVal); - DEBUG_PRINT("meta_in.nFlags = %d\n",meta_in.nFlags); + DEBUG_DETAIL("meta_in.nFlags = %d\n",meta_in.nFlags); } else { DEBUG_PRINT_ERROR("temp meta is null buf\n"); return OMX_ErrorInsufficientResources; @@ -4119,7 +4119,7 @@ OMX_ERRORTYPE omx_amr_aenc::fill_this_buffer_proxy if (true == search_output_bufhdr(buffer)) { - DEBUG_PRINT("\nBefore Read..m_drv_fd = %d,\n",m_drv_fd); + DEBUG_DETAIL("\nBefore Read..m_drv_fd = %d,\n",m_drv_fd); nReadbytes = read(m_drv_fd,buffer->pBuffer,output_buffer_size ); DEBUG_DETAIL("FTBP->Al_len[%lu]buf[%p]size[%d]numOutBuf[%d]\n",\ buffer->nAllocLen,buffer->pBuffer, @@ -4131,7 +4131,7 @@ OMX_ERRORTYPE omx_amr_aenc::fill_this_buffer_proxy frame_done_cb((OMX_BUFFERHEADERTYPE *)buffer); return OMX_ErrorNone; } else - DEBUG_PRINT("Read bytes %d\n",nReadbytes); + DEBUG_DETAIL("Read bytes %d\n",nReadbytes); // Buffer from Driver will have // 1 byte => Nr of frame field // (sizeof(ENC_META_OUT) * Nr of frame) bytes => meta_out->offset_to_frame @@ -4139,7 +4139,7 @@ OMX_ERRORTYPE omx_amr_aenc::fill_this_buffer_proxy meta_out = (ENC_META_OUT *)(buffer->pBuffer + sizeof(unsigned char)); buffer->nTimeStamp = (((OMX_TICKS)meta_out->msw_ts << 32)+ - meta_out->lsw_ts); + meta_out->lsw_ts); buffer->nFlags |= meta_out->nflags; buffer->nOffset = (OMX_U32)(meta_out->offset_to_frame + sizeof(unsigned char)); @@ -4147,9 +4147,9 @@ OMX_ERRORTYPE omx_amr_aenc::fill_this_buffer_proxy ts += FRAMEDURATION; buffer->nTimeStamp = ts; nTimestamp = buffer->nTimeStamp; - DEBUG_PRINT("nflags %d frame_size %d offset_to_frame %d \ - timestamp %lld\n", meta_out->nflags, meta_out->frame_size, - meta_out->offset_to_frame, buffer->nTimeStamp); + DEBUG_DETAIL("nflags %d frame_size %d offset_to_frame %d \ + timestamp %lld\n", meta_out->nflags, meta_out->frame_size, + meta_out->offset_to_frame, buffer->nTimeStamp); if ((buffer->nFlags & OMX_BUFFERFLAG_EOS) == OMX_BUFFERFLAG_EOS ) { @@ -4168,7 +4168,7 @@ OMX_ERRORTYPE omx_amr_aenc::fill_this_buffer_proxy return OMX_ErrorNone; } - DEBUG_PRINT("nState %d \n",nState ); + DEBUG_DETAIL("nState %d \n",nState ); pthread_mutex_lock(&m_state_lock); get_state(&m_cmp, &state); @@ -4588,7 +4588,7 @@ RETURN VALUE ========================================================================== */ bool omx_amr_aenc::release_done(OMX_U32 param1) { - DEBUG_PRINT("Inside omx_amr_aenc::release_done"); + DEBUG_DETAIL("Inside omx_amr_aenc::release_done"); OMX_BOOL bRet = OMX_FALSE; if (param1 == OMX_ALL) |