<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Michael,<div class=""><br class=""></div><div class="">Thanks for your fast help. I was worried about that.</div><div class=""><br class=""></div><div class="">Iíve re-worked the NET_INIT1 logic. Can you try to pull and see if that now works for you?</div><div class=""><br class=""></div><div class="">With the SIM808, we canít include anything else on the AT+CGPSPWR=1 command (or it gives ERROR). But, putting +CGPSRST=0 in front of the first CMINS? seems to work ok.</div><div class=""><br class=""></div><div class="">I tried here on my SIM908 and it seems ok. Could you try on yours?</div><div class=""><br class=""></div><div class="">Regards, Mark.</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 13 Mar 2016, at 7:43 PM, Michael Balzer <<a href="mailto:dexter@expeedo.de" class="">dexter@expeedo.de</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
  
    <meta content="text/html; charset=windows-1252" http-equiv="Content-Type" class="">
  
  <div bgcolor="#FFFFFF" text="#000000" class="">
    Mark,<br class="">
    <br class="">
    unfortunately the new GPS init does not work, the module doesn't get
    coordinates.<br class="">
    <br class="">
    I remember adding +CGPSRST on purpose, somewhere the docs suggested
    it's necessary...<br class="">
    <br class="">
    ...found it: section 12.2.2 AT+CGPSRST GPS Reset Mode
    (HOT/WARM/COLD)<br class="">
    "Note: COLD start mode is recommended For first time reset."<br class="">
    <br class="">
    I now added it to NET_WAKEUP_GPSON, and the GPS init works again:<br class="">
    <br class="">
    rom char NET_WAKEUP_GPSON[] = "AT+CGPSPWR=1;+CGPSRST=0\r";<br class="">
    <br class="">
    ...but according to your findings on SIM808, this will occasionally
    fail, so that's no solution.<br class="">
    <br class="">
    If we need to retry NET_STATE_START if the PWR command fails, maybe
    we can prepend RST to NET_INIT1?<br class="">
    <br class="">
    Or, if the SIM808 doesn't need the RST, maybe a simple solution is
    to create SIM808 / SIM908 specific images?<br class="">
    <br class="">
    Regards,<br class="">
    Michael<br class="">
    <br class="">
    <br class="">
    <br class="">
    <div class="moz-cite-prefix">Am 13.03.2016 um 10:13 schrieb Mark
      Webb-Johnson:<br class="">
    </div>
    <blockquote cite="mid:88404E2B-6077-4771-AF35-5C21546706DA@openvehicles.com" type="cite" class="">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252" class="">
      <div class=""><br class="">
      </div>
      Iíve just committed the code changes discussed below. This is
      v2.8.3.
      <div class=""><br class="">
      </div>
      <div class="">I few changes to be aware of:</div>
      <div class=""><br class="">
      </div>
      <div class="">
        <ul class="MailOutline">
          <li class="">MPLABX 3.1x support</li>
          <li class="">An OVMS_BUILDCONFIG #define has been added to the
            build configs. This passes in a string that is used during
            the VERSION commands to include the build config selected
            during the build. So we can tell a V2_Production build from
            a V2_Twizy build, etc. Should make support easier, but you
            may need to tweak your own build configs to get them to
            build with this.</li>
          <li class="">The GPS initialisation code has been re-worked.
            Instead of the AT command used for wakeup, in the
            configurations with a GPS, I replace that with the
            AT_CGPSPWR command appropriate (to power up the internal
            SIMx08 GPS if internal GPS is requested by the vehicle
            module). As a side-effect, I dropped the AT+CGPSRST=0 that
            was used during initialisation to do a cold boot - I donít
            think that is going to be an issue, but please let me know
            if you think it will.</li>
        </ul>
      </div>
      <div class=""><br class="">
      </div>
      <div class="">The build config strings I am using are three
        character:</div>
      <div class=""><br class="">
      </div>
      <div class="">
        <ul class="MailOutline">
          <li class="">V1P - V1 Production</li>
          <li class="">V2P - V2 Production</li>
          <li class="">V2E - V2 Experimental</li>
          <li class="">TRP - V2 Tesla Roadster Production</li>
          <li class="">RTP - V2 Renault Twizy Production</li>
        </ul>
      </div>
      <div class=""><br class="">
      </div>
      <div class="">I would appreciate it if you guys could test this
        out on your SIM908 modules. In particular, for cars with GPS.
        This is a release candidate firmware and I am intending to send
        it to the factory building the first batch of SIM808 modules,
        later next week. Any feedback you can provide would be
        appreciated.</div>
      <div class=""><br class="">
      </div>
      <div class="">Regards, Mark.</div>
      <div class=""><br class="">
      </div>
      <div class="">P.S. If this is ok, it should bring to a close the
        v2.x firmware that I am working on (other than simple
        maintenance). Iím now full time concentrating on the v3 firmware
        that is being built using FreeRTOS on the FRDM-K64 platform (as
        a development platform for the upcoming v3 OVMS hardware). More
        on this soon...</div>
      <div class=""><br class="">
        <div class="">
          <blockquote type="cite" class="">
            <div class="">On 8 Mar 2016, at 4:24 PM, Mark Webb-Johnson
              <<a moz-do-not-send="true" href="mailto:mark@openvehicles.com" class="">mark@openvehicles.com</a>>
              wrote:</div>
            <br class="Apple-interchange-newline">
            <div class="">
              <meta http-equiv="Content-Type" content="text/html;
                charset=windows-1252" class="">
              <div style="word-wrap: break-word; -webkit-nbsp-mode:
                space; -webkit-line-break: after-white-space;" class="">
                <div class="">An update on SIM808. It is just a stop-gap
                  solution, but it buys us some time.</div>
                <div class=""><br class="">
                </div>
                <div class="">The changes Iíve made / problems Iíve had:</div>
                <div class=""><br class="">
                </div>
                <div class="">
                  <ol class="MailOutline">
                    <li class="">Power button needs to be held for 1
                      second to power down/up the modem (SIM908 was much
                      faster). Not a bad issue, but it did waste a week
                      going back and forth trying to find the cause.</li>
                    <li class="">The SIM808 doesnít echo LineFeed
                      characters. That is a pain for DIAG mode console,
                      but doesnít affect anything else. Only workaround
                      possible is to use a terminal program that can map
                      CR->CRLF, or CF->LF (and live with the
                      horrible display in DIAG mode). Weíll just develop
                      on SIM908 modules or use a fancy terminal
                      emulator. I guess if people are using scripts to
                      talk to DIAG mode, they may need some tuning if
                      they are expecting CRLF and just getting CR now.</li>
                    <li class="">GPS initialisation (this is the biggy -
                      more below).</li>
                    <li class="">Build profile in version string. This
                      is just a simple fix to include a #define in the
                      build profiles, and to include that where we show
                      the version string. It means that as well as the
                      version of firmware, we can now see the build
                      profile. Helpful for support of new users.</li>
                  </ol>
                </div>
                <div class=""><br class="">
                </div>
                <div class="">The biggest issue has been with GPS
                  initialisation.</div>
                <div class=""><br class="">
                </div>
                <div class="">Looking at the current code:</div>
                <div class=""><br class="">
                </div>
                <blockquote style="margin: 0 0 0 40px; border: none;
                  padding: 0px;" class="">
                  <div class="">
                    <div class=""><font class="" face="Andale Mono">#ifdef
                        OVMS_INTERNALGPS</font></div>
                    <div class=""><font class="" face="Andale Mono">//
                        Using internal SIM908 GPS:</font></div>
                    <div class=""><font class="" face="Andale Mono">rom
                        char NET_INIT1[] =
                        "AT+CGPSPWR=1;+CGPSRST=0;+CSMINS?\r";</font></div>
                    <div class=""><font class="" face="Andale Mono">rom
                        char NET_REQGPS[] =
                        "AT+CGPSINF=2;+CGPSINF=64;+CGPSPWR=1\r";</font></div>
                    <div class=""><font class="" face="Andale Mono">#else</font></div>
                    <div class=""><font class="" face="Andale Mono">//
                        Using external GPS from car:</font></div>
                    <div class=""><font class="" face="Andale Mono">rom
                        char NET_INIT1[] = "AT+CSMINS?\r";</font></div>
                    <div class=""><font class="" face="Andale Mono">#endif</font></div>
                  </div>
                </blockquote>
                <div class=""><br class="">
                  So, if the build profile includes OVMS_INTERNALGPS
                  (ie; one or more vehicles in the profile require GPS),
                  then we use a big sequence of initialisations
                  "AT+CGPSPWR=1;+CGPSRST=0;+CSMINS?\rĒ. We have two
                  issues with this:</div>
                <div class=""><br class="">
                </div>
                <div class="">
                  <ol class="MailOutline">
                    <li class="">Weíre powering up the GPS, even if the
                      actual vehicle we selected doesnít need SIM908/808
                      GPS. That costs watts - and with my new fancy
                      bench power supply I got for OVMS v3 development
                      (ok, cheap chinese ebay) I can now see exactly how
                      much ;-).</li>
                    <li class="">The SIM808 randomly doesnít
                      like AT+CGPSPWR=1 and in particular seems to hate
                      it 100% of the time it is included with other
                      options. You get an ERROR response back from the
                      modem, and as this is done in NET_STATE_DOINIT, it
                      looks like a SIM card error (but is nothing to do
                      with SIM card). We just never saw AT+CGPSPWR=1
                      fail on SIM908.</li>
                  </ol>
                </div>
                <div class=""><br class="">
                </div>
                <div class="">Now, as well as the #defined
                  OVMS_INTERNALGPS to enable the SIM908/808 GPS, weíve
                  also got NET_FN_INTERNALGPS in net_fnbits which is set
                  during vehicle module initialisation. The first brings
                  in the code, the second should turn it on. Except it
                  doesnít. The first one turns it on.</div>
                <div class=""><br class="">
                </div>
                <div class="">Accordingly, Iím having to change the
                  net.{h,c} layer to split off GPS initialisation into
                  its own states and use a simple AT+CGPSPWR=1 for
                  vehicles that require SIM908/808 GPS and AT+CGPSPWR=0
                  for vehicles that donít. Iíve taken the opportunity to
                  switch from using OVMS_INTERNALGPS to using
                  NET_FN_INTERNALGPS in net_fnbits as the decision as to
                  whether to power up the SIM908/808 GPS or not.</div>
                <div class=""><br class="">
                </div>
                <div class="">Building and testing that has taken me the
                  most time. It is almost done, and I should be able to
                  commit the code soon.</div>
                <div class=""><br class="">
                </div>
                <div class="">Production firmware then goes to the
                  manufacturer in China, and we give the go-ahead for
                  the production run. We should have the modules back in
                  stock towards the end of March / first thing in April.</div>
                <div class=""><br class="">
                </div>
                <div class="">Regards, Mark.</div>
                <div class=""><br class="">
                </div>
                <div class="">P.S. I can actually recommend that
                  cheap(ish) chinese ebay power supply. Itís available
                  from various eBay sellers, or <a moz-do-not-send="true" href="http://www.banggood.com/CPS-3205C-0-32V-0-5A-Portable-Adjustable-DC-Power-Supply-180-264V-p-974495.html" class="">banggood</a>. Controls are precise, output
                  is rock solid, ammeter is as accurate as my
                  multi-meter, and it is built like a tank.</div>
                <div class=""><br class="">
                </div>
              </div>
              _______________________________________________<br class="">
              OvmsDev mailing list<br class="">
              <a moz-do-not-send="true" href="mailto:OvmsDev@lists.teslaclub.hk" class="">OvmsDev@lists.teslaclub.hk</a><br class="">
              <a class="moz-txt-link-freetext" href="http://lists.teslaclub.hk/mailman/listinfo/ovmsdev">http://lists.teslaclub.hk/mailman/listinfo/ovmsdev</a><br class="">
            </div>
          </blockquote>
        </div>
        <br class="">
      </div>
      <br class="">
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br class="">
      <pre wrap="" class="">_______________________________________________
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>
    <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>

<span id="cid:9C8AD97D-8BE6-4E97-83C9-BB7D95D8FA81"><dexter.vcf></span>_______________________________________________<br class="">OvmsDev mailing list<br class=""><a href="mailto:OvmsDev@lists.teslaclub.hk" class="">OvmsDev@lists.teslaclub.hk</a><br class="">http://lists.teslaclub.hk/mailman/listinfo/ovmsdev<br class=""></div></blockquote></div><br class=""></div></body></html>