fix a problem in the c7x0 backlight handling...

Bah! Got confused by the input interval for ODevice::setDisplayBrightness()
(once) again - it has been ridiculous to scale that down from [0-255] to the actual
interval since the day we invented ODevice::displayBrightnessResolution().
I'm going to remove that antique cruft soon.

1 files changed, 14 insertions, 12 deletions

diff --git a/libopie/odevice.cpp b/libopie/odevice.cpp
index bc09e92..82a0099 100644
--- a/libopie/odevice.cpp
+++ b/libopie/odevice.cpp
@@ -1556,9 +1556,9 @@ bool Zaurus::isZaurus()
                         model = line. mid ( loc + 2 ). simplifyWhiteSpace( );
 
-                if ( model == "Sharp-Collie" 
-                     || model == "Collie" 
+                if ( model == "Sharp-Collie"
+                     || model == "Collie"
                      || model == "SHARP Corgi"
-                     || model == "SHARP Shepherd" 
-                     || model == "SHARP Poodle" 
+                     || model == "SHARP Shepherd"
+                     || model == "SHARP Poodle"
                      || model == "SHARP Husky"
                    )
@@ -1579,8 +1579,8 @@ void Zaurus::init ( )
         QString model;
 
-        // It isn't a good idea to check the system configuration to 
-        // detect the distribution ! 
+        // It isn't a good idea to check the system configuration to
+        // detect the distribution !
         // Otherwise it may happen that any other distribution is detected as openzaurus, just
-        // because it uses a jffs2 filesystem..  
+        // because it uses a jffs2 filesystem..
         // (eilers)
         // if ( f. open ( IO_ReadOnly ) && ( QTextStream ( &f ). read ( ). find ( "\tjffs2\n" ) >= 0 )) {
@@ -1697,7 +1697,7 @@ void Zaurus::initButtons ( )
                 b. setUserText ( QObject::tr ( "Button", zb-> utext ));
                 b. setPixmap ( Resource::loadPixmap ( zb-> pix ));
-                b. setFactoryPresetPressedAction ( OQCopMessage ( makeChannel ( zb-> fpressedservice ), 
+                b. setFactoryPresetPressedAction ( OQCopMessage ( makeChannel ( zb-> fpressedservice ),
                                                                   zb-> fpressedaction ));
-                b. setFactoryPresetHeldAction ( OQCopMessage ( makeChannel ( zb-> fheldservice ), 
+                b. setFactoryPresetHeldAction ( OQCopMessage ( makeChannel ( zb-> fheldservice ),
                                                                zb-> fheldaction ));
 
@@ -1708,5 +1708,5 @@ void Zaurus::initButtons ( )
 
         QCopChannel *sysch = new QCopChannel ( "QPE/System", this );
-        connect ( sysch, SIGNAL( received( const QCString &, const QByteArray & )), 
+        connect ( sysch, SIGNAL( received( const QCString &, const QByteArray & )),
                   this, SLOT( systemMessage ( const QCString &, const QByteArray & )));
 }
@@ -1965,4 +1965,5 @@ bool Zaurus::setSoftSuspend ( bool soft )
 bool Zaurus::setDisplayBrightness ( int bright )
 {
+    //qDebug( "Zaurus::setDisplayBrightness( %d )", bright );
     bool res = false;
     int fd;
@@ -1975,10 +1976,11 @@ bool Zaurus::setDisplayBrightness ( int bright )
         if ( d->m_model == Model_Zaurus_SLC7x0 )
         {
+            //qDebug( "using special treatment for devices with the corgi backlight interface" );
             // special treatment for devices with the corgi backlight interface
             if (( fd = ::open ( "/proc/driver/fl/corgi-bl", O_WRONLY )) >= 0 )
             {
-                if ( bright > 0x11 ) bright = 0x11;
+                int value = ( bright == 1 ) ? 1 : bright * ( 17.0 / 255.0 );
                 char writeCommand[100];
-                const int count = sprintf( writeCommand, "0x%x\n", bright );
+                const int count = sprintf( writeCommand, "0x%x\n", value );
                 res = ( ::write ( fd, writeCommand, count ) != -1 );
                 ::close ( fd );