Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Spatializer: Fix lost settings after reboot" into tm-qpr-dev

Browse Source
This commit is contained in:
Andy Hung
2022-07-21 17:00:21 +00:00
committed by Android (Google) Code Review
parent cdae841b07 8e10acae37
commit a22527d9fa
2 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
services/core/java/com/android/server/audio/AudioService.java
View File

@@ -9181,11 +9181,8 @@ public class AudioService extends IAudioService.Stub
Settings.Secure.SPATIAL_AUDIO_ENABLED, UserHandle.USER_CURRENT); Settings.Secure.SPATIAL_AUDIO_ENABLED, UserHandle.USER_CURRENT);
if (settings == null) { if (settings == null) {
Log.e(TAG, "error reading spatial audio device settings"); Log.e(TAG, "error reading spatial audio device settings");
} else {
Log.v(TAG, "restoring spatial audio device settings: " + settings);
mSpatializerHelper.setSADeviceSettings(settings);
} }
mSpatializerHelper.init(/*effectExpected*/ mHasSpatializerEffect); mSpatializerHelper.init(/*effectExpected*/ mHasSpatializerEffect, settings);
mSpatializerHelper.setFeatureEnabled(mHasSpatializerEffect); mSpatializerHelper.setFeatureEnabled(mHasSpatializerEffect);
} }

13
services/core/java/com/android/server/audio/SpatializerHelper.java
View File

@@ -175,7 +175,7 @@ public class SpatializerHelper {
mASA = asa; mASA = asa;
} }
synchronized void init(boolean effectExpected) { synchronized void init(boolean effectExpected, @Nullable String settings) {
loglogi("init effectExpected=" + effectExpected); loglogi("init effectExpected=" + effectExpected);
if (!effectExpected) { if (!effectExpected) {
loglogi("init(): setting state to STATE_NOT_SUPPORTED due to effect not expected"); loglogi("init(): setting state to STATE_NOT_SUPPORTED due to effect not expected");
@@ -278,6 +278,13 @@ public class SpatializerHelper {
mSACapableDeviceTypes.add(SPAT_MODE_FOR_DEVICE_TYPE.keyAt(i)); mSACapableDeviceTypes.add(SPAT_MODE_FOR_DEVICE_TYPE.keyAt(i));
} }
} }
// When initialized from AudioService, the settings string will be non-null.
// Saved settings need to be applied after spatialization support is initialized above.
if (settings != null) {
setSADeviceSettings(settings);
}
// for both transaural / binaural, we are not forcing enablement as the init() method // for both transaural / binaural, we are not forcing enablement as the init() method
// could have been called another time after boot in case of audioserver restart // could have been called another time after boot in case of audioserver restart
addCompatibleAudioDevice( addCompatibleAudioDevice(
@@ -316,7 +323,7 @@ public class SpatializerHelper {
mState = STATE_UNINITIALIZED; mState = STATE_UNINITIALIZED;
mSpatLevel = Spatializer.SPATIALIZER_IMMERSIVE_LEVEL_NONE; mSpatLevel = Spatializer.SPATIALIZER_IMMERSIVE_LEVEL_NONE;
mActualHeadTrackingMode = Spatializer.HEAD_TRACKING_MODE_UNSUPPORTED; mActualHeadTrackingMode = Spatializer.HEAD_TRACKING_MODE_UNSUPPORTED;
init(true); init(true, null /* settings */);
setSpatializerEnabledInt(featureEnabled); setSpatializerEnabledInt(featureEnabled);
} }
@@ -731,7 +738,7 @@ public class SpatializerHelper {
return; return;
} }
if (mState == STATE_UNINITIALIZED) { if (mState == STATE_UNINITIALIZED) {
init(true); init(true, null /* settings */);
} }
setSpatializerEnabledInt(true); setSpatializerEnabledInt(true);
} else { } else {