commit e37e5711cfbc23433c35b2e39524dcd734480d2b
author Amara Venkata Mastan Manoj Kumar <manojavm@codeaurora.org> Mon Dec 17 17:36:02 2012 -0800
committer Amara Venkata Mastan Manoj Kumar <manojavm@codeaurora.org> Tue Dec 18 14:17:37 2012 -0800
tree 586c946007c782fc33c2c6e89ef4d7d8391e9155
parent d74f5f3e4c4132f71326cec704352da92211cc7c

hwc: Handle ext display device conn/disconn requests

  On HDMI connection, two uevents are generated
    1."change@/devices/virtual/switch/hdmi"
    2."change@/devices/virtual/switch/hdmi_audio"
  Since we look for "change@/devices/virtual/switch/hdmi",
  Display HAL will process the hotplug request twice.
  This causes regression on HDCP enabled devices.

  Change involves to check for uevent str with
  "change@/devices/virtual/switch/hdmi" to ensures
  that we process connection request only once.

Change-Id: Ica97fd4ef3c8e1064e7d757ab754bb1ee4a3fcaa
CRs-Fixed: 429803

libhwcomposer/hwc_uevents.cpp

diff --git a/libhwcomposer/hwc_uevents.cpp b/libhwcomposer/hwc_uevents.cpp
index 78f6788..398634a 100644
--- a/libhwcomposer/hwc_uevents.cpp
+++ b/libhwcomposer/hwc_uevents.cpp
@@ -38,8 +38,8 @@
     int64_t timestamp = 0;
     const char *str = udata;
 
-    if(!strcasestr(str, "change@/devices/virtual/switch/hdmi") &&
-       !strcasestr(str, "change@/devices/virtual/switch/wfd")) {
+    if(!strcasestr("change@/devices/virtual/switch/hdmi", str) &&
+       !strcasestr("change@/devices/virtual/switch/wfd", str)) {
         ALOGD_IF(UEVENT_DEBUG, "%s: Not Ext Disp Event ", __FUNCTION__);
         return;
     }