[PATCH 1/2] modem-helpers: if operator not in UCS2, see if already valid UTF-8

[Colin Helliwell]

> On 15 April 2017 at 08:21 Colin Helliwell <colin.helliwell at ln-systems.com> wrote:
> 
> > On 14 April 2017 at 23:15 Aleksander Morgado <aleksander at aleksander.es> wrote:
> > 
> > On Fri, Apr 14, 2017 at 4:57 PM, Colin Helliwell
> > 
> > <colin.helliwell at ln-systems.com> wrote:
> > 
> > > Might have hung this off the wrong thread, but it's along the same lines.
> > > I've just noticed that whilst the mmcli status now decodes the operator, the --sim status doesn't:
> > > 
> > > # mmcli -m 0
> > > ...
> > >  3GPP | imei: '358606050452806'
> > >  | enabled locks: 'none'
> > >  | operator id: '23415'
> > >  | operator name: 'vodafone UK'
> > >  | subscription: 'unknown'
> > > 
> > > | registration: 'home'
> > > 
> > > -------------------------
> > > ...
> > > 
> > > # mmcli -m 0 --sim=0
> > > 
> > > SIM '/org/freedesktop/ModemManager1/SIM/0'
> > > 
> > > -------------------------
> > >  Properties | imsi : '234159108784146'
> > >  | id : '89441000301658713096'
> > >  | operator id : '23415'
> > >  | operator name : 'unknown'
> > 
> > Could you gather debug logs so that we can see whether this is due to
> > the string being wrongly decoded or just not provided by the SIM?
> 
> Yep, it was something I spotted late on yesterday and fired in in case it was a quicky.
> At first I thought they both would have been coming from the same source command/response, though when I looked at the code quickly it was seeming perhaps not (which your comment also suggests).
> 
> *   I'll try to dig into it further today.
> 

Ah ok, I think it's just that the SIM is responding with a blank name:

[src/mm-port-serial-at.c:459] debug_log(): (ttyMux1): --> 'AT+CRSM=176,28486,0,0,17<CR>'
[src/mm-port-serial-at.c:459] debug_log(): (ttyMux1): <-- '<CR><LF>+CRSM: 144,0,"FFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF"<CR><LF>'
[src/mm-port-serial-at.c:459] debug_log(): (ttyMux1): <-- '<CR><LF>OK<CR><LF>'

So simply hasn't been set by the operator/supplier I guess