Changeset 3ca4ae9 in mainline


Ignore:
Timestamp:
2017-12-20T15:51:07Z (6 years ago)
Author:
Jiří Zárevúcky <zarevucky.jiri@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
4547f11
Parents:
68a2fab2
git-author:
Jiří Zárevúcky <zarevucky.jiri@…> (2017-12-20 15:46:56)
git-committer:
Jiří Zárevúcky <zarevucky.jiri@…> (2017-12-20 15:51:07)
Message:

Fix error code mixing in usbhid.

Location:
uspace/lib/usbhid/src
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • uspace/lib/usbhid/src/hiddescriptor.c

    r68a2fab2 r3ca4ae9  
    7575#define USB_HID_RESET_OFFSET    3
    7676
     77#define USB_HID_INVALID                 -98
    7778/** Unknown tag was founded in report descriptor data*/
    7879#define USB_HID_UNKNOWN_TAG             -99
    … …  
    103104               
    104105                if(usb_hid_report_compare_usage_path(path, cmp_path,
    105                                         USB_HID_PATH_COMPARE_STRICT) == EOK){
     106                                        USB_HID_PATH_COMPARE_STRICT) == 0){
    106107                        break;
    107108                }
    … …  
    553554
    554555                if((ret=usb_hid_report_parse_main_tag(tag, data, item_size,
    555                         report_item, usage_path)) == EOK) {
     556                        report_item, usage_path)) == 0) {
    556557
    557558                        return USB_HID_NEW_REPORT_ITEM;
    … …  
    584585 * @param Length of data buffer
    585586 * @param Current state table
    586  * @return Error code
     587 * @return 0 or USB_HID_ code
    587588 */
    588589
    … …  
    599600        case USB_HID_REPORT_TAG_FEATURE:
    600601                report_item->item_flags = *data;                       
    601                 return EOK;                     
     602                return 0;                       
    602603                break;
    603604                       
    … …  
    639640        }
    640641
    641         return EOK;
     642        return 0;
    642643}
    643644
    … …  
    649650 * @param Length of data buffer
    650651 * @param Current state table
    651  * @return Error code
     652 * @return 0 or USB_HID_ code
    652653 */
    653654int usb_hid_report_parse_global_tag(uint8_t tag, const uint8_t *data,
    … …  
    725726        }
    726727
    727         return EOK;
     728        return 0;
    728729}
    729730
    … …  
    735736 * @param Length of data buffer
    736737 * @param Current state table
    737  * @return Error code
     738 * @return 0 or USB_HID_ code
    738739 */
    739740int usb_hid_report_parse_local_tag(uint8_t tag, const uint8_t *data,
    … …  
    791792                            USB_HID_EXTENDED_USAGE_PAGE(
    792793                            usb_hid_report_tag_data_uint32(data, item_size))) {
    793                                 return EINVAL;
     794                                return USB_HID_INVALID;
    794795                        }
    795796                       
    … …  
    862863        }
    863864       
    864         return EOK;
     865        return 0;
    865866}
    866867

  • uspace/lib/usbhid/src/hidparser.c

    r68a2fab2 r3ca4ae9  
    531531
    532532                        if (usb_hid_report_compare_usage_path(
    533                             field->collection_path, path, flags) == EOK) {
     533                            field->collection_path, path, flags) == 0) {
    534534                                usb_hid_report_remove_last_item(
    535535                                    field->collection_path);

  • uspace/lib/usbhid/src/hidpath.c

    r68a2fab2 r3ca4ae9  
    191191 * @param flags       Flags determining the mode of comparison
    192192 *
    193  * @return EOK if both paths are identical, non zero number otherwise
     193 * @return 0 if both paths are identical, non zero number otherwise
    194194 *
    195195 */
    … …  
    213213        // Empty path match all others
    214214        if (path->depth == 0) {
    215                 return EOK;
     215                return 0;
    216216        }
    217217       
    … …  
    239239                                        if (USB_HID_SAME_USAGE(report_item->usage,
    240240                                            path_item->usage))
    241                                                 return EOK;
     241                                                return 0;
    242242                                } else {
    243                                         return EOK;
     243                                        return 0;
    244244                                }
    245245                        }
    … …  
    285285                    ((report_link == &report_path->items.head) &&
    286286                    (path_link == &path->items.head))) {
    287                         return EOK;
     287                        return 0;
    288288                } else {
    289289                        return 1;
    … …  
    297297               
    298298                if (list_empty(&path->items)) {
    299                         return EOK;
     299                        return 0;
    300300                }
    301301               
    … …  
    320320               
    321321                if (path_link == &path->items.head) {
    322                         return EOK;
     322                        return 0;
    323323                } else {
    324324                        return 1;
    … …  
    327327       
    328328        default:
    329                 return EINVAL;
     329                return -1;
    330330        }
    331331}
Note: See TracChangeset for help on using the changeset viewer.