Changeset a46e56b in mainline for uspace/srv/hid/output/output.c


Ignore:
Timestamp:
2018-03-22T06:49:35Z (6 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

File:
1 edited

Legend:

Unmodified
Added
Removed

  • uspace/srv/hid/output/output.c

    r3e242d2 ra46e56b  
    7878}
    7979
    80 static void srv_yield(cap_call_handle_t iid, ipc_call_t *icall)
     80static void srv_yield(cap_call_handle_t icall_handle, ipc_call_t *icall)
    8181{
    8282        errno_t ret = EOK;
    … …  
    9090        }
    9191
    92         async_answer_0(iid, ret);
    93 }
    94 
    95 static void srv_claim(cap_call_handle_t iid, ipc_call_t *icall)
     92        async_answer_0(icall_handle, ret);
     93}
     94
     95static void srv_claim(cap_call_handle_t icall_handle, ipc_call_t *icall)
    9696{
    9797        errno_t ret = EOK;
    … …  
    105105        }
    106106
    107         async_answer_0(iid, ret);
    108 }
    109 
    110 static void srv_get_dimensions(cap_call_handle_t iid, ipc_call_t *icall)
     107        async_answer_0(icall_handle, ret);
     108}
     109
     110static void srv_get_dimensions(cap_call_handle_t icall_handle, ipc_call_t *icall)
    111111{
    112112        sysarg_t cols = MAX_COLS;
    … …  
    118118        }
    119119
    120         async_answer_2(iid, EOK, cols, rows);
    121 }
    122 
    123 static void srv_get_caps(cap_call_handle_t iid, ipc_call_t *icall)
     120        async_answer_2(icall_handle, EOK, cols, rows);
     121}
     122
     123static void srv_get_caps(cap_call_handle_t icall_handle, ipc_call_t *icall)
    124124{
    125125        console_caps_t caps = 0;
    … …  
    131131        }
    132132
    133         async_answer_1(iid, EOK, caps);
    134 }
    135 
    136 static frontbuf_t *resolve_frontbuf(sysarg_t handle, cap_call_handle_t iid)
     133        async_answer_1(icall_handle, EOK, caps);
     134}
     135
     136static frontbuf_t *resolve_frontbuf(sysarg_t handle, cap_call_handle_t icall_handle)
    137137{
    138138        frontbuf_t *frontbuf = NULL;
    … …  
    145145
    146146        if (frontbuf == NULL) {
    147                 async_answer_0(iid, ENOENT);
     147                async_answer_0(icall_handle, ENOENT);
    148148                return NULL;
    149149        }
    … …  
    152152}
    153153
    154 static void srv_frontbuf_create(cap_call_handle_t iid, ipc_call_t *icall)
     154static void srv_frontbuf_create(cap_call_handle_t icall_handle, ipc_call_t *icall)
    155155{
    156156        frontbuf_t *frontbuf = (frontbuf_t *) malloc(sizeof(frontbuf_t));
    157157        if (frontbuf == NULL) {
    158                 async_answer_0(iid, ENOMEM);
     158                async_answer_0(icall_handle, ENOMEM);
    159159                return;
    160160        }
    … …  
    162162        link_initialize(&frontbuf->link);
    163163
    164         cap_call_handle_t callid;
    165         if (!async_share_out_receive(&callid, &frontbuf->size,
     164        cap_call_handle_t chandle;
     165        if (!async_share_out_receive(&chandle, &frontbuf->size,
    166166            &frontbuf->flags)) {
    167167                free(frontbuf);
    168                 async_answer_0(iid, EINVAL);
    169                 return;
    170         }
    171 
    172         errno_t rc = async_share_out_finalize(callid, &frontbuf->data);
     168                async_answer_0(icall_handle, EINVAL);
     169                return;
     170        }
     171
     172        errno_t rc = async_share_out_finalize(chandle, &frontbuf->data);
    173173        if ((rc != EOK) || (frontbuf->data == AS_MAP_FAILED)) {
    174174                free(frontbuf);
    175                 async_answer_0(iid, ENOMEM);
     175                async_answer_0(icall_handle, ENOMEM);
    176176                return;
    177177        }
    178178
    179179        list_append(&frontbuf->link, &frontbufs);
    180         async_answer_1(iid, EOK, (sysarg_t) frontbuf);
    181 }
    182 
    183 static void srv_frontbuf_destroy(cap_call_handle_t iid, ipc_call_t *icall)
    184 {
    185         frontbuf_t *frontbuf = resolve_frontbuf(IPC_GET_ARG1(*icall), iid);
     180        async_answer_1(icall_handle, EOK, (sysarg_t) frontbuf);
     181}
     182
     183static void srv_frontbuf_destroy(cap_call_handle_t icall_handle, ipc_call_t *icall)
     184{
     185        frontbuf_t *frontbuf = resolve_frontbuf(IPC_GET_ARG1(*icall), icall_handle);
    186186        if (frontbuf == NULL)
    187187                return;
    … …  
    191191        free(frontbuf);
    192192
    193         async_answer_0(iid, EOK);
    194 }
    195 
    196 static void srv_cursor_update(cap_call_handle_t iid, ipc_call_t *icall)
    197 {
    198         frontbuf_t *frontbuf = resolve_frontbuf(IPC_GET_ARG1(*icall), iid);
     193        async_answer_0(icall_handle, EOK);
     194}
     195
     196static void srv_cursor_update(cap_call_handle_t icall_handle, ipc_call_t *icall)
     197{
     198        frontbuf_t *frontbuf = resolve_frontbuf(IPC_GET_ARG1(*icall), icall_handle);
    199199        if (frontbuf == NULL)
    200200                return;
    … …  
    223223        }
    224224
    225         async_answer_0(iid, EOK);
    226 }
    227 
    228 static void srv_set_style(cap_call_handle_t iid, ipc_call_t *icall)
     225        async_answer_0(icall_handle, EOK);
     226}
     227
     228static void srv_set_style(cap_call_handle_t icall_handle, ipc_call_t *icall)
    229229{
    230230        list_foreach(outdevs, link, outdev_t, dev) {
    … …  
    234234        }
    235235
    236         async_answer_0(iid, EOK);
    237 }
    238 
    239 static void srv_set_color(cap_call_handle_t iid, ipc_call_t *icall)
     236        async_answer_0(icall_handle, EOK);
     237}
     238
     239static void srv_set_color(cap_call_handle_t icall_handle, ipc_call_t *icall)
    240240{
    241241        list_foreach(outdevs, link, outdev_t, dev) {
    … …  
    249249        }
    250250
    251         async_answer_0(iid, EOK);
    252 }
    253 
    254 static void srv_set_rgb_color(cap_call_handle_t iid, ipc_call_t *icall)
     251        async_answer_0(icall_handle, EOK);
     252}
     253
     254static void srv_set_rgb_color(cap_call_handle_t icall_handle, ipc_call_t *icall)
    255255{
    256256        list_foreach(outdevs, link, outdev_t, dev) {
    … …  
    260260        }
    261261
    262         async_answer_0(iid, EOK);
     262        async_answer_0(icall_handle, EOK);
    263263}
    264264
    … …  
    302302}
    303303
    304 static void srv_update(cap_call_handle_t iid, ipc_call_t *icall)
    305 {
    306         frontbuf_t *frontbuf = resolve_frontbuf(IPC_GET_ARG1(*icall), iid);
     304static void srv_update(cap_call_handle_t icall_handle, ipc_call_t *icall)
     305{
     306        frontbuf_t *frontbuf = resolve_frontbuf(IPC_GET_ARG1(*icall), icall_handle);
    307307        if (frontbuf == NULL)
    308308                return;
    … …  
    349349
    350350
    351         async_answer_0(iid, EOK);
    352 }
    353 
    354 static void srv_damage(cap_call_handle_t iid, ipc_call_t *icall)
    355 {
    356         frontbuf_t *frontbuf = resolve_frontbuf(IPC_GET_ARG1(*icall), iid);
     351        async_answer_0(icall_handle, EOK);
     352}
     353
     354static void srv_damage(cap_call_handle_t icall_handle, ipc_call_t *icall)
     355{
     356        frontbuf_t *frontbuf = resolve_frontbuf(IPC_GET_ARG1(*icall), icall_handle);
    357357        if (frontbuf == NULL)
    358358                return;
    … …  
    388388
    389389        }
    390         async_answer_0(iid, EOK);
    391 }
    392 
    393 static void client_connection(cap_call_handle_t iid, ipc_call_t *icall, void *arg)
     390        async_answer_0(icall_handle, EOK);
     391}
     392
     393static void client_connection(cap_call_handle_t icall_handle, ipc_call_t *icall, void *arg)
    394394{
    395395        /* Accept the connection */
    396         async_answer_0(iid, EOK);
     396        async_answer_0(icall_handle, EOK);
    397397
    398398        while (true) {
    399399                ipc_call_t call;
    400                 cap_call_handle_t callid = async_get_call(&call);
     400                cap_call_handle_t chandle = async_get_call(&call);
    401401
    402402                if (!IPC_GET_IMETHOD(call)) {
    403                         async_answer_0(callid, EOK);
     403                        async_answer_0(chandle, EOK);
    404404                        break;
    405405                }
    … …  
    407407                switch (IPC_GET_IMETHOD(call)) {
    408408                case OUTPUT_YIELD:
    409                         srv_yield(callid, &call);
     409                        srv_yield(chandle, &call);
    410410                        break;
    411411                case OUTPUT_CLAIM:
    412                         srv_claim(callid, &call);
     412                        srv_claim(chandle, &call);
    413413                        break;
    414414                case OUTPUT_GET_DIMENSIONS:
    415                         srv_get_dimensions(callid, &call);
     415                        srv_get_dimensions(chandle, &call);
    416416                        break;
    417417                case OUTPUT_GET_CAPS:
    418                         srv_get_caps(callid, &call);
     418                        srv_get_caps(chandle, &call);
    419419                        break;
    420420
    421421                case OUTPUT_FRONTBUF_CREATE:
    422                         srv_frontbuf_create(callid, &call);
     422                        srv_frontbuf_create(chandle, &call);
    423423                        break;
    424424                case OUTPUT_FRONTBUF_DESTROY:
    425                         srv_frontbuf_destroy(callid, &call);
     425                        srv_frontbuf_destroy(chandle, &call);
    426426                        break;
    427427
    428428                case OUTPUT_CURSOR_UPDATE:
    429                         srv_cursor_update(callid, &call);
     429                        srv_cursor_update(chandle, &call);
    430430                        break;
    431431                case OUTPUT_SET_STYLE:
    432                         srv_set_style(callid, &call);
     432                        srv_set_style(chandle, &call);
    433433                        break;
    434434                case OUTPUT_SET_COLOR:
    435                         srv_set_color(callid, &call);
     435                        srv_set_color(chandle, &call);
    436436                        break;
    437437                case OUTPUT_SET_RGB_COLOR:
    438                         srv_set_rgb_color(callid, &call);
     438                        srv_set_rgb_color(chandle, &call);
    439439                        break;
    440440                case OUTPUT_UPDATE:
    441                         srv_update(callid, &call);
     441                        srv_update(chandle, &call);
    442442                        break;
    443443                case OUTPUT_DAMAGE:
    444                         srv_damage(callid, &call);
     444                        srv_damage(chandle, &call);
    445445                        break;
    446446
    447447                default:
    448                         async_answer_0(callid, EINVAL);
     448                        async_answer_0(chandle, EINVAL);
    449449                }
    450450        }
Note: See TracChangeset for help on using the changeset viewer.