[PATCH] telit: add error_code recognition to +CSIM parser

[Colin Helliwell]

> On 18 April 2017 at 16:40 Aleksander Morgado <aleksander at aleksander.es> wrote:
> 
> On Tue, Apr 18, 2017 at 5:20 PM, Colin Helliwell
> 
> <colin.helliwell at ln-systems.com> wrote:
> 
> > To keep things simple for now, these logs are with an unlocked SIM. This is with my modified Cinterion plugin, which I've chucked the +CSIM's into.
> > 
> > EHS5:
> >  Doesn't support ^SPIC at all.
> >  But (probably) all good with the +CSIM method.
> 
> Ok.
> 
> > BGS2:
> >  Doesn't support ^SPIC with an additional parameter - docs say that AT^SPIC="SC" will give the 'current' counter i.e. normally PIN retries, but instead the PUK retries after 3 failed PIN attempts.... (similarly for "P2" - might be PIN2, might be PUK2)
> 
> This is bad, we do want a way to query a specific counter, not just
> "the current one" which may be one or another depending on the current
> state...
> 

Yep. Another of Cinterion's naff way of implementing things. (They're clinging very tenuously to the design-in with us!)

> > Not sure why PS and PN fail.
> 
> Sorry, what are PS and PN?
> 

>From docs:
PS = "Phone code" or "device code" (cf. AT+CLCK and AT+CPWD). If incorrectly
entered three times, the Master Phone Code is required to lift the lock and the
number of remaining attempts for the master phonecode will be returned.
PN = Network Personalisation

Maybe not relevant to MM facilities. 
As you say, we need a way to get a *specific* counter, otherwise it's just way too wooly and tortured.