<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body><div>Hey Michael,</div><div><br></div><div>thanks for the great little "howto".</div><div><br></div><div>Worked perfectly fine for me on Linux.</div><div><br></div><div>My new ovms3.bin comiled, installed und runs:</div><div><br></div><div><span style="font-family: monospace;">3.2.013-336-ga4349710/ota_1/main (build idf v3.3.2-879-g0137aef47 Jul 16 2020 12:22:49)</span><br><br></div><div>Greetinx</div><div><br></div><div>Chris</div><div><br></div><div><br></div><div>Am Donnerstag, den 16.07.2020, 11:27 +0200 schrieb Michael Balzer:</div><blockquote type="cite">
Everyone,<br>
<br>
I've merged the latest release/v3.3 esp-idf into our idf fork and
merged the PSRAM fix test branch into our OVMS master.<br>
<br>
To build now, you need to:<br>
<br>
1. Update your toolchain to release 1.22.0-96-g2852398-5.2.0:
download the toolchain for your OS from
<a class="moz-txt-link-freetext" href="https://docs.espressif.com/projects/esp-idf/en/release-v3.3/get-started/index.html#setup-toolchain">https://docs.espressif.com/projects/esp-idf/en/release-v3.3/get-started/index.html#setup-toolchain</a>,
simply replace your existing toolchain installation by unpacking the
archive. Test your toolchain installation by checking
xtensa-esp32-elf-gcc --version.<br>
<br>
2. Pull our latest esp-idf master and -important- update the
submodules<br>
<br>
3. Pull/checkout our latest OVMS master<br>
<br>
4. Update your sdkconfig from support/sdkconfig.default.hw31<br>
<br>
The build should then result in version
3.2.013-298-g025f838b/factory/edge (build idf v3.3.2-879-g0137aef47
Jul 16 2020 10:53:38) or higher.<br>
<br>
Regards,<br>
Michael<br>
<br>
<br>
<br>
<div class="moz-cite-prefix">Am 15.07.20 um 08:07 schrieb Michael
Balzer:<br>
</div>
<blockquote type="cite" cite="mid:d6e85100-d806-297a-3aa5-3e29dc1654b5@expeedo.de">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
Yes, differences on the test branches are minor, also in esp-idf.
<br>
<br>
But we probably also need to pull in the latest changes on the
esp-idf 3.3 branch, which may be some more work.<br>
<br>
I'll fetch the current toolchain version and check what is needed.<br>
<br>
Regards,<br>
Michael<br>
<br>
<br>
<div class="moz-cite-prefix">Am 15.07.20 um 04:13 schrieb Mark
Webb-Johnson:<br>
</div>
<blockquote type="cite" cite="mid:446C1263-521A-4116-9CD0-F38ED02CFE12@webb-johnson.net">
<meta http-equiv="Content-Type" content="text/html;
charset=UTF-8">
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="" moz-do-not-send="true">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="" moz-do-not-send="true">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="" moz-do-not-send="true">dexter@expeedo.de</a>>
wrote:</div>
<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="" moz-do-not-send="true">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="" moz-do-not-send="true">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="">
</div>
_______________________________________________<br class="">
OvmsDev mailing list<br class="">
<a href="mailto:OvmsDev@lists.openvehicles.com" class="" moz-do-not-send="true">OvmsDev@lists.openvehicles.com</a><br class="">
<a class="moz-txt-link-freetext" href="http://lists.openvehicles.com/mailman/listinfo/ovmsdev" moz-do-not-send="true">http://lists.openvehicles.com/mailman/listinfo/ovmsdev</a><br class="">
</div>
</blockquote>
</div>
<br class="">
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
OvmsDev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:OvmsDev@lists.openvehicles.com" moz-do-not-send="true">OvmsDev@lists.openvehicles.com</a>
<a class="moz-txt-link-freetext" href="http://lists.openvehicles.com/mailman/listinfo/ovmsdev" moz-do-not-send="true">http://lists.openvehicles.com/mailman/listinfo/ovmsdev</a>
</pre>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal
Fon 02333 / 833 5735 * Handy 0176 / 206 989 26
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
OvmsDev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:OvmsDev@lists.openvehicles.com">OvmsDev@lists.openvehicles.com</a>
<a class="moz-txt-link-freetext" href="http://lists.openvehicles.com/mailman/listinfo/ovmsdev">http://lists.openvehicles.com/mailman/listinfo/ovmsdev</a>
</pre>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
Michael Balzer * Helkenberger Weg 9 * D-58256 Ennepetal
Fon 02333 / 833 5735 * Handy 0176 / 206 989 26
</pre>
<pre>_______________________________________________
OvmsDev mailing list
<a href="mailto:OvmsDev@lists.openvehicles.com">OvmsDev@lists.openvehicles.com</a>
<a href="http://lists.openvehicles.com/mailman/listinfo/ovmsdev">http://lists.openvehicles.com/mailman/listinfo/ovmsdev</a>
</pre></blockquote></body></html>