[PATCH] Qtopia: SMS Toolkit disable initialization

[Mike (mwester)]

I'm not at all sure what the SMS Toolkit is, there doesn't
seem to be any documentation.

I think it's disabled in the Qtopia build, but I'm not sure --
there's at least part of it in the initialization that isn't
and it seems that it can result in an unsolicited response that
makes Qtopia and the GSM angry with one-another and refuse to
speak with each other again.

At initialization time, we see this in the logs:

5076.460: -> AT%CSQ=1.
5076.465: <- AT%CSQ=1...%CSQ: 1 ....OK..
5076.485: -> AT%SATC=1,"FFFFFFFFFF".AT%CUNS=1.
5076.490: <- AT%SATC=1,"FFFFFFFFFF"...OK..AT%CUNS=1
5076.505: -> AT%CTZV=1.
5076.505: <- ...OK..AT%CTZV=1.

Note the AT%SATC sent to the modem.  Now, some time
later, note an unsolicited response in the log (%SATN):

5118.340: -> AT+CCFC=0,2.
5119.035: <- ..%SATN: "0000"..
5121.490: <- ..OK..
5121.495: -> AT+CCFC=0,2.

If that %SATN is sent as the very first string from the GSM upon
resume, then the GSM will respond with "ERROR" to each AT command
sent by Qtopia.  Perhaps the GSM is requiring some acknowledgment,
or whatever -- in any case, the GSM won't talk, Qtopia stops talking,
and a reboot is the only recourse.

The following patch fixes this.  I rather suspect there's a better way
to fix this, so perhaps if one of the Qtopians can comment that would be
good.  Without this patch, this problem would occur within the first 1/2
hour of use with my T-Mobile SIM.  With this patch, it has not failed
all day today.

Regards,
Mike (mwester)

--- git/devices/ficgta01/src/plugins/phonevendors/ficgta01/vendor_ficgta01.cpp.orig     2008-06-10 21:28:52.000000000 -0500
+++ git/devices/ficgta01/src/plugins/phonevendors/ficgta01/vendor_ficgta01.cpp  2008-06-11 18:38:16.000000000 -0500
@@ -513,7 +513,7 @@
      // Turn on SIM toolkit support in the modem.  This must be done
     // very early in the process, to ensure that it happens before
     // the first AT+CFUN command.
-    chat( "AT%SATC=1,\"FFFFFFFFFF\"", this, SLOT(configureDone(bool)) );
+     //    chat( "AT%SATC=1,\"FFFFFFFFFF\"", this, SLOT(configureDone(bool)) );

     // Enable %CPRI for ciphering indications.
 //    chat( "AT%CPRI=1" );