[Qemu-devel] [PATCH 1/2] pxa2xx_keypad: enhance emulation of KPAS, KPASMKP regs

[Qemu-devel] [PATCH 1/2] pxa2xx_keypad: enhance emulation of KPAS, KPASMKP regs

[Vasily Khoruzhick]

Add emulation of KPAS register and proper emulation of
KPASMKP regs, so now driver supports multipresses and properly
works with Linux driver.

Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com>
---
 hw/pxa2xx_keypad.c |  114 ++++++++++++++++++++++++++++------------------------
 1 files changed, 62 insertions(+), 52 deletions(-)

diff --git a/hw/pxa2xx_keypad.c b/hw/pxa2xx_keypad.c
index 4c99917..5b8890b 100644
--- a/hw/pxa2xx_keypad.c
+++ b/hw/pxa2xx_keypad.c
@@ -82,22 +82,39 @@
 struct PXA2xxKeyPadState {
     qemu_irq    irq;
     struct  keymap *map;
+    int         pressed_cnt;
 
     uint32_t    kpc;
     uint32_t    kpdk;
     uint32_t    kprec;
     uint32_t    kpmk;
     uint32_t    kpas;
-    uint32_t    kpasmkp0;
-    uint32_t    kpasmkp1;
-    uint32_t    kpasmkp2;
-    uint32_t    kpasmkp3;
+    uint32_t    kpasmkp[4];
     uint32_t    kpkdi;
 };
 
+static void pxa27x_keypad_find_pressed_key(PXA2xxKeyPadState *kp, int *row, int *col)
+{
+    int i;
+    for (i = 0; i < 4; i++)
+    {
+        *col = i * 2;
+        for (*row = 0; *row < 8; (*row)++) {
+            if (kp->kpasmkp[i] & (1 << *row))
+                return;
+        }
+        *col = i * 2 + 1;
+        for (*row = 0; *row < 8; (*row)++) {
+            if (kp->kpasmkp[i] & (1 << (*row + 16)))
+                return;
+        }
+    }
+}
+
 static void pxa27x_keyboard_event (PXA2xxKeyPadState *kp, int keycode)
 {
-    int row, col,rel;
+    int row, col, rel, assert_irq = 0;
+    uint32_t val;
 
     if(!(kp->kpc & KPC_ME)) /* skip if not enabled */
         return;
@@ -108,46 +125,39 @@ static void pxa27x_keyboard_event (PXA2xxKeyPadState *kp, int keycode)
 
         rel = (keycode & 0x80) ? 1 : 0; /* key release from qemu */
         keycode &= ~(0x80); /* strip qemu key release bit */
+
         row = kp->map[keycode].row;
         col = kp->map[keycode].column;
         if(row == -1 || col == -1)
             return;
-        switch (col) {
-        case 0:
-        case 1:
-            if(rel)
-                kp->kpasmkp0 = ~(0xffffffff);
-            else
-                kp->kpasmkp0 |= KPASMKPx_MKC(row,col);
-            break;
-        case 2:
-        case 3:
-            if(rel)
-                kp->kpasmkp1 = ~(0xffffffff);
-            else
-                kp->kpasmkp1 |= KPASMKPx_MKC(row,col);
-            break;
-        case 4:
-        case 5:
-            if(rel)
-                kp->kpasmkp2 = ~(0xffffffff);
-            else
-                kp->kpasmkp2 |= KPASMKPx_MKC(row,col);
-            break;
-        case 6:
-        case 7:
-            if(rel)
-                kp->kpasmkp3 = ~(0xffffffff);
-            else
-                kp->kpasmkp3 |= KPASMKPx_MKC(row,col);
-            break;
-        } /* switch */
+
+        val = KPASMKPx_MKC(row, col);
+        if (rel) {
+            if (kp->kpasmkp[col / 2] & val) {
+                kp->kpasmkp[col / 2] &= ~val;
+                kp->pressed_cnt--;
+                assert_irq = 1;
+            }
+        } else {
+            if (!(kp->kpasmkp[col / 2] & val)) {
+                kp->kpasmkp[col / 2] |= val;
+                kp->pressed_cnt++;
+                assert_irq = 1;
+            }
+        }
+        kp->kpas = ((kp->pressed_cnt & 0x1f) << 26) | (0xf << 4) | 0xf;
+        if (kp->pressed_cnt == 1) {
+            kp->kpas &= ~((0xf << 4) | 0xf);
+            if (rel)
+                pxa27x_keypad_find_pressed_key(kp, &row, &col);
+            kp->kpas |= ((row & 0xf) << 4) | (col & 0xf);
+        }
         goto out;
     }
     return;
 
 out:
