[RFC] Deprecate SubscriptionState?

Aleksander Morgado aleksander at aleksander.es
Wed Aug 8 11:35:38 UTC 2018


Hey,

>> > Why raw PCO data in hex string? Why not directly the binary data?
>> > e.g.
>> > "ay" instead of "s". The signature could be then "a(iay)"
>>
>> Yeah I'd vote for byte array eg "a(iay)" (eg array of struct of integer
>> + byte array).
>>
>
> Sounds good. We will need to indicate whether the received PCO is
> partial or complete, so I'll go with something like an array of struct
> of (uint32 + boolean + byte array),  i.e. a(ubay).
>
> I'm planning to do the following:
> 1. introduce the new Pco property under Modem3gpp interface
> 2. set it up for modems supporting MBIM_CID_PCO
> 3. migrate the altair-lte plugin to use the Pco property
> 4. migrate Chromium OS to use the Pco property instead of the
> SubscriptionState property
> 5. remove the SubscriptionState property from Modem3gpp -- this is an
> MM API break, so a version bump will be needed
>
> Does that sound like a plan?
>

LGTM.

For the last step, you could cherry pick this commit, where I don't
totally remove the property, but flag it as deprecated.
https://gitlab.freedesktop.org/mobile-broadband/ModemManager/commit/b382302ef64281329127066253eeeb4aa5736544


-- 
Aleksander
https://aleksander.es


More information about the ModemManager-devel mailing list