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

Commit a135dd88 authored by Eric Laurent's avatar Eric Laurent Committed by Dmitry Shmidt
Browse files

AudioFlinger: mute mic on primary HAL only 



Only the primary HAL has to support mic mute function.
Ignore errors returned by other HALs in setMicMute() and
only report mic mute state from primary HAL in getMicMute().

Bug: 153103117
Test: CTS AudioManagerTest.testMicrophoneMute
Test: manual phone call and verify microphone mute.

Signed-off-by: default avatarDmitry Shmidt <dimitrysh@google.com>
Merged-In: Ic121c49e7e7cd75eeee9342249271c7fca7c1c2a
Change-Id: Icde20a0da2e10eb1b7ac17b4a901d27fe0dca9c6
parent 3ec7c256

services/audioflinger/AudioFlinger.cpp

+18 −14
Original line number Diff line number Diff line
@@ -1175,15 +1175,21 @@ status_t AudioFlinger::setMicMute(bool state) 
    }



    AutoMutex lock(mHardwareLock);

    sp<DeviceHalInterface> primaryDev = mPrimaryHardwareDev->hwDevice();

    if (primaryDev == nullptr) {

        ALOGW("%s: no primary HAL device", __func__);

        return INVALID_OPERATION;

    }

    mHardwareStatus = AUDIO_HW_SET_MIC_MUTE;

    ret = primaryDev->setMicMute(state);

    for (size_t i = 0; i < mAudioHwDevs.size(); i++) {

        sp<DeviceHalInterface> dev = mAudioHwDevs.valueAt(i)->hwDevice();

        status_t result = dev->setMicMute(state);

        if (result != NO_ERROR) {

            ret = result;

        if (dev != primaryDev) {

            (void)dev->setMicMute(state);

        }

    }

    mHardwareStatus = AUDIO_HW_IDLE;

    ALOGW_IF(ret != NO_ERROR, "%s: error %d setting state to HAL", __func__, ret);

    return ret;

}
@@ -1193,20 +1199,18 @@ bool AudioFlinger::getMicMute() const 
    if (ret != NO_ERROR) {

        return false;

    }

    bool mute = true;

    bool state = AUDIO_MODE_INVALID;

    AutoMutex lock(mHardwareLock);

    mHardwareStatus = AUDIO_HW_GET_MIC_MUTE;

    for (size_t i = 0; i < mAudioHwDevs.size(); i++) {

        sp<DeviceHalInterface> dev = mAudioHwDevs.valueAt(i)->hwDevice();

        status_t result = dev->getMicMute(&state);

        if (result == NO_ERROR) {

            mute = mute && state;

        }

    sp<DeviceHalInterface> primaryDev = mPrimaryHardwareDev->hwDevice();

    if (primaryDev == nullptr) {

        ALOGW("%s: no primary HAL device", __func__);

        return false;

    }

    bool state;

    mHardwareStatus = AUDIO_HW_GET_MIC_MUTE;

    ret = primaryDev->getMicMute(&state);

    mHardwareStatus = AUDIO_HW_IDLE;



    return mute;

    ALOGE_IF(ret != NO_ERROR, "%s: error %d getting state from HAL", __func__, ret);

    return (ret == NO_ERROR) && state;

}



void AudioFlinger::setRecordSilenced(uid_t uid, bool silenced)