Parsing Cinterion RadioBand

[Colin Helliwell]

Yes, it looked a bit odd/unexpected to me. No clue in their spec about it being presented in that format rather than just "1-147". I can try asking them, but I suspect the answer will be 'it is what it is'.
There's some other lists given in a similar "x"-"y" format, but not all - doesn't seem consistent!
(Nervous about messing with the regex myself....!)
Full logged response is

ModemManager[904]: <debug> [1488807292.033357] [src/mm-port-serial-at.c:459] debug_log(): (ttyMux0): --> 'AT^SCFG=?<CR>'
ModemManager[904]: <debug> [1488807292.949283] [src/mm-port-serial-at.c:459] debug_log(): (ttyMux0): <-- '<CR><LF>^SCFG: "Audio/Loop",("0","1")<CR><LF>^SCFG: "Call/ECC",("0"-"255")<CR><LF>^SCFG: "Call/Ecall/AckTimeout",("0-2147483646")<CR><LF>^SCFG: "Call/Ecall/Callback",("0","1")<CR><LF>^SCFG: "Call/Ecall/CallbackTimeout",("0-2147483646")<CR><LF>^SCFG: "Call/Ecall/Msd",("280")<CR><LF>^SCFG: "Call/Ecall/Pullmode",("0","1")<CR><LF>^SCFG: "Call/Ecall/SessionTimeout",("0-2147483646")<CR><LF>^SCFG: "Call/Ecall/StartTimeout",("0-2147483646")<CR><LF>^SCFG: "Call/Speech/Codec",("0","1")<CR><LF>^SCFG: "GPRS/AutoAttach",("disabled","enabled")<CR><LF>^SCFG: "Gpio/mode/ASC1",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/DAI",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/DCD0",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/DSR0",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/DTR0",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/FSR",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/PULSE",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/PWM",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/RING0",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/SPI",("std","gpio","rsv")<CR><LF>^SCFG: "Gpio/mode/SYNC",("std","gpio","rsv")<CR><LF>^SCFG: "Ident/Manufacturer",(25)<CR><LF>^SCFG: "Ident/Product",(25)<CR><LF>^SCFG: "MEShutdown/Fso",("0","1")<CR><LF>^SCFG: "MEShutdown/sVsup/threshold",("-4","-3","-2","-1","0","1","2","3","4"),("0")<CR><LF>^SCFG: "MEopMode/CFUN",("0","1"),("1","4")<CR><LF>^SCFG: "MEopMode/Dormancy",("0","1")<CR><LF>^SCFG: "MEopMode/SoR",("off","on")<CR><LF>^SCFG: "Radio/Band",("1"-"147")<CR><LF>^SCFG: "Radio/Mtpl",("0"-"3"),("1"-"8"),("1","8"),("18"-"33"),("18"-"27")<CR><LF>^SCFG: "Radio/Mtpl",("0"-"3"),("1"-"8"),("16","32","64","128","256"),("18"-"24")<CR><LF>^SCFG: "Radio/Mtpl",("0"-"3"),("1"-"8"),("2","4"),("18"-"30"),("18"-"26")<CR><LF>^SCFG: "Radio/OutputPowerReduction",("0","1","2","3","4")<CR><LF>^SCFG: "Serial/Interface/Allocation",("0","1","2"),("0","1","2")<CR><LF>^SCFG: "Serial/USB/DDD",("0","1"),("0"),(4),(4),(4),(63),(63),(4)<CR><LF>^SCFG: "Tcp/IRT",("1"-"60")<CR><LF>^SCFG: "Tcp/MR",("1"-"30")<CR><LF>^SCFG: "Tcp/OT",("1"-"6000")<CR><LF>^SCFG: "Tcp/WithURCs",("on","off")<CR><LF>^SCFG: "Trace/Syslog/OTAP",("0","1"),("null","asc0","asc1","usb","usb1","usb2","usb3","usb4","usb5","file","udp","system"),("1"-"65535"),(125),("buffered","secure"),("off","on")<CR><LF>^SCFG: "URC/R'
ModemManager[904]: <debug> [1488807292.953537] [src/mm-port-serial-at.c:459] debug_log(): (ttyMux0): <-- 'ingline",("off","local","asc0")<CR><LF>^SCFG: "URC/Ringline/ActiveTime",("0","1","2")<CR><LF>^SCFG: "Userware/Autostart",("0","1")<CR><LF>^SCFG: "Userware/Autostart/Delay",("0"-"10000")<CR><LF>^SCFG: "Userware/DebugInterface",("0"-"255")|("FE80::"-"FE80::FFFFFFFFFFFFFFFF"),("0"-"255")|("FE80::"-"FE80::FFFFFFFFFFFFFFFF"),("0","1")<CR><LF>^SCFG: "Userware/DebugMode",("off","on")<CR><LF>^SCFG: "Userware/Passwd",("0"-"8")<CR><LF>^SCFG: "Userware/Stdout",("null","asc0","asc1","usb","usb1","usb2","usb3","usb4","usb5","file","udp","system"),("1"-"65535"),("0"-"125"),("buffered","secure"),("off","on")<CR><LF>^SCFG: "Userware/Watchdog",("0","1","2")<CR><LF><CR><LF>OK<CR><LF>'

> On 06 March 2017 at 12:00 Aleksander Morgado <aleksander at aleksander.es> wrote:
> 
> On Mon, Mar 6, 2017 at 11:38 AM, <colin.helliwell at ln-systems.com> wrote:
> 
> > I’m using the Cinterion plugin on an EHS5. This does include ‘Radio Band’ in
> > its response to ‘AT^SCFG=?’, but in a format that
> > mm_cinterion_parse_scfg_test () may not be able to parse?
> > 
> > ^SCFG: "Radio/Band",("1"-"147")
> > 
> > (Which I take to probably mean any bitmask from 1 to 128+16+2+1)
> 
> Yeah, that format isn't expected, see:
> https://cgit.freedesktop.org/ModemManager/ModemManager/tree/plugins/cinterion/mm-modem-helpers-cinterion.c#n68
> 
> We were expecting something like "1-147".
> 
> Could you share the whole response, not just that line, so that a full
> unit test can be added? see:
> https://cgit.freedesktop.org/ModemManager/ModemManager/tree/plugins/cinterion/tests/test-modem-helpers-cinterion.c#n72
> 
> Patches welcome, btw :)