After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 697368 - [MM 0.8] Samsung Z810 replies "NA" to unsupported facility locks
[MM 0.8] Samsung Z810 replies "NA" to unsupported facility locks
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: ModemManager
unspecified
Other Linux
: Normal normal
: ---
Assigned To: NetworkManager maintainer(s)
NetworkManager maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-04-05 15:26 UTC by Dan Williams
Modified: 2013-04-05 17:12 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Dan Williams 2013-04-05 15:26:53 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>'
Comment 1 Aleksander Morgado 2013-04-05 15:45:37 UTC
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...
Comment 2 Aleksander Morgado 2013-04-05 16:10:57 UTC
Please test with the 'aleksander/na-fix' branch...
Comment 3 Dan Williams 2013-04-05 16:41:24 UTC
Works well, looks good.
Comment 4 Aleksander Morgado 2013-04-05 17:12:17 UTC
Merged to git master.