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
diff --git a/hal/Makefile.am b/hal/Makefile.am
index 0096bf7..8ab3e7c 100644
--- a/hal/Makefile.am
+++ b/hal/Makefile.am
@@ -215,4 +215,5 @@
audio_primary_default_la_CFLAGS += -DINT_MAX=0x7fffffff
audio_primary_default_la_CFLAGS += -D__unused=__attribute__\(\(__unused__\)\)
audio_primary_default_la_CFLAGS += -DLINUX_ENABLED $(TARGET_CFLAGS) -DAUDIO_EXTN_FORMATS_ENABLED
+audio_primary_default_la_CFLAGS += -DNDEBUG
audio_primary_default_la_LDFLAGS = -module -shared -avoid-version
diff --git a/mm-audio/aenc-aac/qdsp6/Makefile.am b/mm-audio/aenc-aac/qdsp6/Makefile.am
index a79ce70..08c9bee 100644
--- a/mm-audio/aenc-aac/qdsp6/Makefile.am
+++ b/mm-audio/aenc-aac/qdsp6/Makefile.am
@@ -5,6 +5,7 @@
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_LINUX
AM_CPPFLAGS += -DFEATURE_NATIVELINUX
AM_CPPFLAGS += -DFEATURE_DSM_DUP_ITEMS
-AM_CPPFLAGS += -D_DEBUG
+AM_CPPFLAGS += -DNDEBUG
AM_CPPFLAGS += -Iinc
AM_CPPFLAGS += -I ${WORKSPACE}/hardware/qcom/media/mm-core/inc/
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 6041ffe..38e1d9a 100644
--- a/mm-audio/aenc-aac/qdsp6/inc/omx_aac_aenc.h
+++ b/mm-audio/aenc-aac/qdsp6/inc/omx_aac_aenc.h
@@ -82,7 +82,7 @@
#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/Makefile.am b/mm-audio/aenc-amrnb/qdsp6/Makefile.am
index 13379a3..8becd07 100644
--- a/mm-audio/aenc-amrnb/qdsp6/Makefile.am
+++ b/mm-audio/aenc-amrnb/qdsp6/Makefile.am
@@ -19,7 +19,7 @@
AM_CPPFLAGS += -I ${WORKSPACE}/hardware/qcom/media/mm-core/inc/
AM_CPPFLAGS += -g
-AM_CPPFLAGS += -D_DEBUG
+AM_CPPFLAGS += -DNDEBUG
AM_CPPFLAGS += -Iinc
c_sources = src/omx_amr_aenc.cpp \
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 8236c03..54935c7 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 @@
#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 5532515..47b984c 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 @@
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 @@
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 @@
@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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
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 @@
========================================================================== */
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)
diff --git a/mm-audio/aenc-g711/qdsp6/Makefile.am b/mm-audio/aenc-g711/qdsp6/Makefile.am
index f5ccca8..02b0d57 100644
--- a/mm-audio/aenc-g711/qdsp6/Makefile.am
+++ b/mm-audio/aenc-g711/qdsp6/Makefile.am
@@ -8,7 +8,7 @@
-D_ANDROID_ \
-D_ENABLE_QC_MSG_LOG_ \
-DVERBOSE \
- -D_DEBUG \
+ -DNDEBUG \
-DAUDIOV2 \
-I inc \
-I ${WORKSPACE}/hardware/qcom/media/mm-core/inc/
diff --git a/mm-audio/aenc-g711/qdsp6/inc/omx_g711_aenc.h b/mm-audio/aenc-g711/qdsp6/inc/omx_g711_aenc.h
index 4c8be66..0f415df 100644
--- a/mm-audio/aenc-g711/qdsp6/inc/omx_g711_aenc.h
+++ b/mm-audio/aenc-g711/qdsp6/inc/omx_g711_aenc.h
@@ -82,7 +82,7 @@
#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/qahw/Makefile.am b/qahw/Makefile.am
index 2f33c9f..1375157 100644
--- a/qahw/Makefile.am
+++ b/qahw/Makefile.am
@@ -18,4 +18,5 @@
libqahwwrapper_la_CFLAGS += -Dstrlcpy=g_strlcpy $(GLIB_CFLAGS) -include glib.h
libqahwwrapper_la_CFLAGS += -D__unused=__attribute__\(\(__unused__\)\)
libqahwwrapper_la_CFLAGS += -Werror -Wall
+libqahwwrapper_la_CFLAGS += -DNDEBUG
libqahwwrapper_la_LDFLAGS = -shared -avoid-version -llog -lcutils -lhardware $(GLIB_LIBS)
diff --git a/qahw_api/Makefile.am b/qahw_api/Makefile.am
index 13fe417..5892bb9 100644
--- a/qahw_api/Makefile.am
+++ b/qahw_api/Makefile.am
@@ -21,6 +21,7 @@
libqahw_la_CPPFLAGS := $(AM_CPPFLAGS)
libqahw_la_CPPFLAGS += -std=c++11 -DHAVE_PTHREADS -DHAVE_ANDROID_OS
libqahw_la_CPPFLAGS += -DDEBUG_REFS_CALLSTACK_ENABLED=0
+libqahw_la_CPPFLAGS += -DNDEBUG
libqahw_la_LDFLAGS = -ltinyalsa -lhardware -lexpat -lcutils -llog -ldl -lbinder -shared -avoid-version -llog -lcutils -lpthread -lutils
if QTI_AUDIO_SERVER_ENABLED
AM_CPPFLAGS += -DQTI_AUDIO_SERVER_ENABLED