-    if(kp->kpc & KPC_MIE) {
+    if (assert_irq && (kp->kpc & KPC_MIE)) {
         kp->kpc |= KPC_MI;
         qemu_irq_raise(kp->irq);
     }
@@ -194,16 +204,16 @@ static uint32_t pxa2xx_keypad_read(void *opaque, target_phys_addr_t offset)
         return s->kpas;
         break;
     case KPASMKP0:
-        return s->kpasmkp0;
+        return s->kpasmkp[0];
         break;
     case KPASMKP1:
-        return s->kpasmkp1;
+        return s->kpasmkp[1];
         break;
     case KPASMKP2:
-        return s->kpasmkp2;
+        return s->kpasmkp[2];
         break;
     case KPASMKP3:
-        return s->kpasmkp3;
+        return s->kpasmkp[3];
         break;
     case KPKDI:
         return s->kpkdi;
@@ -237,16 +247,16 @@ static void pxa2xx_keypad_write(void *opaque,
         s->kpas = value;
         break;
     case KPASMKP0:
-        s->kpasmkp0 = value;
+        s->kpasmkp[0] = value;
         break;
     case KPASMKP1:
-        s->kpasmkp1 = value;
+        s->kpasmkp[1] = value;
         break;
     case KPASMKP2:
-        s->kpasmkp2 = value;
+        s->kpasmkp[2] = value;
         break;
     case KPASMKP3:
-        s->kpasmkp3 = value;
+        s->kpasmkp[3] = value;
         break;
     case KPKDI:
         s->kpkdi = value;
@@ -278,10 +288,10 @@ static void pxa2xx_keypad_save(QEMUFile *f, void *opaque)
     qemu_put_be32s(f, &s->kprec);
     qemu_put_be32s(f, &s->kpmk);
     qemu_put_be32s(f, &s->kpas);
-    qemu_put_be32s(f, &s->kpasmkp0);
-    qemu_put_be32s(f, &s->kpasmkp1);
-    qemu_put_be32s(f, &s->kpasmkp2);
-    qemu_put_be32s(f, &s->kpasmkp3);
+    qemu_put_be32s(f, &s->kpasmkp[0]);
+    qemu_put_be32s(f, &s->kpasmkp[1]);
+    qemu_put_be32s(f, &s->kpasmkp[2]);
+    qemu_put_be32s(f, &s->kpasmkp[3]);
     qemu_put_be32s(f, &s->kpkdi);
 
 }
@@ -295,10 +305,10 @@ static int pxa2xx_keypad_load(QEMUFile *f, void *opaque, int version_id)
     qemu_get_be32s(f, &s->kprec);
     qemu_get_be32s(f, &s->kpmk);
     qemu_get_be32s(f, &s->kpas);
-    qemu_get_be32s(f, &s->kpasmkp0);
-    qemu_get_be32s(f, &s->kpasmkp1);
-    qemu_get_be32s(f, &s->kpasmkp2);
-    qemu_get_be32s(f, &s->kpasmkp3);
+    qemu_get_be32s(f, &s->kpasmkp[0]);
+    qemu_get_be32s(f, &s->kpasmkp[1]);
+    qemu_get_be32s(f, &s->kpasmkp[2]);
+    qemu_get_be32s(f, &s->kpasmkp[3]);
     qemu_get_be32s(f, &s->kpkdi);
 
     return 0;
-- 
1.7.4

[Qemu-devel] [PATCH 2/2] pxa2xx_keypad: Handle 0xe0xx keycodes

[Vasily Khoruzhick]

Add handling of 0xe0xx keycodes to pxa2xx_driver.
Extended keycodes in keymap should be marked with most significant
bit set (i.e. 0x80). Without this patch it's not possible to handle
i.e. cursor keys.

Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com>
---
 hw/pxa2xx_keypad.c |   10 ++++++++++
 1 files changed, 10 insertions(+), 0 deletions(-)

diff --git a/hw/pxa2xx_keypad.c b/hw/pxa2xx_keypad.c
index 5b8890b..d77dbf1 100644
--- a/hw/pxa2xx_keypad.c
+++ b/hw/pxa2xx_keypad.c
@@ -83,6 +83,7 @@ struct PXA2xxKeyPadState {
     qemu_irq    irq;
     struct  keymap *map;
     int         pressed_cnt;
+    int         alt_code;
 
     uint32_t    kpc;
     uint32_t    kpdk;
@@ -116,6 +117,11 @@ static void pxa27x_keyboard_event (PXA2xxKeyPadState *kp, int keycode)
     int row, col, rel, assert_irq = 0;
     uint32_t val;
 
+    if (keycode == 0xe0) {
+        kp->alt_code = 1;
+        return;
+    }
+
     if(!(kp->kpc & KPC_ME)) /* skip if not enabled */
         return;
 
@@ -125,6 +131,10 @@ static void pxa27x_keyboard_event (PXA2xxKeyPadState *kp, int keycode)
 
         rel = (keycode & 0x80) ? 1 : 0; /* key release from qemu */
         keycode &= ~(0x80); /* strip qemu key release bit */
+        if (kp->alt_code) {
+            keycode |= 0x80;
+            kp->alt_code = 0;
+        }
 
         row = kp->map[keycode].row;
         col = kp->map[keycode].column;
-- 
1.7.4

[Qemu-devel] Re: [PATCH 1/2] pxa2xx_keypad: enhance emulation of KPAS, KPASMKP regs

[Vasily Khoruzhick]

On Tuesday 15 February 2011 15:27:28 Vasily Khoruzhick wrote:
> Add emulation of KPAS register and proper emulation of
> KPASMKP regs, so now driver supports multipresses and properly
> works with Linux driver.
> 

Ping

Re: [Qemu-devel] [PATCH 1/2] pxa2xx_keypad: enhance emulation of KPAS, KPASMKP regs

[Aurelien Jarno]

On Tue, Feb 15, 2011 at 03:27:28PM +0200, Vasily Khoruzhick wrote:
> Add emulation of KPAS register and proper emulation of
> KPASMKP regs, so now driver supports multipresses and properly
> works with Linux driver.
> 
> Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com>
> ---
>  hw/pxa2xx_keypad.c |  114 ++++++++++++++++++++++++++++------------------------
>  1 files changed, 62 insertions(+), 52 deletions(-)

Thanks, both applied.

> diff --git a/hw/pxa2xx_keypad.c b/hw/pxa2xx_keypad.c
> index 4c99917..5b8890b 100644
> --- a/hw/pxa2xx_keypad.c
> +++ b/hw/pxa2xx_keypad.c
> @@ -82,22 +82,39 @@
>  struct PXA2xxKeyPadState {
>      qemu_irq    irq;
>      struct  keymap *map;
> +    int         pressed_cnt;
>  
>      uint32_t    kpc;
>      uint32_t    kpdk;
>      uint32_t    kprec;
>      uint32_t    kpmk;
>      uint32_t    kpas;
> -    uint32_t    kpasmkp0;
> -    uint32_t    kpasmkp1;
> -    uint32_t    kpasmkp2;
> -    uint32_t    kpasmkp3;
> +    uint32_t    kpasmkp[4];
>      uint32_t    kpkdi;
>  };
>  
> +static void pxa27x_keypad_find_pressed_key(PXA2xxKeyPadState *kp, int *row, int *col)
> +{
> +    int i;
> +    for (i = 0; i < 4; i++)
> +    {
> +        *col = i * 2;
> +        for (*row = 0; *row < 8; (*row)++) {
> +            if (kp->kpasmkp[i] & (1 << *row))
> +                return;
> +        }
> +        *col = i * 2 + 1;
> +        for (*row = 0; *row < 8; (*row)++) {
> +            if (kp->kpasmkp[i] & (1 << (*row + 16)))
> +                return;
> +        }
> +    }
> +}
> +
>  static void pxa27x_keyboard_event (PXA2xxKeyPadState *kp, int keycode)
>  {
> -    int row, col,rel;
> +    int row, col, rel, assert_irq = 0;
> +    uint32_t val;
>  
>      if(!(kp->kpc & KPC_ME)) /* skip if not enabled */
>          return;
> @@ -108,46 +125,39 @@ static void pxa27x_keyboard_event (PXA2xxKeyPadState *kp, int keycode)
>  
>          rel = (keycode & 0x80) ? 1 : 0; /* key release from qemu */
>          keycode &= ~(0x80); /* strip qemu key release bit */
> +
>          row = kp->map[keycode].row;
>          col = kp->map[keycode].column;
>          if(row == -1 || col == -1)
>              return;
> -        switch (col) {
> -        case 0:
> -        case 1:
> -            if(rel)
> -                kp->kpasmkp0 = ~(0xffffffff);
> -            else
> -                kp->kpasmkp0 |= KPASMKPx_MKC(row,col);
> -            break;
> -        case 2:
> -        case 3:
> -            if(rel)
> -                kp->kpasmkp1 = ~(0xffffffff);
> -            else
> -                kp->kpasmkp1 |= KPASMKPx_MKC(row,col);
> -            break;
> -        case 4:
> -        case 5:
> -            if(rel)
> -                kp->kpasmkp2 = ~(0xffffffff);
> -            else
> -                kp->kpasmkp2 |= KPASMKPx_MKC(row,col);
> -            break;
> -        case 6:
> -        case 7:
> -            if(rel)
> -                kp->kpasmkp3 = ~(0xffffffff);
> -            else
> -                kp->kpasmkp3 |= KPASMKPx_MKC(row,col);
> -            break;
> -        } /* switch */
> +
> +        val = KPASMKPx_MKC(row, col);
> +        if (rel) {
> +            if (kp->kpasmkp[col / 2] & val) {
> +                kp->kpasmkp[col / 2] &= ~val;
> +                kp->pressed_cnt--;
> +                assert_irq = 1;
> +            }
> +        } else {
> +            if (!(kp->kpasmkp[col / 2] & val)) {
> +                kp->kpasmkp[col / 2] |= val;
> +                kp->pressed_cnt++;
> +                assert_irq = 1;
> +            }
> +        }
> +        kp->kpas = ((kp->pressed_cnt & 0x1f) << 26) | (0xf << 4) | 0xf;
> +        if (kp->pressed_cnt == 1) {
> +            kp->kpas &= ~((0xf << 4) | 0xf);
> +            if (rel)
> +                pxa27x_keypad_find_pressed_key(kp, &row, &col);
> +            kp->kpas |= ((row & 0xf) << 4) | (col & 0xf);
> +        }
>          goto out;
>      }
>      return;
>  
>  out:
> -    if(kp->kpc & KPC_MIE) {
> +    if (assert_irq && (kp->kpc & KPC_MIE)) {
>          kp->kpc |= KPC_MI;
>          qemu_irq_raise(kp->irq);
>      }
> @@ -194,16 +204,16 @@ static uint32_t pxa2xx_keypad_read(void *opaque, target_phys_addr_t offset)
>          return s->kpas;
>          break;
>      case KPASMKP0:
> -        return s->kpasmkp0;
> +        return s->kpasmkp[0];
>          break;
>      case KPASMKP1:
> -        return s->kpasmkp1;
> +        return s->kpasmkp[1];
>          break;
>      case KPASMKP2:
> -        return s->kpasmkp2;
> +        return s->kpasmkp[2];
>          break;
>      case KPASMKP3:
> -        return s->kpasmkp3;
> +        return s->kpasmkp[3];
>          break;
>      case KPKDI:
>          return s->kpkdi;
> @@ -237,16 +247,16 @@ static void pxa2xx_keypad_write(void *opaque,
>          s->kpas = value;
>          break;
>      case KPASMKP0:
> -        s->kpasmkp0 = value;
> +        s->kpasmkp[0] = value;
>          break;
>      case KPASMKP1:
> -        s->kpasmkp1 = value;
> +        s->kpasmkp[1] = value;
>          break;
>      case KPASMKP2:
> -        s->kpasmkp2 = value;
> +        s->kpasmkp[2] = value;
>          break;
>      case KPASMKP3:
> -        s->kpasmkp3 = value;
> +        s->kpasmkp[3] = value;
>          break;
>      case KPKDI:
>          s->kpkdi = value;
> @@ -278,10 +288,10 @@ static void pxa2xx_keypad_save(QEMUFile *f, void *opaque)
>      qemu_put_be32s(f, &s->kprec);
>      qemu_put_be32s(f, &s->kpmk);
>      qemu_put_be32s(f, &s->kpas);
> -    qemu_put_be32s(f, &s->kpasmkp0);
> -    qemu_put_be32s(f, &s->kpasmkp1);
> -    qemu_put_be32s(f, &s->kpasmkp2);
> -    qemu_put_be32s(f, &s->kpasmkp3);
> +    qemu_put_be32s(f, &s->kpasmkp[0]);
> +    qemu_put_be32s(f, &s->kpasmkp[1]);
> +    qemu_put_be32s(f, &s->kpasmkp[2]);
> +    qemu_put_be32s(f, &s->kpasmkp[3]);
>      qemu_put_be32s(f, &s->kpkdi);
>  
>  }
> @@ -295,10 +305,10 @@ static int pxa2xx_keypad_load(QEMUFile *f, void *opaque, int version_id)
>      qemu_get_be32s(f, &s->kprec);
>      qemu_get_be32s(f, &s->kpmk);
>      qemu_get_be32s(f, &s->kpas);
> -    qemu_get_be32s(f, &s->kpasmkp0);
> -    qemu_get_be32s(f, &s->kpasmkp1);
> -    qemu_get_be32s(f, &s->kpasmkp2);
> -    qemu_get_be32s(f, &s->kpasmkp3);
> +    qemu_get_be32s(f, &s->kpasmkp[0]);
> +    qemu_get_be32s(f, &s->kpasmkp[1]);
> +    qemu_get_be32s(f, &s->kpasmkp[2]);
> +    qemu_get_be32s(f, &s->kpasmkp[3]);
>      qemu_get_be32s(f, &s->kpkdi);
>  
>      return 0;
> -- 
> 1.7.4
> 
> 
> 

-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net