Send buglog mailing list submissions to
        [email protected]

To subscribe or unsubscribe via the World Wide Web, visit
        http://lists.openmoko.org/mailman/listinfo/buglog
or, via email, send a message with subject or body 'help' to
        [EMAIL PROTECTED]

You can reach the person managing the list at
        [EMAIL PROTECTED]

When replying, please edit your Subject line so it is more specific
than "Re: Contents of buglog digest..."
Today's Topics:

   1. [Bug 653] [PATCH] dfu-util - Some devices boot up in a DFU
      state     instead of app state ([EMAIL PROTECTED])
   2. [Bug 654] New: [PATCH] dfu-util - transfer-size setting code
      rearranged ([EMAIL PROTECTED])
   3. [Bug 654] [PATCH] dfu-util - transfer-size setting code
      rearranged ([EMAIL PROTECTED])
   4. [Bug 655] New: [PATCH] dfu-util - to quick, add sleep
      ([EMAIL PROTECTED])
--- Begin Message ---
http://bugzilla.openmoko.org/cgi-bin/bugzilla/show_bug.cgi?id=653





------- Additional Comments From [EMAIL PROTECTED]  2007-07-24 11:08 -------
Eh, the complete patch being:

Index: src/main.c
===================================================================
--- src/main.c  (revision 2384)
+++ src/main.c  (working copy)
@@ -507,9 +507,7 @@
                        }
                        break;
                default: 
-                       fprintf(stderr, "IMPOSSIBLE: Runtime device already in 
DFU state ?!?\n"); 
-                       exit(1); 
-                       break; 
+                       goto dfustate; 
                } 
  
                /* now we need to re-scan the bus and locate our device */ 
@@ -540,6 +538,7 @@ 
                 * procedure */ 
        } 
  
+dfustate: 
        if (alt_name) { 
                int n; 
 




------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.



--- End Message ---
--- Begin Message ---
http://bugzilla.openmoko.org/cgi-bin/bugzilla/show_bug.cgi?id=654

           Summary: [PATCH] dfu-util - transfer-size setting code rearranged
           Product: OpenMoko
           Version: current svn head
          Platform: Other
        OS/Version: FreeBSD
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: OE bitbake recipes  / build system
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]
                CC: [email protected]


The piece of code that determines the transfer-size does a control request and 
then checks whether it actually needs that information. The TAS1020B chip 
doesn't have the descriptor and therefore rearranging the code and specifying 
the transfer-size on the command line makes this work for me. It should not 
break anything on other devices (unless they require you to request the 
descriptor first, before you do anything, which sounds highly unlikely to me).



------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.



--- End Message ---
--- Begin Message ---
http://bugzilla.openmoko.org/cgi-bin/bugzilla/show_bug.cgi?id=654





------- Additional Comments From [EMAIL PROTECTED]  2007-07-24 11:13 -------
Created an attachment (id=219)
 --> 
(http://bugzilla.openmoko.org/cgi-bin/bugzilla/attachment.cgi?id=219&action=view)
Patch rearranging the transfer-size code




------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.



--- End Message ---
--- Begin Message ---
http://bugzilla.openmoko.org/cgi-bin/bugzilla/show_bug.cgi?id=655

           Summary: [PATCH] dfu-util - to quick, add sleep
           Product: OpenMoko
           Version: current svn head
          Platform: Other
        OS/Version: FreeBSD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: OE bitbake recipes  / build system
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]
                CC: [email protected]


For the TAS1020b chip the control requests for completion are slightly to 
quick, so adding a sleep stops the errors from appearing.

Index: src/sam7dfu.c
===================================================================
--- src/sam7dfu.c       (revision 2384)
+++ src/sam7dfu.c       (working copy)
@@ -160,6 +160,7 @@
        switch (dst.bState) {
        case DFU_STATE_dfuMANIFEST_SYNC:
        case DFU_STATE_dfuMANIFEST:
+               sleep(1);
                goto get_status;
                break;
        case DFU_STATE_dfuIDLE:

(The break after the goto is unnecessary).

Off-topic remark: usb_rest support in libusb will be available in FreeBSD 
soon, as soon as I've added the patch to the FreeBSD sources and submitted a 
patch to the libusb maintainers.



------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.



--- End Message ---
_______________________________________________
buglog mailing list
[email protected]
http://lists.openmoko.org/mailman/listinfo/buglog

Reply via email to