diff options
author | John Grossman <johngro@google.com> | 2012-07-20 14:51:35 -0700 |
---|---|---|
committer | John Grossman <johngro@google.com> | 2012-08-08 11:28:15 -0700 |
commit | d8f178d613821c3f61a5c5e391eb275339e526a9 (patch) | |
tree | 93038d06c70535eeebc364e9f86cedbec074802c /services/audioflinger/AudioFlinger.cpp | |
parent | 3478eaa23c19dbbe473be70e8f3cc0c904d2465c (diff) | |
download | frameworks_av-d8f178d613821c3f61a5c5e391eb275339e526a9.zip frameworks_av-d8f178d613821c3f61a5c5e391eb275339e526a9.tar.gz frameworks_av-d8f178d613821c3f61a5c5e391eb275339e526a9.tar.bz2 |
Change audio flinger to user HAL master mute if available
(cherry picked from commit 91de9b56282d126ffb36344266af5fee3cefcfdd)
> Change audio flinger to user HAL master mute if available
>
> Hand merge from ics-aah
>
> > Change audio flinger to user HAL master mute if available: DO NOT MERGE
> >
> > Replicate the pattern used for HAL master volume support to make use
> > of master mute support if the HAL supports it. This is part of the
> > change needed to address bug 6828363. Because of the divergences
> > between ICS and master, this change will need to be merged by hand.
> >
> > Signed-off-by: John Grossman <johngro@google.com>
> > Change-Id: I6d83be524021d273d093bcb117b8f2fe57c23685
>
> Change-Id: I32280582905c969aaec2bb166ec5c61df82d737a
> Signed-off-by: John Grossman <johngro@google.com>
Change-Id: I5cd709187221d307fe25c5117ccaadca5f6b197b
Signed-off-by: John Grossman <johngro@google.com>
Diffstat (limited to 'services/audioflinger/AudioFlinger.cpp')
-rw-r--r-- | services/audioflinger/AudioFlinger.cpp | 95 |
1 files changed, 86 insertions, 9 deletions
diff --git a/services/audioflinger/AudioFlinger.cpp b/services/audioflinger/AudioFlinger.cpp index d65a2b6..41ae70e 100644 --- a/services/audioflinger/AudioFlinger.cpp +++ b/services/audioflinger/AudioFlinger.cpp @@ -219,6 +219,8 @@ AudioFlinger::AudioFlinger() mMasterVolumeSW(1.0f), mMasterVolumeSupportLvl(MVS_NONE), mMasterMute(false), + mMasterMuteSW(false), + mMasterMuteSupportLvl(MMS_NONE), mNextUniqueId(1), mMode(AUDIO_MODE_INVALID), mBtNrecIsOff(false) @@ -699,16 +701,40 @@ bool AudioFlinger::getMicMute() const status_t AudioFlinger::setMasterMute(bool muted) { + status_t ret = initCheck(); + if (ret != NO_ERROR) { + return ret; + } + // check calling permissions if (!settingsAllowed()) { return PERMISSION_DENIED; } + bool swmm = muted; + + // when hw supports master mute, don't mute in sw mixer + if (MMS_NONE != mMasterMuteSupportLvl) { + for (size_t i = 0; i < mAudioHwDevs.size(); i++) { + AutoMutex lock(mHardwareLock); + audio_hw_device_t *dev = mAudioHwDevs.valueAt(i)->hwDevice(); + + mHardwareStatus = AUDIO_HW_SET_MASTER_MUTE; + if (NULL != dev->set_master_mute) { + dev->set_master_mute(dev, muted); + } + mHardwareStatus = AUDIO_HW_IDLE; + } + + swmm = false; + } + Mutex::Autolock _l(mLock); // This is an optimization, so PlaybackThread doesn't have to look at the one from AudioFlinger - mMasterMute = muted; + mMasterMute = muted; + mMasterMuteSW = swmm; for (size_t i = 0; i < mPlaybackThreads.size(); i++) - mPlaybackThreads.valueAt(i)->setMasterMute(muted); + mPlaybackThreads.valueAt(i)->setMasterMute(swmm); return NO_ERROR; } @@ -731,6 +757,12 @@ bool AudioFlinger::masterMute() const return masterMute_l(); } +bool AudioFlinger::masterMuteSW() const +{ + Mutex::Autolock _l(mLock); + return masterMuteSW_l(); +} + float AudioFlinger::masterVolume_l() const { if (MVS_FULL == mMasterVolumeSupportLvl) { @@ -750,6 +782,24 @@ float AudioFlinger::masterVolume_l() const return mMasterVolume; } +bool AudioFlinger::masterMute_l() const +{ + if (MMS_FULL == mMasterMuteSupportLvl) { + bool ret_val; + AutoMutex lock(mHardwareLock); + + mHardwareStatus = AUDIO_HW_GET_MASTER_MUTE; + assert(NULL != mPrimaryHardwareDev); + assert(NULL != mPrimaryHardwareDev->get_master_mute); + + mPrimaryHardwareDev->get_master_mute(mPrimaryHardwareDev, &ret_val); + mHardwareStatus = AUDIO_HW_IDLE; + return ret_val; + } + + return mMasterMute; +} + status_t AudioFlinger::setStreamVolume(audio_stream_type_t stream, float value, audio_io_handle_t output) { @@ -1503,7 +1553,7 @@ AudioFlinger::PlaybackThread::PlaybackThread(const sp<AudioFlinger>& audioFlinge mMixBuffer(NULL), mSuspended(0), mBytesWritten(0), // Assumes constructor is called by AudioFlinger with it's mLock held, // but it would be safer to explicitly pass initial masterMute as parameter - mMasterMute(audioFlinger->masterMute_l()), + mMasterMute(audioFlinger->masterMuteSW_l()), // mStreamTypes[] initialized in constructor body mOutput(output), // Assumes constructor is called by AudioFlinger with it's mLock held, @@ -6863,10 +6913,13 @@ audio_io_handle_t AudioFlinger::openOutput(audio_module_handle_t module, mHardwareStatus = AUDIO_HW_SET_MODE; outHwDev->set_mode(outHwDev, mMode); - // Determine the level of master volume support the primary audio HAL has, - // and set the initial master volume at the same time. + // Determine the level of master volume/master mute support the primary + // audio HAL has, and set the initial master volume/mute state at the same + // time. float initialVolume = 1.0; + bool initialMute = false; mMasterVolumeSupportLvl = MVS_NONE; + mMasterMuteSupportLvl = MMS_NONE; mHardwareStatus = AUDIO_HW_GET_MASTER_VOLUME; if ((NULL != outHwDev->get_master_volume) && @@ -6882,20 +6935,44 @@ audio_io_handle_t AudioFlinger::openOutput(audio_module_handle_t module, (NO_ERROR != outHwDev->set_master_volume(outHwDev, initialVolume))) { mMasterVolumeSupportLvl = MVS_NONE; } - // now that we have a primary device, initialize master volume on other devices + + mHardwareStatus = AUDIO_HW_GET_MASTER_MUTE; + if ((NULL != outHwDev->get_master_mute) && + (NO_ERROR == outHwDev->get_master_mute(outHwDev, &initialMute))) { + mMasterMuteSupportLvl = MMS_FULL; + } else { + mMasterMuteSupportLvl = MMS_SETONLY; + initialMute = 0; + } + + mHardwareStatus = AUDIO_HW_SET_MASTER_MUTE; + if ((NULL == outHwDev->set_master_mute) || + (NO_ERROR != outHwDev->set_master_mute(outHwDev, initialMute))) { + mMasterMuteSupportLvl = MMS_NONE; + } + + // now that we have a primary device, initialize master volume/mute + // on other devices for (size_t i = 0; i < mAudioHwDevs.size(); i++) { audio_hw_device_t *dev = mAudioHwDevs.valueAt(i)->hwDevice(); if ((dev != mPrimaryHardwareDev) && (NULL != dev->set_master_volume)) { + mHardwareStatus = AUDIO_HW_SET_MASTER_VOLUME; dev->set_master_volume(dev, initialVolume); } + + if (NULL != dev->set_master_mute) { + mHardwareStatus = AUDIO_HW_SET_MASTER_MUTE; + dev->set_master_mute(dev, initialMute); + } } + mHardwareStatus = AUDIO_HW_IDLE; - mMasterVolumeSW = (MVS_NONE == mMasterVolumeSupportLvl) - ? initialVolume - : 1.0; + mMasterVolumeSW = initialVolume; mMasterVolume = initialVolume; + mMasterMuteSW = initialMute; + mMasterMute = initialMute; } return id; } |