Attention is currently required from: jolly. dexter 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: Code-Review+1 (1 comment) File src/ipa/main.c: https://gerrit.osmocom.org/c/onomondo-ipa/+/43060/comment/04720e8d_74ac1a90?usp=email : PS1, Line 264: case 'D': I wonder if it would make sense to accept a string like "00001111". A one byte hex number would also be ok. Somewhat unrelated, but helpful when fixed/improved: One thing that would be really helpful would be a converter that converts from ASCII hex digits to binary hex. This would solve a very annoying problem: The eIM generates an eIM configuration and outputs it as ASCII hex digits on the REST API. To use this output we have to convert it to binary using external tools to get the sample_eim_cfg.ber file you find in contrib. When we have a conveter, we could just use a text file with the ASCII hex string or even switch to a commandline parameter. After all, you could then use the converter here to set cfg.device_capabilities from a 2-hex-digit string (If you think this would make sense). -- 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 <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <[email protected]> Gerrit-Attention: jolly <[email protected]> Gerrit-Comment-Date: Thu, 23 Jul 2026 09:11:32 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes
