commit 9dc7c7f242cbb216f1298a1046e2bd4ca509a27c
author samin.ryu <samin.ryu@lge.com> Sun Oct 07 23:44:39 2012 -0700
committer Android Git Automerger <android-git-automerger@android.com> Sun Oct 07 23:44:39 2012 -0700
tree 4d1cf7a2c366d85a06ec8c2ebc2704e7641129f8
parent 04c661479d9a6c4f16b2d31f52d4288dbd5e661f
parent 72f83fe83242142ad3f10fb87f19e6d7d33b7c49

am 72f83fe8: am 2c798912: alsa_sound: avoid pcm_open error

* commit '72f83fe83242142ad3f10fb87f19e6d7d33b7c49':
  alsa_sound: avoid pcm_open error

alsa_sound/ALSAStreamOps.cpp

diff --git a/alsa_sound/ALSAStreamOps.cpp b/alsa_sound/ALSAStreamOps.cpp
index aafc736..4e534fd 100644
--- a/alsa_sound/ALSAStreamOps.cpp
+++ b/alsa_sound/ALSAStreamOps.cpp
@@ -214,12 +214,16 @@
 
     if (param.getInt(key, device) == NO_ERROR) {
         // Ignore routing if device is 0.
-        ALOGD("setParameters(): keyRouting with device %d", device);
+        ALOGD("setParameters(): keyRouting with device 0x%x", device);
         // reset to speaker when disconnecting HDMI to avoid timeout due to write errors
         if ((device == 0) && (mDevices == AudioSystem::DEVICE_OUT_AUX_DIGITAL)) {
             device = AudioSystem::DEVICE_OUT_SPEAKER;
         }
-        mDevices = device;
+        if (device)
+            mDevices = device;
+        else
+            ALOGV("must not change mDevices to 0");
+
         if(device) {
             mParent->doRouting(device);
         }

alsa_sound/AudioStreamOutALSA.cpp

diff --git a/alsa_sound/AudioStreamOutALSA.cpp b/alsa_sound/AudioStreamOutALSA.cpp
index 64492ba..d047cd4 100644
--- a/alsa_sound/AudioStreamOutALSA.cpp
+++ b/alsa_sound/AudioStreamOutALSA.cpp
@@ -178,8 +178,6 @@
             mHandle->module->route(mHandle, mDevices , mParent->mode());
 #endif
         } else {
-            if (!mDevices)
-               mDevices = mParent->mCurDevice;
             mHandle->module->route(mHandle, mDevices , mParent->mode());
         }
         if (!strcmp(mHandle->useCase, SND_USE_CASE_VERB_HIFI) ||