Ignore:
Timestamp:
2018-03-22T06:49:35Z (7 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/lib/drv/generic/remote_usbdiag.c

    r3e242d2 ra46e56b  
    144144
    145145void remote_usbdiag_test_in(ddf_fun_t *fun, void *iface,
    146     cap_call_handle_t callid, ipc_call_t *call)
     146    cap_call_handle_t chandle, ipc_call_t *call)
    147147{
    148148        const usbdiag_iface_t *diag_iface = (usbdiag_iface_t *) iface;
    149149
    150150        size_t size;
    151         cap_call_handle_t data_callid;
    152         if (!async_data_write_receive(&data_callid, &size)) {
    153                 async_answer_0(data_callid, EINVAL);
    154                 async_answer_0(callid, EINVAL);
     151        cap_call_handle_t data_chandle;
     152        if (!async_data_write_receive(&data_chandle, &size)) {
     153                async_answer_0(data_chandle, EINVAL);
     154                async_answer_0(chandle, EINVAL);
    155155                return;
    156156        }
    157157
    158158        if (size != sizeof(usbdiag_test_params_t)) {
    159                 async_answer_0(data_callid, EINVAL);
    160                 async_answer_0(callid, EINVAL);
     159                async_answer_0(data_chandle, EINVAL);
     160                async_answer_0(chandle, EINVAL);
    161161                return;
    162162        }
    163163
    164164        usbdiag_test_params_t params;
    165         if (async_data_write_finalize(data_callid, &params, size) != EOK) {
    166                 async_answer_0(callid, EINVAL);
     165        if (async_data_write_finalize(data_chandle, &params, size) != EOK) {
     166                async_answer_0(chandle, EINVAL);
    167167                return;
    168168        }
     
    173173
    174174        if (ret != EOK) {
    175                 async_answer_0(callid, ret);
    176                 return;
    177         }
    178 
    179         if (!async_data_read_receive(&data_callid, &size)) {
    180                 async_answer_0(data_callid, EINVAL);
    181                 async_answer_0(callid, EINVAL);
     175                async_answer_0(chandle, ret);
     176                return;
     177        }
     178
     179        if (!async_data_read_receive(&data_chandle, &size)) {
     180                async_answer_0(data_chandle, EINVAL);
     181                async_answer_0(chandle, EINVAL);
    182182                return;
    183183        }
    184184
    185185        if (size != sizeof(usbdiag_test_results_t)) {
    186                 async_answer_0(data_callid, EINVAL);
    187                 async_answer_0(callid, EINVAL);
    188                 return;
    189         }
    190 
    191         if (async_data_read_finalize(data_callid, &results, size) != EOK) {
    192                 async_answer_0(callid, EINVAL);
    193                 return;
    194         }
    195 
    196         async_answer_0(callid, ret);
     186                async_answer_0(data_chandle, EINVAL);
     187                async_answer_0(chandle, EINVAL);
     188                return;
     189        }
     190
     191        if (async_data_read_finalize(data_chandle, &results, size) != EOK) {
     192                async_answer_0(chandle, EINVAL);
     193                return;
     194        }
     195
     196        async_answer_0(chandle, ret);
    197197}
    198198
    199199void remote_usbdiag_test_out(ddf_fun_t *fun, void *iface,
    200     cap_call_handle_t callid, ipc_call_t *call)
     200    cap_call_handle_t chandle, ipc_call_t *call)
    201201{
    202202        const usbdiag_iface_t *diag_iface = (usbdiag_iface_t *) iface;
    203203
    204204        size_t size;
    205         cap_call_handle_t data_callid;
    206         if (!async_data_write_receive(&data_callid, &size)) {
    207                 async_answer_0(data_callid, EINVAL);
    208                 async_answer_0(callid, EINVAL);
     205        cap_call_handle_t data_chandle;
     206        if (!async_data_write_receive(&data_chandle, &size)) {
     207                async_answer_0(data_chandle, EINVAL);
     208                async_answer_0(chandle, EINVAL);
    209209                return;
    210210        }
    211211
    212212        if (size != sizeof(usbdiag_test_params_t)) {
    213                 async_answer_0(data_callid, EINVAL);
    214                 async_answer_0(callid, EINVAL);
     213                async_answer_0(data_chandle, EINVAL);
     214                async_answer_0(chandle, EINVAL);
    215215                return;
    216216        }
    217217
    218218        usbdiag_test_params_t params;
    219         if (async_data_write_finalize(data_callid, &params, size) != EOK) {
    220                 async_answer_0(callid, EINVAL);
     219        if (async_data_write_finalize(data_chandle, &params, size) != EOK) {
     220                async_answer_0(chandle, EINVAL);
    221221                return;
    222222        }
     
    227227
    228228        if (ret != EOK) {
    229                 async_answer_0(callid, ret);
    230                 return;
    231         }
    232 
    233         if (!async_data_read_receive(&data_callid, &size)) {
    234                 async_answer_0(data_callid, EINVAL);
    235                 async_answer_0(callid, EINVAL);
     229                async_answer_0(chandle, ret);
     230                return;
     231        }
     232
     233        if (!async_data_read_receive(&data_chandle, &size)) {
     234                async_answer_0(data_chandle, EINVAL);
     235                async_answer_0(chandle, EINVAL);
    236236                return;
    237237        }
    238238
    239239        if (size != sizeof(usbdiag_test_results_t)) {
    240                 async_answer_0(data_callid, EINVAL);
    241                 async_answer_0(callid, EINVAL);
    242                 return;
    243         }
    244 
    245         if (async_data_read_finalize(data_callid, &results, size) != EOK) {
    246                 async_answer_0(callid, EINVAL);
    247                 return;
    248         }
    249 
    250         async_answer_0(callid, ret);
     240                async_answer_0(data_chandle, EINVAL);
     241                async_answer_0(chandle, EINVAL);
     242                return;
     243        }
     244
     245        if (async_data_read_finalize(data_chandle, &results, size) != EOK) {
     246                async_answer_0(chandle, EINVAL);
     247                return;
     248        }
     249
     250        async_answer_0(chandle, ret);
    251251}
    252252
Note: See TracChangeset for help on using the changeset viewer.