GNOME Bugzilla – Bug 697368
[MM 0.8] Samsung Z810 replies "NA" to unsupported facility locks
Last modified: 2013-04-05 17:12:25 UTC
Not a huge problem, but it causes an initialization delay because "NA" isn't a recognized response. Incredibly stupid firmware. [1365174577.715994] [mm-broadband-modem.c:3090] modem_3gpp_load_enabled_facility_locks(): loading enabled facility locks... [1365174577.716030] [mm-serial-port.c:958] mm_serial_port_open(): (ttyUSB6) device open count is 3 (open) [1365174577.716068] [mm-serial-port.c:1003] mm_serial_port_close(): (ttyUSB6) device open count is 2 (close) [1365174577.716107] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): --> 'AT+CLCK=?<CR>' [1365174577.728834] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): <-- '<CR><LF>+CLCK: ("AB","AC","AG","AI","AO","IR","OI","OX","SC","FD","PN","PU","PP","PC","PF")<CR><LF><CR><LF>OK<CR><LF>' [1365174577.728968] [mm-serial-port.c:958] mm_serial_port_open(): (ttyUSB6) device open count is 3 (open) [1365174577.729012] [mm-serial-port.c:1003] mm_serial_port_close(): (ttyUSB6) device open count is 2 (close) [1365174577.729051] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): --> 'AT+CLCK="SC",2<CR>' [1365174577.748887] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): <-- '<CR><LF>+CLCK: 1<CR><LF><CR><LF>OK<CR><LF>' [1365174577.749118] [mm-serial-port.c:958] mm_serial_port_open(): (ttyUSB6) device open count is 3 (open) [1365174577.749222] [mm-serial-port.c:1003] mm_serial_port_close(): (ttyUSB6) device open count is 2 (close) [1365174577.749290] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): --> 'AT+CLCK="FD",2<CR>' [1365174577.766829] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): <-- '<CR><LF>+CLCK: 0<CR><LF><CR><LF>OK<CR><LF>' [1365174577.766925] [mm-serial-port.c:958] mm_serial_port_open(): (ttyUSB6) device open count is 3 (open) [1365174577.766969] [mm-serial-port.c:1003] mm_serial_port_close(): (ttyUSB6) device open count is 2 (close) [1365174577.767000] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): --> 'AT+CLCK="PF",2<CR>' [1365174577.790722] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): <-- '<CR><LF>NA<CR><LF>' [1365174581.028352] [mm-serial-port.c:958] mm_serial_port_open(): (ttyUSB6) device open count is 3 (open) [1365174581.028499] [mm-serial-port.c:1003] mm_serial_port_close(): (ttyUSB6) device open count is 2 (close) [1365174581.028577] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): --> 'AT+CLCK="PN",2<CR>' [1365174581.047865] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): <-- '<CR><LF>NA<CR><LF>' [1365174584.028226] [mm-serial-port.c:958] mm_serial_port_open(): (ttyUSB6) device open count is 3 (open) [1365174584.028386] [mm-serial-port.c:1003] mm_serial_port_close(): (ttyUSB6) device open count is 2 (close) [1365174584.028465] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): --> 'AT+CLCK="PU",2<CR>' [1365174584.046869] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): <-- '<CR><LF>NA<CR><LF>' [1365174587.028291] [mm-serial-port.c:958] mm_serial_port_open(): (ttyUSB6) device open count is 3 (open) [1365174587.028446] [mm-serial-port.c:1003] mm_serial_port_close(): (ttyUSB6) device open count is 2 (close) [1365174587.028525] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): --> 'AT+CLCK="PP",2<CR>' [1365174587.047840] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): <-- '<CR><LF>NA<CR><LF>' [1365174590.028096] [mm-serial-port.c:958] mm_serial_port_open(): (ttyUSB6) device open count is 3 (open) [1365174590.028243] [mm-serial-port.c:1003] mm_serial_port_close(): (ttyUSB6) device open count is 2 (close) [1365174590.028320] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): --> 'AT+CLCK="PC",2<CR>' [1365174590.047218] [mm-at-serial-port.c:408] debug_log(): (ttyUSB6): <-- '<CR><LF>NA<CR><LF>'
The real problem, as far as I can see, is the lack of "OK" after that NA, which makes us not process even the "NA" string. Easy fix I believe...
Please test with the 'aleksander/na-fix' branch...
Works well, looks good.
Merged to git master.