Changeset a46e56b in mainline for uspace/drv/hid


Ignore:
Timestamp:
2018-03-22T06:49:35Z (8 years ago)
Author:
Jakub Jermar <jakub@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
77f0a1d
Parents:
3e242d2
git-author:
Jakub Jermar <jakub@…> (2018-03-21 23:29:06)
git-committer:
Jakub Jermar <jakub@…> (2018-03-22 06:49:35)
Message:

Prefer handle over ID in naming handle variables

Location:
uspace/drv/hid
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • uspace/drv/hid/adb-kbd/adb-kbd.c

    r3e242d2 ra46e56b  
    130130}
    131131
    132 static void adb_kbd_events(cap_call_handle_t iid, ipc_call_t *icall, void *arg)
     132static void adb_kbd_events(cap_call_handle_t icall_handle, ipc_call_t *icall, void *arg)
    133133{
    134134        adb_kbd_t *kbd = (adb_kbd_t *) arg;
    … …  
    138138
    139139                ipc_call_t call;
    140                 cap_call_handle_t callid = async_get_call(&call);
     140                cap_call_handle_t chandle = async_get_call(&call);
    141141
    142142                errno_t retval = EOK;
    … …  
    154154                        retval = ENOENT;
    155155                }
    156                 async_answer_0(callid, retval);
     156                async_answer_0(chandle, retval);
    157157        }
    158158}
    … …  
    190190
    191191/** Handle client connection */
    192 static void adb_kbd_conn(cap_call_handle_t iid, ipc_call_t *icall, void *arg)
    193 {
    194         cap_call_handle_t callid;
     192static void adb_kbd_conn(cap_call_handle_t icall_handle, ipc_call_t *icall, void *arg)
     193{
     194        cap_call_handle_t chandle;
    195195        ipc_call_t call;
    196196        sysarg_t method;
    … …  
    200200         * Answer the first IPC_M_CONNECT_ME_TO call.
    201201         */
    202         async_answer_0(iid, EOK);
     202        async_answer_0(icall_handle, EOK);
    203203
    204204        kbd = (adb_kbd_t *)ddf_dev_data_get(ddf_fun_get_dev((ddf_fun_t *)arg));
    205205
    206206        while (true) {
    207                 callid = async_get_call(&call);
     207                chandle = async_get_call(&call);
    208208                method = IPC_GET_IMETHOD(call);
    209209
    210210                if (!method) {
    211211                        /* The other side has hung up. */
    212                         async_answer_0(callid, EOK);
     212                        async_answer_0(chandle, EOK);
    213213                        return;
    214214                }
    … …  
    218218                if (sess != NULL) {
    219219                        kbd->client_sess = sess;
    220                         async_answer_0(callid, EOK);
     220                        async_answer_0(chandle, EOK);
    221221                } else {
    222                         async_answer_0(callid, EINVAL);
     222                        async_answer_0(chandle, EINVAL);
    223223                }
    224224        }

  • uspace/drv/hid/adb-mouse/adb-mouse.c

    r3e242d2 ra46e56b  
    8888}
    8989
    90 static void adb_mouse_events(cap_call_handle_t iid, ipc_call_t *icall, void *arg)
     90static void adb_mouse_events(cap_call_handle_t icall_handle, ipc_call_t *icall, void *arg)
    9191{
    9292        adb_mouse_t *mouse = (adb_mouse_t *) arg;
    … …  
    9595        while (true) {
    9696                ipc_call_t call;
    97                 cap_call_handle_t callid = async_get_call(&call);
     97                cap_call_handle_t chandle = async_get_call(&call);
    9898
    9999                errno_t retval = EOK;
    … …  
    112112                }
    113113
    114                 async_answer_0(callid, retval);
     114                async_answer_0(chandle, retval);
    115115        }
    116116}
    … …  
    200200
    201201/** Handle client connection */
    202 static void adb_mouse_conn(cap_call_handle_t iid, ipc_call_t *icall, void *arg)
    203 {
    204         cap_call_handle_t callid;
     202static void adb_mouse_conn(cap_call_handle_t icall_handle, ipc_call_t *icall, void *arg)
     203{
     204        cap_call_handle_t chandle;
    205205        ipc_call_t call;
    206206        sysarg_t method;
    … …  
    210210         * Answer the first IPC_M_CONNECT_ME_TO call.
    211211         */
    212         async_answer_0(iid, EOK);
     212        async_answer_0(icall_handle, EOK);
    213213
    214214        mouse = (adb_mouse_t *)ddf_dev_data_get(ddf_fun_get_dev((ddf_fun_t *)arg));
    215215
    216216        while (true) {
    217                 callid = async_get_call(&call);
     217                chandle = async_get_call(&call);
    218218                method = IPC_GET_IMETHOD(call);
    219219
    220220                if (!method) {
    221221                        /* The other side has hung up. */
    222                         async_answer_0(callid, EOK);
     222                        async_answer_0(chandle, EOK);
    223223                        return;
    224224                }
    … …  
    228228                if (sess != NULL) {
    229229                        mouse->client_sess = sess;
    230                         async_answer_0(callid, EOK);
     230                        async_answer_0(chandle, EOK);
    231231                } else {
    232                         async_answer_0(callid, EINVAL);
     232                        async_answer_0(chandle, EINVAL);
    233233                }
    234234        }

  • uspace/drv/hid/atkbd/atkbd.c

    r3e242d2 ra46e56b  
    293293 *
    294294 * @param fun     Device function handling the call.
    295  * @param icallid Call id.
     295 * @param icall_handle Call id.
    296296 * @param icall   Call data.
    297297 *
    298298 */
    299299static void default_connection_handler(ddf_fun_t *fun,
    300     cap_call_handle_t icallid, ipc_call_t *icall)
     300    cap_call_handle_t icall_handle, ipc_call_t *icall)
    301301{
    302302        const sysarg_t method = IPC_GET_IMETHOD(*icall);
    … …  
    306306        switch (method) {
    307307        case KBDEV_SET_IND:
    308                 async_answer_0(icallid, ENOTSUP);
     308                async_answer_0(icall_handle, ENOTSUP);
    309309                break;
    310310        /*
    … …  
    319319                        ddf_msg(LVL_WARN,
    320320                            "Failed creating callback session");
    321                         async_answer_0(icallid, EAGAIN);
     321                        async_answer_0(icall_handle, EAGAIN);
    322322                        break;
    323323                }
    … …  
    326326                        kbd->client_sess = sess;
    327327                        ddf_msg(LVL_DEBUG, "Set client session");
    328                         async_answer_0(icallid, EOK);
     328                        async_answer_0(icall_handle, EOK);
    329329                } else {
    330330                        ddf_msg(LVL_ERROR, "Client session already set");
    331                         async_answer_0(icallid, ELIMIT);
     331                        async_answer_0(icall_handle, ELIMIT);
    332332                }
    333333
    … …  
    335335        default:
    336336                ddf_msg(LVL_ERROR, "Unknown method: %d.", (int)method);
    337                 async_answer_0(icallid, EINVAL);
     337                async_answer_0(icall_handle, EINVAL);
    338338                break;
    339339        }

  • uspace/drv/hid/ps2mouse/ps2mouse.c

    r3e242d2 ra46e56b  
    400400 *
    401401 * @param fun Device function handling the call.
    402  * @param icallid Call id.
     402 * @param icall_handle Call id.
    403403 * @param icall Call data.
    404404 */
    405405void default_connection_handler(ddf_fun_t *fun,
    406     cap_call_handle_t icallid, ipc_call_t *icall)
     406    cap_call_handle_t icall_handle, ipc_call_t *icall)
    407407{
    408408        const sysarg_t method = IPC_GET_IMETHOD(*icall);
    … …  
    419419                        ddf_msg(LVL_WARN,
    420420                            "Failed creating client callback session");
    421                         async_answer_0(icallid, EAGAIN);
     421                        async_answer_0(icall_handle, EAGAIN);
    422422                        break;
    423423                }
    … …  
    425425                        mouse->client_sess = sess;
    426426                        ddf_msg(LVL_DEBUG, "Set client session");
    427                         async_answer_0(icallid, EOK);
     427                        async_answer_0(icall_handle, EOK);
    428428                } else {
    429429                        ddf_msg(LVL_ERROR, "Client session already set");
    430                         async_answer_0(icallid, ELIMIT);
     430                        async_answer_0(icall_handle, ELIMIT);
    431431                }
    432432                break;
    433433        default:
    434434                ddf_msg(LVL_ERROR, "Unknown method: %d.", (int)method);
    435                 async_answer_0(icallid, EINVAL);
     435                async_answer_0(icall_handle, EINVAL);
    436436                break;
    437437        }

  • uspace/drv/hid/usbhid/kbd/kbddev.c

    r3e242d2 ra46e56b  
    157157 *
    158158 * @param fun Device function handling the call.
    159  * @param icallid Call id.
     159 * @param icall_handle Call id.
    160160 * @param icall Call data.
    161161 */
    162162static void default_connection_handler(ddf_fun_t *fun,
    163     cap_call_handle_t icallid, ipc_call_t *icall)
     163    cap_call_handle_t icall_handle, ipc_call_t *icall)
    164164{
    165165        const sysarg_t method = IPC_GET_IMETHOD(*icall);
    … …  
    171171                kbd_dev->mods = IPC_GET_ARG1(*icall);
    172172                usb_kbd_set_led(kbd_dev->hid_dev, kbd_dev);
    173                 async_answer_0(icallid, EOK);
     173                async_answer_0(icall_handle, EOK);
    174174                break;
    175175        /* This might be ugly but async_callback_receive_start makes no
    … …  
    181181                        usb_log_warning(
    182182                            "Failed to create start console session.\n");
    183                         async_answer_0(icallid, EAGAIN);
     183                        async_answer_0(icall_handle, EAGAIN);
    184184                        break;
    185185                }
    … …  
    187187                        kbd_dev->client_sess = sess;
    188188                        usb_log_debug("%s: OK", __FUNCTION__);
    189                         async_answer_0(icallid, EOK);
     189                        async_answer_0(icall_handle, EOK);
    190190                } else {
    191191                        usb_log_error("%s: console session already set",
    192192                           __FUNCTION__);
    193                         async_answer_0(icallid, ELIMIT);
     193                        async_answer_0(icall_handle, ELIMIT);
    194194                }
    195195                break;
    … …  
    197197                        usb_log_error("%s: Unknown method: %d.",
    198198                            __FUNCTION__, (int) method);
    199                         async_answer_0(icallid, EINVAL);
     199                        async_answer_0(icall_handle, EINVAL);
    200200                        break;
    201201        }

  • uspace/drv/hid/usbhid/mouse/mousedev.c

    r3e242d2 ra46e56b  
    110110 *
    111111 * @param fun Device function handling the call.
    112  * @param icallid Call id.
     112 * @param icall_handle Call id.
    113113 * @param icall Call data.
    114114 */
    115115static void default_connection_handler(ddf_fun_t *fun,
    116     cap_call_handle_t icallid, ipc_call_t *icall)
     116    cap_call_handle_t icall_handle, ipc_call_t *icall)
    117117{
    118118        usb_mouse_t *mouse_dev = ddf_fun_data_get(fun);
    … …  
    120120        if (mouse_dev == NULL) {
    121121                usb_log_debug("%s: Missing parameters.", __FUNCTION__);
    122                 async_answer_0(icallid, EINVAL);
     122                async_answer_0(icall_handle, EINVAL);
    123123                return;
    124124        }
    … …  
    135135                        usb_log_debug("Console session to %s set ok (%p).",
    136136                            ddf_fun_get_name(fun), sess);
    137                         async_answer_0(icallid, EOK);
     137                        async_answer_0(icall_handle, EOK);
    138138                } else {
    139139                        usb_log_error("Console session to %s already set.",
    140140                            ddf_fun_get_name(fun));
    141                         async_answer_0(icallid, ELIMIT);
     141                        async_answer_0(icall_handle, ELIMIT);
    142142                        async_hangup(sess);
    143143                }
    144144        } else {
    145145                usb_log_debug("%s: Invalid function.", __FUNCTION__);
    146                 async_answer_0(icallid, EINVAL);
     146                async_answer_0(icall_handle, EINVAL);
    147147        }
    148148}

  • uspace/drv/hid/usbhid/multimedia/multimedia.c

    r3e242d2 ra46e56b  
    8181 *
    8282 * @param fun Device function handling the call.
    83  * @param icallid Call id.
     83 * @param icall_handle Call id.
    8484 * @param icall Call data.
    8585 */
    8686static void default_connection_handler(ddf_fun_t *fun,
    87     cap_call_handle_t icallid, ipc_call_t *icall)
     87    cap_call_handle_t icall_handle, ipc_call_t *icall)
    8888{
    8989        usb_log_debug(NAME " default_connection_handler()");
    … …  
    9898                        usb_log_debug(NAME " Saved session to console: %p",
    9999                            sess);
    100                         async_answer_0(icallid, EOK);
     100                        async_answer_0(icall_handle, EOK);
    101101                } else
    102                         async_answer_0(icallid, ELIMIT);
     102                        async_answer_0(icall_handle, ELIMIT);
    103103        } else
    104                 async_answer_0(icallid, EINVAL);
     104                async_answer_0(icall_handle, EINVAL);
    105105}
    106106

  • uspace/drv/hid/xtkbd/xtkbd.c

    r3e242d2 ra46e56b  
    329329 *
    330330 * @param fun     Device function handling the call.
    331  * @param icallid Call id.
     331 * @param icall_handle Call id.
    332332 * @param icall   Call data.
    333333 *
    334334 */
    335335static void default_connection_handler(ddf_fun_t *fun,
    336     cap_call_handle_t icallid, ipc_call_t *icall)
     336    cap_call_handle_t icall_handle, ipc_call_t *icall)
    337337{
    338338        const sysarg_t method = IPC_GET_IMETHOD(*icall);
    … …  
    357357                errno_t rc = chardev_write(kbd->chardev, &cmds[0], 1, &nwr);
    358358                if (rc != EOK) {
    359                         async_answer_0(icallid, rc);
     359                        async_answer_0(icall_handle, rc);
    360360                        break;
    361361                }
    362362
    363363                rc = chardev_write(kbd->chardev, &cmds[1], 1, &nwr);
    364                 async_answer_0(icallid, rc);
     364                async_answer_0(icall_handle, rc);
    365365                break;
    366366        /*
    … …  
    375375                        ddf_msg(LVL_WARN,
    376376                            "Failed creating callback session");
    377                         async_answer_0(icallid, EAGAIN);
     377                        async_answer_0(icall_handle, EAGAIN);
    378378                        break;
    379379                }
    … …  
    382382                        kbd->client_sess = sess;
    383383                        ddf_msg(LVL_DEBUG, "Set client session");
    384                         async_answer_0(icallid, EOK);
     384                        async_answer_0(icall_handle, EOK);
    385385                } else {
    386386                        ddf_msg(LVL_ERROR, "Client session already set");
    387                         async_answer_0(icallid, ELIMIT);
     387                        async_answer_0(icall_handle, ELIMIT);
    388388                }
    389389
    … …  
    391391        default:
    392392                ddf_msg(LVL_ERROR, "Unknown method: %d.", (int)method);
    393                 async_answer_0(icallid, EINVAL);
     393                async_answer_0(icall_handle, EINVAL);
    394394                break;
    395395        }
Note: See TracChangeset for help on using the changeset viewer.