[PATCH 4/4] telit: remove custom +CMER enabling

[Aleksander Morgado]

On Mon, May 22, 2017 at 6:09 PM, Dan Williams <dcbw at redhat.com> wrote:
>> And now that I see this...
>>  * Shouldn't we enable +CIEV only for "signal" (that's the only thing
>> we end up parsing in the generic +CIEV parser)?
>>  * Shouldn't we move this setting to the generic broadband modem
>> implementation?
>
> All three of these (signal, service, roam) are pretty standard.  Why
> not move all of them to the generic implementation and make use of them
> if we can?

In the generic implementation we already have those 3 being
"supported" more or less. We do run +CIND=? and get the indices for
each item, but if a +CIEV URC arrives with any of them set, we only
really parse and use the "signal". We don't parse and use service or
roam because we are already doing processing the same info via
+CREG/+CGREG/+CEREG indications. Question now being, should we parse
and use service/roam values reported via +CIEV URCs? Is there any info
returned in that way that we wouldn't get from +CREG/+CGREG/+CEREG
indications?

The change in the Telit plugin also seems to explicitly enable those
indications; while the generic plugin assumes they're enabled. Maybe
we shouldn't assume anything and we should directly request them. But
we wouldn't have a fixed command like in the Telit plugin, instead we
would make use of the indices of each item to construct the correct
command (i.e. without assuming in which index each item is).

-- 
Aleksander
https://aleksander.es