<div dir="ltr"><div>Hi to everybody,</div><div><br></div><div>I am working on a fix for Bug 93135 - Telit HE910: `+CPMS="",...` fails as "Operation Not Supported". </div><div>The problem is that Telit modems does not support empty strings as +CPMS parameter, </div><div>while in MM the empty parameter is used in order to keep the current memory configuration for</div><div>that field (like AT+CPMS="","ME")</div><div><br></div><div>I followed the hint given by Aleksander in a previous e-mail in order to have mem1 set to the current</div><div>supported storage, if not set otherwise, in place of being unknown</div><div><br></div><div>> [...] what I would do, is to provide in MMBroadbandModemClass a new</div><div>> async virtual method to "load_current_sms_mem_storage()" (or something</div><div>> like that). The logic in MMBroadbandModem which checks which are the</div><div>> supported storages could then have a new step to query which is the</div><div>> current storage (which we only need to do once), all this in the</div><div>> MMBroadbandModem class.</div><div><br></div><div>however the same kind of AT command is emitted in sms_store function (mm-sms-base.c), where</div><div>mm_broadband_modem_lock_sms_storages is called with mem1 explicitly set to UNKNOWN:</div><div><br></div><div>    mm_broadband_modem_lock_sms_storages (</div><div>        MM_BROADBAND_MODEM (self->priv->modem),</div><div>        MM_SMS_STORAGE_UNKNOWN, /* none required for mem1 */</div><div>        ctx->storage,</div><div>        (GAsyncReadyCallback)store_lock_sms_storages_ready,</div><div>        ctx);</div><div><br></div><div>I do not get why none is required for mem1 in this case and I would like to know if can be considered</div><div>safe to use the value of "current_sms_mem1_storage" when the local value of mem1 in</div><div>mm_broadband_modem_lock_sms_storages is UNKNOWN</div><div><br></div><div>Best regards,</div><div>Carlo Lobrano</div></div>