lpac: add uqmi backend #24883
Merged
lpac: add uqmi backend #24883
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BKPepe
reviewed
Aug 28, 2024
Comment on lines
+1
to
+4
| From 96c73de212c84caa1cc2796980e762321e0acdc3 Mon Sep 17 00:00:00 2001 | ||
| From: David Bauer <[email protected]> | ||
| Date: Wed, 27 Mar 2024 22:20:16 +0100 | ||
| Subject: [PATCH] driver: add uqmi backend |
There was a problem hiding this comment.
I couldn't find that it was submitted to lpac repository. Even though, this is OpenWrt specific, you should try to upstream it, because we will care your patch for ages and if there will be conflict every single time, someone from us will need to rebase it.
There was a problem hiding this comment.
Decision on this is pending and dependent on the progress on uqmid. No need to upstream if the rewrite is months away.
Also i don't think a segregation in "you" and "we" is neither respectful nor adequate regardless of who opened the PR.
There was a problem hiding this comment.
Same could be said in self merging PRs. ;-)
added 3 commits
August 28, 2024 23:51
fd58ad7 Bump 2.1.0 319f55f chore(docs): add interactive preview mode option da1cf1a update download cancel process cee7190 workaround for es9p cancel session override error_detail b11878f typo e6b32e1 Merge branch 'main' of github.com:estkme/lpac 302984f preview && cancel session 56352c5 docs: Add parameters for 'notification process' (openwrt#144) 19df553 chore(driver): improve memory management in at driver (openwrt#140) c86edd5 chore(driver): set pcsc and curl as default drivers. (openwrt#137) 3bde4a1 driver(APDU): add QMI backend (openwrt#131) 0011ea6 docs: add link to lpa-gtk ui for lpac (openwrt#125) d508b28 refactor: bash scripts (openwrt#123) f5956dd chore: move rlpa-server to independent repo (openwrt#122) Signed-off-by: David Bauer <[email protected]>
This adds an additional OpenWrt specific backend to use lpac with the uqmi tooling used by OpenWrt to manage QMI based modems. This allows lpac to manage eUICC chips without the need for other, potentially bigger, software using the installed modem. Also set this backend as the new default, as users probably expect being able to download profiles using their cellular modem. Profile-Switching ----------------- With some eUICC modem combinations you might require to powercycle the UIM slot using uqmi after changing the active profile. To do this, simply execute $ uqmi -d /dev/cdc-wdm0 --uim-power-off --uim-slot=1 $ uqmi -d /dev/cdc-wdm0 --uim-power-on --uim-slot=1 Note ---- SM-DP+ might use encryption parameters which are incompatible with mbedtls which cURL is by default compiled for. This was observed when attempting to download a profile from Vodafone DE. If you encounter issues in that regard, try to install a version of libcurl which is compiled with OpenSSL support. Signed-off-by: David Bauer <[email protected]>
Signed-off-by: David Bauer <[email protected]>
|
Updates: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Maintainer: me
Compile tested: ath79-nand
Run tested: ath79-nand / GL.iNet XE300
Description:
This adds an additional OpenWrt specific backend to use lpac with the
uqmi tooling used by OpenWrt to manage QMI based modems.
This allows lpac to manage eUICC chips without the need for other,
potentially bigger, software using the installed modem.
Also set this backend as the new default, as users probably expect being
able to download profiles using their cellular modem.
Profile-Switching
With some eUICC modem combinations you might require to powercycle the
UIM slot using uqmi after changing the active profile. To do this,
simply execute
$ uqmi -d /dev/cdc-wdm0 --uim-power-off --uim-slot=1
$ uqmi -d /dev/cdc-wdm0 --uim-power-on --uim-slot=1
Note
SM-DP+ might use encryption parameters which are incompatible with
mbedtls which cURL is by default compiled for.
This was observed when attempting to download a profile from
Vodafone DE.
If you encounter issues in that regard, try to install a version of
libcurl which is compiled with OpenSSL support.