[Android Builders Collective] Seik19:
Anyone knows if there is a problem with the qcom-audio-caf/msm8960? My build breaks at this point, I'm trying with a custom toolchain but it breaks with stock too.

target thumb C++: libmemalloc <= hardware/qcom/display-caf/msm8960/libgralloc/alloc_controller.cpp

In file included from hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:25:0:

system/core/include/system/audio.h:901:1: warning: missing initializer for member 'audio_offload_info_t::bit_width' [-Wmissing-field-initializers]

};

^

system/core/include/system/audio.h:901:1: warning: missing initializer for member 'audio_offload_info_t::use_small_bufs' [-Wmissing-field-initializers]

In file included from hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:32:0:

hardware/libhardware_legacy/include/hardware_legacy/AudioHardwareInterface.h:126:25: warning: unused parameter 'observer' [-Wunused-parameter]

virtual int setObserver(void *observer) {return INVALID_OPERATION;}

^

hardware/libhardware_legacy/include/hardware_legacy/AudioHardwareInterface.h:127:25: warning: unused parameter 'buf' [-Wunused-parameter]

virtual status_t getBufferInfo(buf_info **buf) {return INVALID_OPERATION;}

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:164:12: warning: unused parameter 'rate' [-Wunused-parameter]

static int out_set_sample_rate(struct audio_stream *stream, uint32_t rate)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:195:12: warning: unused parameter 'format' [-Wunused-parameter]

static int out_set_format(struct audio_stream *stream, audio_format_t format)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:341:12: warning: unused parameter 'stream' [-Wunused-parameter]

static int out_add_audio_effect(const struct audio_stream *stream, effect_handle_t effect)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:341:12: warning: unused parameter 'effect' [-Wunused-parameter]

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:346:12: warning: unused parameter 'stream' [-Wunused-parameter]

static int out_remove_audio_effect(const struct audio_stream *stream, effect_handle_t effect)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:346:12: warning: unused parameter 'effect' [-Wunused-parameter]

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp: In function 'int android_audio_legacy::out_get_presentation_position(const audio_stream_out*, uint64_t*, timespec*)':

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:364:27: error: 'class android_audio_legacy::AudioStreamOut' has no member named 'getPresentationPosition'

return out->qcom_out->getPresentationPosition(frames, timestamp);

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp: At global scope:

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:376:12: warning: unused parameter 'rate' [-Wunused-parameter]

static int in_set_sample_rate(struct audio_stream *stream, uint32_t rate)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:407:12: warning: unused parameter 'format' [-Wunused-parameter]

static int in_set_format(struct audio_stream *stream, audio_format_t format)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:516:17: warning: unused parameter 'dev' [-Wunused-parameter]

static uint32_t adev_get_supported_devices(const struct audio_hw_device *dev)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp: In function 'int android_audio_legacy::adev_open_output_stream(audio_hw_device*, audio_io_handle_t, audio_devices_t, audio_output_flags_t, audio_config*, audio_stream_out**, const char*)':

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:673:69: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]

(int *)&config->format,

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp: At global scope:

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:652:12: warning: unused parameter 'handle' [-Wunused-parameter]

static int adev_open_output_stream(struct audio_hw_device *dev,

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp: In function 'int android_audio_legacy::adev_open_input_stream(audio_hw_device*, audio_io_handle_t, audio_devices_t, audio_config*, audio_stream_in**, audio_input_flags_t, const char*, audio_source_t)':

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:748:73: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]

