summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSachin Mohan Gadag <sgadag@codeaurora.org>2017-10-30 19:14:52 +0530
committerArne Coucheron <arco68@gmail.com>2019-12-04 14:20:31 +0100
commit1c7bc5c3146cd703c5ab42c56199acde208074a7 (patch)
tree4ccc4824898ab62e632cc8b5e4d65dd91d7d4fc2
parent93ec5ef40038b1b233c4431f3b9c5dd8d8101133 (diff)
downloadandroid_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.am3
-rw-r--r--mm-audio/aenc-aac/qdsp6/inc/omx_aac_aenc.h2
-rw-r--r--mm-audio/aenc-amrnb/qdsp6/inc/omx_amr_aenc.h2
-rw-r--r--mm-audio/aenc-amrnb/qdsp6/src/omx_amr_aenc.cpp30
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)