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-broadband-bearer.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-broadband-bearer.c')
| -rw-r--r-- | src/mm-broadband-bearer.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/mm-broadband-bearer.c b/src/mm-broadband-bearer.c index bfe16183..109e8741 100644 --- a/src/mm-broadband-bearer.c +++ b/src/mm-broadband-bearer.c @@ -1011,10 +1011,10 @@ connect (MMBaseBearer *self, return; } - /* If no multiplex setting given by the user, assume default */ + /* If no multiplex setting given by the user, assume requested */ multiplex = mm_bearer_properties_get_multiplex (mm_base_bearer_peek_config (MM_BASE_BEARER (self))); if (multiplex == MM_BEARER_MULTIPLEX_SUPPORT_UNKNOWN) - multiplex = MM_BASE_BEARER_MULTIPLEX_SUPPORT_DEFAULT; + multiplex = MM_BEARER_MULTIPLEX_SUPPORT_REQUESTED; /* The generic broadband bearer doesn't support multiplexing */ if (multiplex == MM_BEARER_MULTIPLEX_SUPPORT_REQUIRED) { |