diff options
| author | Aleksander Morgado <aleksander@aleksander.es> | 2021-07-28 23:42:56 +0200 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2021-08-09 12:22:26 +0000 |
| commit | 8d019a399ca8dd29f959477e2be36b252497a90b (patch) | |
| tree | d587b9162b0885b7ff5b8e681c1b0b6c72f4720f /src/mm-bearer-qmi.c | |
| parent | 4a07ab39f14d3879c9168e5d7da3b8a0a32bfb61 (diff) | |
base-bearer: remove default multiplex support symbol
Using "requested" as a common default for every possible
implementation works, because it can handle setups without
multiplexing capabilities (e.g. plain AT+PPP modems) and also setups
without non-multiplexing capabilities (e.g. IPA based SoCs).
But if we don't want "requested" as default, then there will be
failing cases; e.g. we cannot use "none" as default for all as it
would break IPA, and we cannot use "required" as default for all as it
would break AT+PPP setups.
So, remove the common default, it's not flexible enough.
Diffstat (limited to 'src/mm-bearer-qmi.c')
| -rw-r--r-- | src/mm-bearer-qmi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/mm-bearer-qmi.c b/src/mm-bearer-qmi.c index 0bb5b81c..86959765 100644 --- a/src/mm-bearer-qmi.c +++ b/src/mm-bearer-qmi.c @@ -2141,10 +2141,10 @@ load_settings_from_bearer (MMBearerQmi *self, GError *inner_error = NULL; const gchar *str; - /* If no multiplex setting given by the user, assume default */ + /* If no multiplex setting given by the user, assume requested */ ctx->multiplex = mm_bearer_properties_get_multiplex (properties); if (ctx->multiplex == MM_BEARER_MULTIPLEX_SUPPORT_UNKNOWN) - ctx->multiplex = MM_BASE_BEARER_MULTIPLEX_SUPPORT_DEFAULT; + ctx->multiplex = MM_BEARER_MULTIPLEX_SUPPORT_REQUESTED; /* The link prefix hint given must be modem-specific */ if (ctx->multiplex == MM_BEARER_MULTIPLEX_SUPPORT_REQUESTED || |