Changes in uspace/drv/hid/atkbd/atkbd.c [5d50c419:d420b22] in mainline

File:

• uspace/drv/hid/atkbd/atkbd.c (modified) (6 diffs)

uspace/drv/hid/atkbd/atkbd.c

@@ -1 +1 @@
 /*
- * Copyright (c) 2017 Jiri Svoboda
  * Copyright (c) 2011 Jan Vesely
  * Copyright (c) 2009 Vineeth Pillai
@@ -202 +201 @@
 static int polling(void *arg)
 {
-        at_kbd_t *kbd = arg;
-        size_t nwr;
-        int rc;
+        const at_kbd_t *kbd = arg;
+        
+        assert(kbd);
+        assert(kbd->parent_sess);
+        
+        async_exch_t *parent_exch = async_exchange_begin(kbd->parent_sess);
         
         while (true) {
+                if (!parent_exch)
+                        parent_exch = async_exchange_begin(kbd->parent_sess);
+
                 uint8_t code = 0;
-                rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                if (rc != EOK)
+                ssize_t size = chardev_read(parent_exch, &code, 1);
+                if (size != 1)
                         return EIO;
                 
@@ -219 +224 @@
                         map_size = sizeof(scanmap_e0) / sizeof(unsigned int);
                         
-                        rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                        if (rc != EOK)
+                        size = chardev_read(parent_exch, &code, 1);
+                        if (size != 1)
                                 return EIO;
                 } else if (code == KBD_SCANCODE_SET_EXTENDED_SPECIAL) {
-                        rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                        if (rc != EOK)
+                        size = chardev_read(parent_exch, &code, 1);
+                        if (size != 1)
                                 return EIO;
                         if (code != 0x14)
                                 continue;
 
-                        rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                        if (rc != EOK)
+                        size = chardev_read(parent_exch, &code, 1);
+                        if (size != 1)
                                 return EIO;
                         if (code != 0x77)
                                 continue;
 
-                        rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                        if (rc != EOK)
+                        size = chardev_read(parent_exch, &code, 1);
+                        if (size != 1)
                                 return EIO;
                         if (code != 0xe1)
                                 continue;
 
-                        rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                        if (rc != EOK)
+                        size = chardev_read(parent_exch, &code, 1);
+                        if (size != 1)
                                 return EIO;
                         if (code != 0xf0)
                                 continue;
 
-                        rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                        if (rc != EOK)
+                        size = chardev_read(parent_exch, &code, 1);
+                        if (size != 1)
                                 return EIO;
                         if (code != 0x14)
                                 continue;
 
-                        rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                        if (rc != EOK)
+                        size = chardev_read(parent_exch, &code, 1);
+                        if (size != 1)
                                 return EIO;
                         if (code != 0xf0)
                                 continue;
 
-                        rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                        if (rc != EOK)
+                        size = chardev_read(parent_exch, &code, 1);
+                        if (size != 1)
                                 return EIO;
                         if (code == 0x77)
@@ -274 +279 @@
                 if (code == KBD_SCANCODE_KEY_RELEASE) {
                         type = KEY_RELEASE;
-                        rc = chardev_read(kbd->chardev, &code, 1, &nwr);
-                        if (rc != EOK)
+                        size = chardev_read(parent_exch, &code, 1);
+                        if (size != 1)
                                 return EIO;
                 } else {
@@ -357 +362 @@
 int at_kbd_init(at_kbd_t *kbd, ddf_dev_t *dev)
 {
-        async_sess_t *parent_sess;
-        int rc;
-        
         assert(kbd);
         assert(dev);
         
         kbd->client_sess = NULL;
-        parent_sess = ddf_dev_parent_sess_get(dev);
-        if (parent_sess == NULL) {
+        kbd->parent_sess = ddf_dev_parent_sess_get(dev);
+        
+        if (!kbd->parent_sess) {
                 ddf_msg(LVL_ERROR, "Failed creating parent session.");
-                rc = EIO;
-                goto error;
-        }
-        
-        rc = chardev_open(parent_sess, &kbd->chardev);
-        if (rc != EOK) {
-                ddf_msg(LVL_ERROR, "Failed opening character device.");
                 return EIO;
         }
@@ -411 +407 @@
         fibril_add_ready(kbd->polling_fibril);
         return EOK;
-error:
-        chardev_close(kbd->chardev);
-        kbd->chardev = NULL;
-        return rc;
 }