<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
Greg,<br>
<br>
regarding your changes to reduce to TX buffer 1, I think they will
do so. I don't think it's necessary to check buffers 2 & 3 busy
flags at all, but it shouldn't hurt either.<br>
<br>
On the TX limits this introduces: each TX needs…<br>
<ul>
<li>4 bytes for the interrupt check</li>
<li>4 bytes for the TX clear<br>
</li>
<li>2 bytes for the status read</li>
<li>14 bytes for the buffer</li>
<li>1 byte for the RTS</li>
</ul>
<p>…total 25 bytes = 200 bits. So at 1 MHz the throughput will be
limited to 5,000 frames per second, and the latency will be at
least 200 us.</p>
<p>That should still be sufficient for most situations. If you need
more speed → can1. Plus maybe the 1 MHz is a typo.<br>
</p>
So, while my basic issues still apply, we can use that solution for
now.<br>
<br>
Regards,<br>
Michael<br>
<br>
<br>
<div class="moz-cite-prefix">Am 16.01.2018 um 18:59 schrieb Michael
Balzer:<br>
</div>
<blockquote type="cite"
cite="mid:8749fa15-5e5c-2a3f-57de-52bffb2680db@expeedo.de">
<pre wrap="">Greg, Mark,
I'm having two basic issues with this:
a) This will limit the TX speed achievable on the mcp interfaces. I generally hate not fully utilizing available hardware capabilities. And this is SPI, filling
a TX buffer is expensive.
b) It again feels like fixing without understanding. I really think we should first understand how the reported race condition of the mcp interrupt signal
affects our driver design, and we should check the effect of level interrupts before resorting to limiting the speed.
The mcp SPI interface is running at 1 MHz, so a TX will … wait a moment, why is it running at 1 MHz? According to the data sheet, it can do 10 MHz?
Mark, is there some scaling done in the spinodma module, or is there a reason (other than typo / copy-paste) for this?
Am 16.01.2018 um 09:47 schrieb Greg D.:
</pre>
<blockquote type="cite">
<pre wrap="">Hi Mark,
Yes, absolutely. In fact, I just thought of a corner case while nodding
off to sleep (hate it when that happens!). Fixed, verified, and pushed
to my branch. Anxiously awaiting review from Michael...
Greg
Mark Webb-Johnson wrote:
</pre>
<blockquote type="cite">
<pre wrap="">I think best for @michael to review. A single tx buffer is certainly easier to manage.
Regards, Mark.
</pre>
<blockquote type="cite">
<pre wrap="">On 16 Jan 2018, at 1:59 PM, Greg D. <a class="moz-txt-link-rfc2396E" href="mailto:gregd2350@gmail.com"><gregd2350@gmail.com></a> wrote:
Mark Webb-Johnson wrote:
</pre>
<blockquote type="cite">
<pre wrap="">In vino veritas.
</pre>
</blockquote>
<pre wrap="">Indeed!
Fixes look good (can't seem to kill it). Pushed to my fork on Git Hub,
along with removal of the delays in the obd2ecu.cpp code.
This was done on the older tool chain; wanted to be sure the fix was
real before upgrading and potentially hiding the issue with a slight
change in timing. Will upgrade the toolchain tomorrow and verify.
Greg
_______________________________________________
OvmsDev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:OvmsDev@lists.teslaclub.hk">OvmsDev@lists.teslaclub.hk</a>
<a class="moz-txt-link-freetext" href="http://lists.teslaclub.hk/mailman/listinfo/ovmsdev">http://lists.teslaclub.hk/mailman/listinfo/ovmsdev</a>
</pre>
</blockquote>
<pre wrap="">_______________________________________________
OvmsDev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:OvmsDev@lists.teslaclub.hk">OvmsDev@lists.teslaclub.hk</a>
<a class="moz-txt-link-freetext" href="http://lists.teslaclub.hk/mailman/listinfo/ovmsdev">http://lists.teslaclub.hk/mailman/listinfo/ovmsdev</a>
</pre>
</blockquote>
<pre wrap="">_______________________________________________
OvmsDev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:OvmsDev@lists.teslaclub.hk">OvmsDev@lists.teslaclub.hk</a>
<a class="moz-txt-link-freetext" href="http://lists.teslaclub.hk/mailman/listinfo/ovmsdev">http://lists.teslaclub.hk/mailman/listinfo/ovmsdev</a>
</pre>
</blockquote>
<pre wrap="">
</pre>
</blockquote>
<br>
<pre class="moz-signature" cols="160">--
Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal
Fon 02333 / 833 5735 * Handy 0176 / 206 989 26
</pre>
</body>
</html>