http://forums.leaflabs.com/topic.php?id=645 A place to share, learn, and grow... en-US Fri, 22 Jan 2016 00:20:01 +0000 http://bbpress.org/?v=1.0.2 q http://forums.leaflabs.com/search.php http://forums.leaflabs.com/topic.php?id=645#post-3595 Thu, 10 Feb 2011 11:24:09 +0000 mbolivar 3595@http://forums.leaflabs.com/ <p>Hi Stephen,</p> <p>If you find a bug, you can choose any of the following methods to report it:</p> <p>1) Make a forum post.</p> <p>2) If you have a Google account, post an issue in our bugtracker:</p> <p><a href="http://code.google.com/p/leaflabs/issues/list" rel="nofollow">http://code.google.com/p/leaflabs/issues/list</a></p> <p>3) If you have a Github account, comment on the line of source you believe to be in error, and we'll see it on our Github dashboard.</p> <p>4) Email <a href="mailto:bugs@leaflabs.com">bugs@leaflabs.com</a> (which I've just created).</p> <p>Regarding your question about replacing usb.c, you should be able to make this change in the 0.0.9 IDE release without breaking libmaple 0.0.9. Your use of the word "dependency" is correct here; there haven't been any changes in usb.c's dependencies which would cause geoffreymbrown's patch to break if applied to 0.0.9. However, it is unlikely to solve your problems with SerialUSB.</p> <p>There are a variety of fixes we've made to the SerialUSB code which should improve its stability significantly. These occurred in the debug-serialusb libmaple branch, which will be merged into the master branch by the end of the week. </p> <p>At that time, we'll put together "snapshot" releases of the IDE (not a full release, more like a preview) containing the new libmaple code, so IDE users can try it out, instead of just the users of the Unix toolchain. We've recently spent some time automating the production of IDE snapshots, and we plan on releasing one any time we add any important new features or bugfixes, in order to let the community try them out (and hopefully provide feedback).</p> <p>Expect a forum post when it's done, and we look forward to your feedback! </p> http://forums.leaflabs.com/topic.php?id=645#post-3592 Thu, 10 Feb 2011 09:59:13 +0000 StephenFromNYC 3592@http://forums.leaflabs.com/ <p>Hello,</p> <p>Geoffrey, thanks for catching the usb.c bug.</p> <p>Is there a place where users can find or upload code which reveals Maple software bugs? This might be one way to test the Maple IDE before each release.</p> <p>If someone wrote some simple code using Maple 0.0.9 (using the unmodified usb.c code) which would show the race (which Geoffrey said he did not observe) this could be added to the "test code" library.</p> <p>I am interested in seeing if the change in usb.c makes my sketches which use SerialUSB() more stable.</p> <p>Besides replacing (of course after backing up the original files) usb.c which other files should I replace?</p> <p>In other words, since the last release of the maple IDE, were there any changes in other files which are required for the most current usb.c code to work. And of course, are there any recent changes in the most current usb.c code which required modifications in other files.</p> <p>Is this is called "dependency"?</p> <p>Or I could wait until the next IDE release if it is coming out soon. Any date?</p> <p>Thanks for the hard work!</p> <p>Stephen from NYC </p> http://forums.leaflabs.com/topic.php?id=645#post-3580 Wed, 09 Feb 2011 18:00:53 +0000 mbolivar 3580@http://forums.leaflabs.com/ <p>Fixed, thanks!</p> <p><a href="https://github.com/leaflabs/libmaple/commit/d0074d2922de87d08745c99665ae6590933d1e94" rel="nofollow">https://github.com/leaflabs/libmaple/commit/d0074d2922de87d08745c99665ae6590933d1e94</a> </p> http://forums.leaflabs.com/topic.php?id=645#post-3577 Wed, 09 Feb 2011 15:03:36 +0000 geoffreymbrown 3577@http://forums.leaflabs.com/ <p>The following code is from libmaple/usb/usb.c line 376</p> <p> if (loaded) {<br /> UserToPMABufferCopy(sendBuf,VCOM_TX_ADDR + countTx, loaded);<br /> _SetEPTxCount(VCOM_TX_ENDP, countTx+loaded);<br /> _SetEPTxValid(VCOM_TX_ENDP);<br /> countTx += loaded;<br /> }</p> <p>Seems like there's a race condition on countTx and the code<br /> should be:</p> <p>if (loaded) {<br /> UserToPMABufferCopy(sendBuf,VCOM_TX_ADDR + countTx, loaded);<br /> _SetEPTxCount(VCOM_TX_ENDP, countTx+loaded);<br /> countTx += loaded;<br /> _SetEPTxValid(VCOM_TX_ENDP);</p> <p> }</p> <p>I haven't seen a race, just seems like there might be one. </p>