match h1910 and asus 716 to other pda latest changes

2 files changed, 32 insertions, 36 deletions

diff --git a/libopie2/opiecore/device/odevice_ipaq.cpp b/libopie2/opiecore/device/odevice_ipaq.cpp
index 6dc1295..b18fd32 100644
--- a/libopie2/opiecore/device/odevice_ipaq.cpp
+++ b/libopie2/opiecore/device/odevice_ipaq.cpp
@@ -451,5 +451,4 @@ int iPAQ::displayBrightnessResolution() const
         case Model_iPAQ_H4xxx:
-            return 255;
         case Model_iPAQ_H191x:
-            return 7;
+            return 255;
         case Model_iPAQ_H1940:
diff --git a/libopie2/opiecore/device/odevice_mypal.cpp b/libopie2/opiecore/device/odevice_mypal.cpp
index 73b31cb..28e92bf 100644
--- a/libopie2/opiecore/device/odevice_mypal.cpp
+++ b/libopie2/opiecore/device/odevice_mypal.cpp
@@ -142,14 +142,18 @@ bool MyPal::filter ( int /*unicode*/, int keycode, int modifiers, bool isPress,
     switch ( keycode ) {
-        case Key_Left :
-        case Key_Right:
-        case Key_Up   :
-        case Key_Down : {
-            // Rotate the cursor keys by 270
-            // keycode - Key_Left = position of the button starting from left clockwise
-            // add the rotation to it and modolo. No we've the original offset
-            // add the offset to the Key_Left key
-            if ( d->m_model == Model_MyPal_716 )
-                newkeycode = Key_Left + ( keycode - Key_Left + 3 ) % 4;
-            break;
-        }
+        // QT has strange screen coordinate system, so depending
+        // on native device screen orientation, we need to rotate cursor keys
+        case Key_Left :
+        case Key_Right:
+        case Key_Up   :
+        case Key_Down : {
+            int quarters;
+            switch (d->m_rotation) {
+                case Rot0:   quarters = 3/*270deg*/; break;
+                case Rot90:  quarters = 2/*270deg*/; break;
+                case Rot180: quarters = 1/*270deg*/; break;
+                case Rot270: quarters = 0/*270deg*/; break;
+            }
+            newkeycode = Key_Left + ( keycode - Key_Left + quarters ) % 4;
+            break;
+        }
 
@@ -212,22 +216,13 @@ bool MyPal::setDisplayBrightness ( int bright )
 
-    QDir sysClass( "/sys/class/backlight/pxafb/" );
+    QDir sysClass( "/sys/class/backlight/" );
     sysClass.setFilter(QDir::Dirs);
-    int fd;
-    if ( sysClass.exists() ) {
-        QString sysClassPath = sysClass.absFilePath( "/sys/class/backlight/pxafb/power" );
-        fd = ::open( sysClassPath, O_WRONLY | O_NONBLOCK );
-        if ( fd ) {
-            char buf[10];
-            buf[0] = bright ? 0 : 4;
-            buf[1] = '\0';
-            res = ( ::write( fd, &buf[0], 2 ) == 0 );
-            ::close( fd );
-        }
-        sysClassPath = sysClass.absFilePath( "/sys/class/backlight/pxafb/brightness" );
-        fd = ::open( sysClassPath, O_WRONLY | O_NONBLOCK );
+    if ( sysClass.exists() && sysClass.count() > 2 ) {
+        QString sysClassPath = sysClass.absFilePath( sysClass[2] + "/brightness" );
+        int fd = ::open( sysClassPath, O_WRONLY|O_NONBLOCK );
         if ( fd ) {
-            char buf[100];
-            int len = ::snprintf( &buf[0], sizeof buf, "%d", bright );
-            res = ( ::write( fd, &buf[0], len ) == 0 );
-            ::close( fd );
+                char buf[100];
+                int val = bright * displayBrightnessResolution() / 255;
+                int len = ::snprintf( &buf[0], sizeof buf, "%d", val );
+                res = ( ::write( fd, &buf[0], len ) == 0 );
+                ::close( fd );
         }
@@ -242,3 +237,3 @@ int MyPal::displayBrightnessResolution() const
         case Model_MyPal_716:
-            return 7;
+            return 255;
         default:
@@ -254,5 +249,5 @@ bool MyPal::setDisplayStatus ( bool on )
     sysClass.setFilter(QDir::Dirs);
-    if ( sysClass.exists() ) {
-        QString sysClassPath = sysClass.absFilePath( "/sys/class/lcd/pxafb/power" );
-        int fd = ::open( sysClassPath, O_WRONLY | O_NONBLOCK );
+    if ( sysClass.exists() && sysClass.count() > 2 ) {
+        QString sysClassPath = sysClass.absFilePath( sysClass[2] + "/power" );
+        int fd = ::open( sysClassPath, O_WRONLY|O_NONBLOCK );
         if ( fd ) {
@@ -264,2 +259,4 @@ bool MyPal::setDisplayStatus ( bool on )
         }
+    } else {
+        res = OAbstractMobileDevice::setDisplayStatus(on);
     }