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?... : 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).