e200: Better sync of display enabling and display update. The display would be remain stale after being turned back on with no screen activity. Block out the fiddling with DMA when writing to driver framebuffer. Melts seem too rare to catch now. There seems to be no significant advantage from an appearance point of view to be meddling around with the transfer anyway.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@13582 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 5 insertions, 10 deletions

diff --git a/firmware/target/arm/sandisk/sansa-e200/backlight-e200.c b/firmware/target/arm/sandisk/sansa-e200/backlight-e200.c
index b608c1b9e6..29c6ab4b45 100644
--- a/firmware/target/arm/sandisk/sansa-e200/backlight-e200.c
+++ b/firmware/target/arm/sandisk/sansa-e200/backlight-e200.c
@@ -23,32 +23,27 @@
 #include "i2c-pp.h"
 
 static unsigned short backlight_brightness = DEFAULT_BRIGHTNESS_SETTING;
-static bool backlight_is_on = true;
-
-int __backlight_is_on(void)
-{
-    return (int)backlight_is_on;
-}
 
 void __backlight_set_brightness(int brightness)
 {
     backlight_brightness = brightness;
-    pp_i2c_send( 0x46, 0x23, backlight_brightness);
-    backlight_is_on = true;
+
+    if (brightness > 0)
+        __backlight_on();
+    else
+        __backlight_off();
 }
 
 void __backlight_on(void)
 {
     lcd_enable(true); /* power on lcd */
     pp_i2c_send( 0x46, 0x23, backlight_brightness);
-    backlight_is_on = true;
 }
 
 void __backlight_off(void)
 {
     pp_i2c_send( 0x46, 0x23, 0x0);
     lcd_enable(false); /* power off lcd */
-    backlight_is_on = false;
 }