in->qcom_in = qadev->hwif->openInputStream(devices, (int *)&config->format,

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp: At global scope:

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:729:12: warning: unused parameter 'handle' [-Wunused-parameter]

static int adev_open_input_stream(struct audio_hw_device *dev,

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:729:12: warning: unused parameter 'flags' [-Wunused-parameter]

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp: In function 'int android_audio_legacy::out_get_presentation_position(const audio_stream_out*, uint64_t*, timespec*)':

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/audio_hw_hal.cpp:365:1: error: control reaches end of non-void function [-Werror=return-type]

}

^

cc1plus: some warnings being treated as errors

make: *** [/home/seik/android/slim/out/target/product/jfltexx/obj/SHARED_LIBRARIES/audio.primary.msm8960_intermediates/audio_hw_hal.o] Error 1

make: *** Se espera a que terminen otras tareas....

In file included from hardware/libhardware_legacy/include/hardware_legacy/AudioHardwareBase.h:20:0,

from hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/AudioHardwareALSA.h:23,

from hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:25:

hardware/libhardware_legacy/include/hardware_legacy/AudioHardwareInterface.h:126:25: warning: unused parameter 'observer' [-Wunused-parameter]

virtual int setObserver(void *observer) {return INVALID_OPERATION;}

^

hardware/libhardware_legacy/include/hardware_legacy/AudioHardwareInterface.h:127:25: warning: unused parameter 'buf' [-Wunused-parameter]

virtual status_t getBufferInfo(buf_info **buf) {return INVALID_OPERATION;}

^

In file included from hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:25:0:

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/AudioHardwareALSA.h:731:22: warning: unused parameter 'effect' [-Wunused-parameter]

virtual status_t addAudioEffect(effect_handle_t effect)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/AudioHardwareALSA.h:736:22: warning: unused parameter 'effect' [-Wunused-parameter]

virtual status_t removeAudioEffect(effect_handle_t effect)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp: In member function 'android::status_t android_audio_legacy::ALSADevice::setHDMIChannelCount()':

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:248:32: warning: missing initializer for member 'EDID_AUDIO_INFO::nSpeakerAllocation' [-Wmissing-field-initializers]

EDID_AUDIO_INFO info = { 0 };

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:248:32: warning: missing initializer for member 'EDID_AUDIO_INFO::AudioBlocksArray' [-Wmissing-field-initializers]

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:248:32: warning: missing initializer for member 'EDID_AUDIO_INFO::channelMap' [-Wmissing-field-initializers]

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:248:32: warning: missing initializer for member 'EDID_AUDIO_INFO::channelAllocation' [-Wmissing-field-initializers]

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp: In member function 'android::status_t android_audio_legacy::ALSADevice::setHardwareParams(android_audio_legacy::alsa_handle_t*)':

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:351:39: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

for (int i = 0; i < compr_cap.num_codecs; i++) {

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp: At global scope:

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:1423:10: warning: unused parameter 'vsid' [-Wunused-parameter]

status_t ALSADevice::close(alsa_handle_t *handle, uint32_t vsid)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:2329:6: warning: unused parameter 'vsid' [-Wunused-parameter]

void ALSADevice::enableWideVoice(bool flag, uint32_t vsid)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:2364:6: warning: unused parameter 'vsid' [-Wunused-parameter]

void ALSADevice::enableFENS(bool flag, uint32_t vsid)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:2395:6: warning: unused parameter 'vsid' [-Wunused-parameter]

void ALSADevice::enableSlowTalk(bool flag, uint32_t vsid)

^

In file included from system/core/include/cutils/log.h:1:0,

from system/core/include/utils/Log.h:31,

from hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:21:

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp: In member function 'ssize_t android_audio_legacy::ALSADevice::readFromProxy(void**, ssize_t*)':

system/core/include/log/log.h:505:39: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long int' [-Wformat=]

__android_log_print(prio, tag, fmt)

^

system/core/include/log/log.h:440:5: note: in expansion of macro 'android_printLog'

android_printLog(priority, tag, __VA_ARGS__)

^

system/core/include/log/log.h:432:5: note: in expansion of macro 'LOG_PRI'

LOG_PRI(ANDROID_##priority, tag, __VA_ARGS__)

^

system/core/include/log/log.h:143:27: note: in expansion of macro 'ALOG'

#define ALOGE(...) ((void)ALOG(LOG_ERROR, LOG_TAG, __VA_ARGS__))

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:2727:13: note: in expansion of macro 'ALOGE'

ALOGE("Error mProxyParams.mFrames = %d", mProxyParams.mFrames);

^

system/core/include/log/log.h:505:39: warning: format '%d' expects argument of type 'int', but argument 4 has type 'long int' [-Wformat=]

__android_log_print(prio, tag, fmt)

^

system/core/include/log/log.h:440:5: note: in expansion of macro 'android_printLog'

android_printLog(priority, tag, __VA_ARGS__)

^

system/core/include/log/log.h:432:5: note: in expansion of macro 'LOG_PRI'

LOG_PRI(ANDROID_##priority, tag, __VA_ARGS__)

^

system/core/include/log/log.h:76:29: note: in expansion of macro 'ALOG'

#define __ALOGV(...) ((void)ALOG(LOG_VERBOSE, LOG_TAG, __VA_ARGS__))

^

system/core/include/log/log.h:78:34: note: in expansion of macro '__ALOGV'

#define ALOGV(...) do { if (0) { __ALOGV(__VA_ARGS__); } } while (0)

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:2745:9: note: in expansion of macro 'ALOGV'

ALOGV("Calling sync_ptr for proxy after sync with MFrames is %d", mProxyParams.mFrames);

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp: In member function 'android::status_t android_audio_legacy::ALSADevice::openProxyDevice()':

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:2832:77: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]

mProxyParams.mProxyPcmHandle = pcm_open(flags, PROXY_CAPTURE_DEVICE_NAME);

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp: In member function 'android::status_t android_audio_legacy::ALSADevice::getEDIDData(char*)':

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:3027:28: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

for(int i=0; i<count; i++) {

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp: In member function 'void android_audio_legacy::ALSADevice::switchDevice(android_audio_legacy::alsa_handle_t*, uint32_t, uint32_t)':

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:831:10: warning: 'tx_dev_id' may be used uninitialized in this function [-Wmaybe-uninitialized]

&& tx_dev_id == DEVICE_HANDSET_TX_ACDB_ID) {

^

hardware/qcom/audio-caf/msm8960/legacy/alsa_sound/ALSADevice.cpp:866:73: warning: 'rx_dev_id' may be used uninitialized in this function [-Wmaybe-uninitialized]

err = csd_enable_device(tmp_rx_id, tmp_tx_id, adjustedFlags);

^

hardware/qcom/display-caf/msm8960/libcopybit/copybit_c2d.cpp:1070:13: warning: unused parameter 'ctx' [-Wunused-parameter]

static bool need_to_execute_draw(struct copybit_context_t* ctx,

^

hardware/qcom/display-caf/msm8960/libcopybit/copybit_c2d.cpp:1486:12: warning: unused parameter 'name' [-Wunused-parameter]

static int open_copybit(const struct hw_module_t* module, const char* name,

^

hardware/qcom/display-caf/msm8960/libgralloc/ionalloc.cpp:167:5: warning: unused parameter 'offset' [-Wunused-parameter]

int IonAlloc::unmap_buffer(void *base, size_t size, int offset)

^

#### make failed to build some targets (05:53 (mm:ss)) ####