Hi Christophe,
> This worked on a bunch of QMI based devices, so I have to assume
that
> this fails on Sierra? If so, then this probably should be quirked to
> be Sierra-specific behavior. E.g. via a mechanism similar to
> drivers/atmodem/vendor.h
IMSI reading on my MC7430 modem do fail without this patch.
I have the following traces in this case:
Jan 5 11:08:41 klk-lpbs_040070 daemon.debug ofonod[869]:
../ofono-1.19/drivers/qmimodem/sim.c:qmi_read_transparent() file id
0x6f07 path len 0
Jan 5 11:08:41 klk-lpbs_040070 daemon.debug ofonod[869]:
../ofono-1.19/drivers/qmimodem/sim.c:read_generic_cb()
Jan 5 11:08:41 klk-lpbs_040070 daemon.err ofonod[869]: Unable to read
IMSI, emergency calls only
Do any reads from the SIM succeed on the sierra devices? Perhaps they
need a different set of magic than what is currently implemented? I
know we tested this on Huawei and a few other vendors and reading the
IMSI via qmi_read_transparent worked fine.
>
> ofono_sim_remove will call .remove method. So this is likely to cause
> a crash since uim and dms have already been unref-ed.
OK, I have not tested this error case. The
"qmi_service_unref(data->uim)" that's here in this function without the
patch must cause the same crash.
It probably is a bug as well, yes.
Regards,
-Denis