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

Commit 1c7bc5c3 authored by Sachin Mohan Gadag's avatar Sachin Mohan Gadag Committed by Arne Coucheron
Browse files

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
parent 93ec5ef4

mm-audio/aenc-aac/qdsp6/Makefile.am

+2 −1
Original line number Diff line number Diff line
@@ -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

mm-audio/aenc-aac/qdsp6/inc/omx_aac_aenc.h

+1 −1
Original line number Diff line number Diff line
@@ -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,   \

mm-audio/aenc-amrnb/qdsp6/inc/omx_amr_aenc.h

+1 −1
Original line number Diff line number Diff line
@@ -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,   \

mm-audio/aenc-amrnb/qdsp6/src/omx_amr_aenc.cpp

+15 −15
Original line number Diff line number Diff line
@@ -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;

}
@@ -3980,7 +3980,7 @@ OMX_ERRORTYPE omx_amr_aenc::empty_this_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
@@ -4147,7 +4147,7 @@ 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 \

          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);
@@ -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)