Ticket #1842: dfu-util-verbosity.patch

File dfu-util-verbosity.patch, 3.6 KB (added by wiml, 11 years ago)

make some of dfu-util's messages more informative

  • dfu-util/doc/dfu-util.1

     
    22.SH NAME 
    33dfu-util \- Device firmware update (DFU) USB programmer 
    44.SH SYNOPSIS 
    5 .B dfu-util \fR[\fB\-ldpciatUDRhV\fR] 
     5.B dfu-util \fR[\fB\-ldpciatUDRhvV\fR] 
    66.SH DESCRIPTION 
    77.B dfu-util 
    88is a program that implements the host (PC) side of the USB DFU 
     
    6767.B "\-h, \-\-help" 
    6868Show a help text and exit. 
    6969.TP 
     70.B "\-v, \-\-verbose" 
     71Print more information about dfu-util's operation. A second 
     72.B -v 
     73will turn on verbose logging of USB requests. 
     74.TP 
    7075.B "\-V, \-\-version" 
    7176Show version information and exit. 
    7277.SH EXAMPLES 
  • dfu-util/src/dfu.c

     
    4444        dfu_timeout = timeout; 
    4545    } else { 
    4646        if( 0 != dfu_debug_level ) 
    47             fprintf( stderr, "dfu_init: Invalid timeout value.\n" ); 
     47            fprintf( stderr, "dfu_init: Invalid timeout value %d.\n", timeout ); 
    4848    } 
    4949} 
    5050 
     
    142142          /* wLength       */ length, 
    143143                              dfu_timeout ); 
    144144    if( status < 0 ) { 
    145         fprintf( stderr, "%s error %d\n", __FUNCTION__, status ); 
     145        fprintf( stderr, "%s: usb_control_msg returned %d: %s\n", 
     146                 __FUNCTION__, 
     147                 status, usb_strerror() ); 
    146148    } 
    147149 
    148150    return status; 
     
    188190          /* wLength       */ length, 
    189191                              dfu_timeout ); 
    190192    if( status < 0 ) { 
    191         fprintf( stderr, "%s error %d\n", __FUNCTION__, status ); 
     193        fprintf( stderr, "%s: usb_control_msg returned %d: %s\n", 
     194                 __FUNCTION__, 
     195                 status, usb_strerror() ); 
    192196    } 
    193197 
    194198    return status; 
  • dfu-util/src/main.c

     
    460460                        exit(0); 
    461461                        break; 
    462462                case 'v': 
     463                        if (verbose) { 
     464                                usb_set_debug(255); 
     465                        } 
    463466                        verbose = 1; 
    464467                        break; 
    465468                case 'l': 
     
    469472                case 'd': 
    470473                        /* Parse device */ 
    471474                        if (parse_vendprod(&vendprod, optarg) < 0) { 
    472                                 fprintf(stderr, "unable to parse `%s'\n", optarg); 
     475                                fprintf(stderr, "unable to parse `%s' as a vendor:product\n", optarg); 
     476 
    473477                                exit(2); 
    474478                        } 
    475479                        dif->vendor = vendprod.vendor; 
     
    535539        } 
    536540 
    537541        if (!filename) { 
    538                 fprintf(stderr, "You need to specify a filename to -D -r -U\n"); 
     542                fprintf(stderr, "You need to specify a filename to -D or -U\n"); 
    539543                help(); 
    540544                exit(2); 
    541545        } 
     
    559563        if (!get_first_dfu_device(dif)) 
    560564                exit(3); 
    561565 
    562         /* We have exactly one device. It's usb_device is now in dif->dev */ 
     566        /* We have exactly one device. Its usb_device is now in dif->dev */ 
    563567 
    564568        printf("Opening USB Device 0x%04x:0x%04x...\n", dif->vendor, dif->product); 
    565569        dif->dev_handle = usb_open(dif->dev); 
     
    581585 
    582586                printf("Claiming USB DFU Runtime Interface...\n"); 
    583587                if (usb_claim_interface(_rt_dif.dev_handle, _rt_dif.interface) < 0) { 
    584                         fprintf(stderr, "Cannot claim interface: %s\n", usb_strerror()); 
     588                        fprintf(stderr, "Cannot claim interface %d: %s\n",  
     589                                _rt_dif.interface, 
     590                                usb_strerror()); 
    585591                        exit(1); 
    586592                } 
    587593 
     
    713719                exit(1); 
    714720        } 
    715721 
    716         printf("Setting Alternate Setting ...\n"); 
     722        printf("Setting Alternate Setting #%d ...\n", dif->altsetting); 
    717723        if (usb_set_altinterface(dif->dev_handle, dif->altsetting) < 0) { 
    718724                fprintf(stderr, "Cannot set alternate interface: %s\n", 
    719725                        usb_strerror());