<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Finally… Crazy that this took so long.<div class=""><br class=""></div><div class="">We have a GetOVMSHardware() in ovms_version.cpp that picks up the esp_chip_info_t structure and has chip.revision (documented as "chip revision number”). That is stored in metric m.hardware. My bench unit is showing rev=ESP32/1. I checked a bunch of recent cars connected to <a href="http://api.openvehicles.com" class="">api.openvehicles.com</a>, but all showed the same. So not sure if the v3 silicon is not reported in that chip.revision, if we are just using existing stocks, or an IDF update is required to pickup the new silicon version number. The v3 seems fairly recent (March 2020?) and I don’t see a v2 listed. Anyway, I’ve just yesterday received a new batch of modules so will check those to see if it shows up.</div><div class=""><br class=""></div><div class="">What do we need to do to switch to this? Presumably I turn off auto-build of the firmware on <a href="http://api.openvehicles.com" class="">api.openvehicles.com</a>, you merge your SPIRAM branch to master, we tell everyone to download the new toolchain, then we switch both our edge builds back to master branch with new toolchain?</div><div class=""><br class=""></div><div class="">Apart from the CONFIG_SPIRAM_SUPPORT 2MB workaround, I can’t see any significant differences between remotes/origin/spiram-fix-test and master? Is the only difference the ovms_websockethandler.cpp change to std::string (vs extram::string)?</div><div class=""><br class=""></div><div class="">Regards, Mark.<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 15 Jul 2020, at 2:11 AM, Michael Balzer <<a href="mailto:dexter@expeedo.de" class="">dexter@expeedo.de</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
<meta http-equiv="content-type" content="text/html; charset=UTF-8" class="">
<div class="">
Mark,<br class="">
<br class="">
it seems the fix has finally arrived in the official release 3.3
branch:<br class="">
<br class="">
<a href="https://github.com/espressif/esp-idf/issues/2892#issuecomment-658327913" class="">https://github.com/espressif/esp-idf/issues/2892#issuecomment-658327913</a><br class="">
<br class="">
<a href="https://github.com/espressif/esp-idf/commit/f4333c8e3a554e8bb4210d825cbdf4bcaa1fc1b8" class="">https://github.com/espressif/esp-idf/commit/f4333c8e3a554e8bb4210d825cbdf4bcaa1fc1b8</a><br class="">
<br class="">
Btw:<br class="">
<br class="">
<blockquote type="cite" class="">
<div class=" js-timeline-progressive-focus-container js-timeline-item
" data-gid="MDEyOklzc3VlQ29tbWVudDY0NTQyMDA5OQ==" style="box-sizing: border-box; color: rgb(36, 41, 46);
font-family: -apple-system, BlinkMacSystemFont, "Segoe
UI", Helvetica, Arial, sans-serif, "Apple Color
Emoji", "Segoe UI Emoji"; font-size: 14px;
font-style: normal; font-variant-ligatures: normal;
font-variant-caps: normal; font-weight: 400; letter-spacing:
normal; orphans: 2; text-align: start; text-indent: 0px;
text-transform: none; white-space: normal; widows: 2;
word-spacing: 0px; -webkit-text-stroke-width: 0px;
background-color: rgb(255, 255, 255); text-decoration-style:
initial; text-decoration-color: initial;">
<div class="js-comment-container TimelineItem" data-gid="MDEyOklzc3VlQ29tbWVudDY0NTQyMDA5OQ==" data-url="/_render_node/MDEyOklzc3VlQ29tbWVudDY0NTQyMDA5OQ==/timeline/issue_comment" style="box-sizing: border-box; position: relative; display:
flex; padding: 16px 0px; margin-left: 16px;">
<div class=" TimelineItem-body js-minimizable-comment-group my-0 timeline-comment-group
js-targetable-element
" id="issuecomment-645420099" style="box-sizing: border-box; margin-top: 4px;
margin-bottom: 0px !important; min-width: 0px; max-width:
100%; color: rgb(68, 77, 86); flex: 1 1 auto;">
<div class=" timeline-comment previewable-edit js-task-list-container js-comment
unminimized-comment
ml-n3 reorderable-task-lists comment timeline-comment--caret" data-body-version="d9e7b21e6119fab731f2490347034550" data-unfurl-hide-url="/content_reference_attachments/hide" style="box-sizing: border-box; margin-left: -16px
!important; position: relative; color: rgb(36, 41, 46);
background-color: rgb(255, 255, 255); border: 1px solid
rgb(225, 228, 232); border-radius: 6px;">
<div class="edit-comment-hide" style="box-sizing:
border-box;">
<table class="d-block" data-paste-markdown-skip="" style="box-sizing: border-box; border-spacing: 0px;
border-collapse: collapse; display: block !important;">
<tbody class="d-block" style="box-sizing: border-box;
display: block !important;">
<tr class="d-block" style="box-sizing: border-box;
display: block !important;">
<td class=" js-comment-body comment-body markdown-body
d-block" style="box-sizing: border-box;
padding: 15px; display: block !important;
font-family: -apple-system, BlinkMacSystemFont,
"Segoe UI", Helvetica, Arial,
sans-serif, "Apple Color Emoji",
"Segoe UI Emoji"; font-size: 14px;
line-height: 1.5; overflow-wrap: break-word;
width: 846px; overflow: visible;"><div style="box-sizing: border-box; margin-top: 0px; margin-bottom: 0px !important;" class="">In the v3
silicon, the issue has been fixed, and the
workaround is no longer required.</div>
</td>
</tr>
</tbody>
</table>
</div>
</div>
</div>
</div>
</div>
</blockquote>
<br class="">
I guess that means from some manufacturing point on we should
provide two builds, one for the older hardware and one for the
modules with v3 chips, as the fix definitely has a performance
impact.<br class="">
<br class="">
Regards,<br class="">
Michael<br class="">
<br class="">
<pre class="moz-signature" cols="72">--
Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal
Fon 02333 / 833 5735 * Handy 0176 / 206 989 26
</pre>
</div>
_______________________________________________<br class="">OvmsDev mailing list<br class=""><a href="mailto:OvmsDev@lists.openvehicles.com" class="">OvmsDev@lists.openvehicles.com</a><br class="">http://lists.openvehicles.com/mailman/listinfo/ovmsdev<br class=""></div></blockquote></div><br class=""></div></body></html>