<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Hey Adam,</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Looking at the patch, it says the real fix is the sleep for 1 second, correct? I did verify that setting DTS = True is redundant as you said, it's included in pyserial as a default state. I don't see anything wrong with the patch in general, but do have one
question for the more senior devs here:</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
A question for the larger group here, I wonder if we could/should just push that line of code, time.sleep(1), up to the higher level serial class so that all drivers could take advantage of it? Specifically, do_upload() and do_download() in ./chirp/ui/mainapp.py</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Tony</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> chirp_devel-bounces@intrepid.danplanet.com <chirp_devel-bounces@intrepid.danplanet.com> on behalf of interfect--- via chirp_devel <chirp_devel@intrepid.danplanet.com><br>
<b>Sent:</b> Saturday, September 5, 2020 11:45 PM<br>
<b>To:</b> chirp_devel@intrepid.danplanet.com <chirp_devel@intrepid.danplanet.com><br>
<b>Subject:</b> [chirp_devel] Patch for UV5R driver to fix communication on Ubuntu 16.04 with pl2303-based cables (fixes #4165)</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Hello,<br>
<br>
Bernhard Hailer has asked me to resubmit my patch for the issue with modern Ubuntu serial drivers<br>
having DTR low, at least when the port isn't open, causing the radio to be in transmit mode and not<br>
ready to accept CHIRP's commands as soon as the port is open. This fixes #4165.<br>
<br>
Issue: <a href="https://chirp.danplanet.com/issues/4165">https://chirp.danplanet.com/issues/4165</a><br>
Original patch submission (did anybody get it?):<br>
<a href="http://intrepid.danplanet.com/pipermail/chirp_devel/2016-October/004329.html">http://intrepid.danplanet.com/pipermail/chirp_devel/2016-October/004329.html</a><br>
<br>
I tried to send this message earlier after subscribing to the list and clicking the confirmation link in the subscription e-mail, but I think web based confirmation is broken. When I sent the patch I got a message saying it was held for moderation because I
wasn't a member of the list. I had to subscribe again and confirm by e-mail instead to actually get onto the list.<br>
<br>
Thanks,<br>
-Adam<br>
</div>
</span></font></div>
</body>
</html>