Attention is currently required from: dexter.
laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/42994?usp=email )
Change subject: doc/build.md: fix typo
......................................................................
Patch Set 3:
(1 comment)
Patchset:
PS3:
it's not really a typo, but a question whether you'd want american or british english?
--
To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42994?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: onomondo-eim
Gerrit-Branch: master
Gerrit-Change-Id: Ie6a066f3289fbdde40f55bd9f38a3dd90102e7e6
Gerrit-Change-Number: 42994
Gerrit-PatchSet: 3
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 16 Jul 2026 14:43:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: dexter.
laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/onomondo-eim/+/42993?usp=email )
Change subject: mnesia_db_euicc: renovate handling of euicc table (mnesia, REST)
......................................................................
Patch Set 3:
(1 comment)
Patchset:
PS3:
as with other patches changing the JSON schema: shouldn't there be some kind of version in it, and that version being bumped before making incompatible changes? So that any user of the JSON interface could at runtime inquire about the schema version used by the eIM and decide if it can work with that version or raise an error?
--
To view, visit https://gerrit.osmocom.org/c/onomondo-eim/+/42993?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: onomondo-eim
Gerrit-Branch: master
Gerrit-Change-Id: I74c602555b194a28d6eb9dd67ec4c6a8865fcb77
Gerrit-Change-Number: 42993
Gerrit-PatchSet: 3
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Thu, 16 Jul 2026 14:42:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Jenkins Builder has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43032?usp=email )
Change subject: V1.2: Add new fields to ProfileInfo list
......................................................................
Patch Set 1:
(3 comments)
File src/ipa/libipa/es10c_get_prfle_info.c:
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/onomondo-ipa/+/43032/comment/2247934f_d9252347… :
PS1, Line 78: if (conv_req && conv_req->fallback_attribute_removed) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/onomondo-ipa/+/43032/comment/8dc9f3f6_a02c5a59… :
PS1, Line 109: struct ipa_es10c_get_prfle_info_req *conv_get_prfle_info_req(const struct ipa_es10c_get_prfle_info_req *req) {
open brace '{' following function definitions go on the next line
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/onomondo-ipa/+/43032/comment/d5fb15a4_452e81f6… :
PS1, Line 110: uint8_t default_tags[] = { 0x5A, 0x4F, 0x9F, 0x70, 0x91, 0x92, 0x95, 0x9F,0x7B, 0x9F,0x26, 0x9F,0x67 };
space required after that ',' (ctx:VxV)
--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43032?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: I6667dba8dd84817cfee69daa1c44b98239fa171c
Gerrit-Change-Number: 43032
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 16 Jul 2026 14:23:21 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Jenkins Builder has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43029?usp=email )
Change subject: V1.2: Add EimConfigurationData.indirectProfileDownload
......................................................................
Patch Set 1:
(1 comment)
File src/ipa/libipa/proc_eim_pkg_retr.c:
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/onomondo-ipa/+/43029/comment/d2cfef33_d353eb21… :
PS1, Line 122: "Indirect profile download is not suppoted by eIM, we don't support direct profile download, -- cannot continue!\n");
'suppoted' may be misspelled - perhaps 'supported'?
--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43029?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: Ic40111531e6f35d5ea7f3789de6cae0f1c0d5ba7
Gerrit-Change-Number: 43029
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 16 Jul 2026 14:22:58 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Jenkins Builder has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43060?usp=email )
Change subject: V1.2: Add function to set Device Capabilities
......................................................................
Patch Set 1:
(2 comments)
File include/onomondo/ipa/ipad.h:
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/onomondo-ipa/+/43060/comment/c3823fc2_cabc9083… :
PS1, Line 90: /*! Set the Device Capabilites.
'Capabilites' may be misspelled - perhaps 'Capabilities'?
File src/ipa/main.c:
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/onomondo-ipa/+/43060/comment/b2f7a23b_6e5266b4… :
PS1, Line 67: printf(" -D ......................... set SGP.22 device capabilites other than the 3 that are required by this IPAd\n");
'capabilites' may be misspelled - perhaps 'capabilities'?
--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43060?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: I866787b8cda9a87a9f2e0db4965fbb554d339a6c
Gerrit-Change-Number: 43060
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 16 Jul 2026 14:22:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Jenkins Builder has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43047?usp=email )
Change subject: V1.2: Add TERMINAL CAPABILITY for IoT support
......................................................................
Patch Set 1:
(2 comments)
File src/ipa/libipa/euicc.c:
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/onomondo-ipa/+/43047/comment/fbd21064_30159478… :
PS1, Line 539: /* Set capabilites to enable support for eUICC functions. */
'capabilites' may be misspelled - perhaps 'capabilities'?
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/onomondo-ipa/+/43047/comment/b2eb8cae_9c3fe749… :
PS1, Line 544: /* Set capabilites to enable support for IoT functions. */
'capabilites' may be misspelled - perhaps 'capabilities'?
--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43047?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: I1116bee93b31ee4a159125d840d7c5866be6fdb4
Gerrit-Change-Number: 43047
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Thu, 16 Jul 2026 14:21:50 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
jolly has uploaded this change for review. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43060?usp=email )
Change subject: V1.2: Add function to set Device Capabilities
......................................................................
V1.2: Add function to set Device Capabilities
Add new library function to set the SGP.22 Device Capabilities. Also
add an option to the command line interface to call the library
function.
The IoT device (modem) may and the IPA must set Device Capabilities.
They cannot be set independently. The IoT device would overwrite what
IPA has set and vice versa. Therefore add a library function for the
IoT device that takes care about setting the required capabilities for
IPA.
Reference: SGP.22 Section 3.4.2
Related: SYS#8101
Change-Id: I866787b8cda9a87a9f2e0db4965fbb554d339a6c
---
M include/onomondo/ipa/ipad.h
M src/ipa/libipa/euicc.c
M src/ipa/main.c
3 files changed, 13 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/onomondo-ipa refs/changes/60/43060/1
diff --git a/include/onomondo/ipa/ipad.h b/include/onomondo/ipa/ipad.h
index ff3bd68..d5d4b58 100644
--- a/include/onomondo/ipa/ipad.h
+++ b/include/onomondo/ipa/ipad.h
@@ -86,6 +86,12 @@
* function here to handle the consent request. In case no callback function is provided onomondo-eim will
* automatically consent to any profile installation. */
ipa_prfle_inst_consent_cb prfle_inst_consent_cb;
+
+ /*! Set the Device Capabilites.
+ * As IPAd need to set the device capabilities, it overwrites any other capabilities that might have set by
+ * the modem or IoT device. The upper 5 bits can be set according to section 3.4.2 of SGG.22. The lower 3
+ * bit are set automatically by this IPAd. */
+ uint8_t device_capabilities;
};
struct ipa_context *ipa_new_ctx(struct ipa_config *cfg, struct ipa_buf *nvstate);
diff --git a/src/ipa/libipa/euicc.c b/src/ipa/libipa/euicc.c
index efda3c7..ac03d0f 100644
--- a/src/ipa/libipa/euicc.c
+++ b/src/ipa/libipa/euicc.c
@@ -532,11 +532,12 @@
* \returns 0 on success, negative on error. */
int ipa_euicc_init_es10x(struct ipa_context *ctx)
{
- const uint8_t termcap_euicc[] = { 0xA9, 0x03, 0x83, 0x01, 0x07 };
+ uint8_t termcap_euicc[] = { 0xA9, 0x03, 0x83, 0x01, 0x07 };
const uint8_t termcap_iot[] = { 0xA9, 0x03, 0x84, 0x01, 0x01 };
int rc;
/* Set capabilites to enable support for eUICC functions. */
+ termcap_euicc[4] |= ctx->cfg->device_capabilities;
rc = send_termcap(ctx, termcap_euicc, sizeof(termcap_euicc));
if (rc < 0)
return rc;
diff --git a/src/ipa/main.c b/src/ipa/main.c
index 3e7b7a7..840aaba 100644
--- a/src/ipa/main.c
+++ b/src/ipa/main.c
@@ -64,6 +64,7 @@
printf(" -I ......................... disable SSL certificate verification (insecure)\n");
printf(" -E ......................... emulate IoT eUICC (compatibility mode to use consumer eUICCs)\n");
printf(" -1 ......................... force the IPAd to process only one eUICC package (debug, use with caution)\n");
+ printf(" -D ......................... set SGP.22 device capabilites other than the 3 that are required by this IPAd\n");
}
struct ipa_buf *load_ber_from_file(char *dir, char *file)
@@ -260,6 +261,9 @@
case '1':
getopt_one_euicc_pkg_only = true;
break;
+ case 'D':
+ cfg.device_capabilities = atoi(optarg);
+ break;
default:
printf("unhandled option: %c!\n", opt);
break;
@@ -280,6 +284,7 @@
printf(" iot_euicc_emu_enabled = %u\n", cfg.iot_euicc_emu_enabled);
printf(" esipa_req_retries = %u\n", cfg.esipa_req_retries);
printf(" refresh_flag = %u\n", cfg.refresh_flag);
+ printf(" device_capabilities = %u\n", cfg.device_capabilities);
printf("\n");
if (cfg.eim_cabundle) {
--
To view, visit https://gerrit.osmocom.org/c/onomondo-ipa/+/43060?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: onomondo-ipa
Gerrit-Branch: master
Gerrit-Change-Id: I866787b8cda9a87a9f2e0db4965fbb554d339a6c
Gerrit-Change-Number: 43060
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>