Notify LED Light

[GoogleCodeExporter]

What steps will reproduce the problem?
1.Get a SMS.
2.Wake up Phone
3.Let the phone sleep again with the Lock Button
4.LED Light goes off too.

What is the expected output? What do you see instead?
The Light should be on until i read the SMS.

What version of the product are you using?
Latest System.img & Boot.img

On what operating system?
Windows 7 Ultimate 64-bit or you mean my latest OpenEve?

Do you have logcat or dmesg ?
 (adb shell logcat > logcat.txt ^C, adb shell dmesg > dmesg.txt)
No :(

What version of the *Official LG* Firmware were before using Custom
firmware ?
(V20G, V20D, V15x, V16x etc. )
V10e

Please provide any additional information below.

Original issue reported on code.google.com by Katzen...@gmail.com on 5 May 2011 at 8:51

[GoogleCodeExporter]

why you put a V10e issue here ?? put it on http://forum.openetna.com

Original comment by justsad.eng@gmail.com on 5 May 2011 at 9:44

[GoogleCodeExporter]

because this issue is since i updated to your the system image.
(From system-gb-0330.zip to system-gb-110430-snapshot.zip)
and since i used openeve my phone only says "unknown" if i want to know the 
firmware.
before i changed to openneve from openetna it was V10e.

Original comment by Katzen...@gmail.com on 5 May 2011 at 9:47

[GoogleCodeExporter]

i don't know what firmware version i have now since openeve.

Original comment by Katzen...@gmail.com on 5 May 2011 at 9:49

[GoogleCodeExporter]

But guys, the question is what firmware version did you have BEFORE 
OpenEtna/OpenEVE, when you had the official LG firmware, eh? :)

Original comment by rais...@gmail.com on 6 May 2011 at 11:01

[GoogleCodeExporter]

sorry Katzen/rais ...  i was confused and didn't read the issue. i'm sorry..

Original comment by justsad.eng@gmail.com on 6 May 2011 at 11:06

[GoogleCodeExporter]

No problem. ^^

Original comment by Katzen...@gmail.com on 7 May 2011 at 7:59

[GoogleCodeExporter]

currently notification light has some issue.

Original comment by wkp...@gmail.com on 7 May 2011 at 10:17

[GoogleCodeExporter]

Original comment by wkp...@gmail.com on 7 May 2011 at 10:18

• Changed state: Accepted

[GoogleCodeExporter]

Issue 105 has been merged into this issue.

Original comment by wkp...@gmail.com on 16 May 2011 at 2:55

[GoogleCodeExporter]

It works since midnight now.
Every day 0:00 the notify lights goes on without any alert or reminder.
since the first 0:00 o'clock notify activation since installing the new 
system.img it works now.

Original comment by Katzen...@gmail.com on 16 May 2011 at 8:55

[GoogleCodeExporter]

Notification lights need some major work. I'd like to have the option of 
disabling the backlight on Home and Back buttons completely, because I tend to 
receive SMS messages during night time, and since I'm asleep the buttons drain 
the battery.

Original comment by savalent...@gmail.com on 19 May 2011 at 9:42

[GoogleCodeExporter]

This are the changes that permit the use of the notification lite on openetna, 
and add support to enable and disable it.

https://github.com/OpenEtna/android_device_lg_eve/commit/55232453fef74c8af8ef9ab
2c6e74508c0775c93

https://github.com/OpenEtna/android_packages_apps_Settings/commit/4644065cf35ab3
062a138d5a76bd183429564307

https://github.com/OpenEtna/android_hardware_msm7k/commit/2e5ab845afe8811a7df289
049e15824c02ebb4ee

Original comment by ivolu...@gmail.com on 26 May 2011 at 8:46

[GoogleCodeExporter]

thankyou for informations.
but CM7 already have that feature. so I did not apply that patch

CM7 already have this feature. you can try to disable notification function by 
"settings=>sound=>notifications=>..."

Original comment by wkp...@gmail.com on 26 May 2011 at 9:13

[GoogleCodeExporter]

I disable notifications but still the lights turn on

Original comment by ivolu...@gmail.com on 26 May 2011 at 1:43

[GoogleCodeExporter]

ivolucas is right, disabling led notifications doesn't work. Maybe CM7 isn't 
configured to use those lights but others that gw620 doesn't have.
Maybe you could apply the patches referenced above and assign them to the 
"settings>sound>notifications" option in CM7.

Original comment by savalent...@gmail.com on 26 May 2011 at 5:30

[GoogleCodeExporter]

@ivolu, @savalentini  // you can see that this issue is not solved at all.
I just pointed out that above patch is not needed anymore for CM7.
so I did not apply it and I will not apply it.

and there are other issues also exists with "light.eve.so" and
above patch do not solve all other issue at once.

Original comment by wkp...@gmail.com on 26 May 2011 at 8:04

[GoogleCodeExporter]

with following patch, the disabling of led notification works.
-------
diff --git a/services/java/com/android/server/NotificationManagerService.java 
b/services/java/com/android/server/NotificationManagerService.java
index 5aa0022..2fd653d 100755
--- a/services/java/com/android/server/NotificationManagerService.java
+++ b/services/java/com/android/server/NotificationManagerService.java
@@ -1053,7 +1053,7 @@ public class NotificationManagerService extends 
INotificationManager.Stub
             //updatePackageList(pkg);
             //Slog.i(TAG, "notification.lights="
             //        + ((old.notification.lights.flags & Notification.FLAG_SHOW_LIGHTS) != 0));
-            //if ((notification.flags & Notification.FLAG_SHOW_LIGHTS) != 0) {
+            if ((notification.flags & Notification.FLAG_SHOW_LIGHTS) != 0) {
             if (checkLight(notification, pkg)) {
                 mLights.add(r);
                 updateLightsLocked();
@@ -1063,6 +1063,7 @@ public class NotificationManagerService extends 
INotificationManager.Stub
                         updateLightsLocked();
                 }
             }
+            }
         }

         idOut[0] = id;
---------

Original comment by wkp...@gmail.com on 29 May 2011 at 6:43

[GoogleCodeExporter]

above patch is not working perfectly...

Original comment by wkp...@gmail.com on 29 May 2011 at 1:11

[GoogleCodeExporter]

Issue 132 has been merged into this issue.

Original comment by wkp...@gmail.com on 1 Jun 2011 at 2:28

[GoogleCodeExporter]

Original comment by wkp...@gmail.com on 6 Sep 2011 at 11:02

[GoogleCodeExporter]

Original comment by wkp...@gmail.com on 10 Nov 2011 at 8:26

• Changed state: Fixed