tty: n_gsm: Fix DLCI handling for ADM mode if debug & 2 is not set
authorTony Lindgren <>
Sat, 7 Apr 2018 17:19:51 +0000 (10:19 -0700)
committerGreg Kroah-Hartman <>
Sun, 22 Apr 2018 15:34:40 +0000 (17:34 +0200)
tty: n_gsm: Fix DLCI handling for ADM mode if debug & 2 is not set

At least on droid 4 with control channel in ADM mode, there is no response
to Modem Status Command (MSC). Currently gsmtty_modem_update() expects to
have data in dlci->modem_rx unless debug & 2 is set. This means that on
droid 4, things only work if debug & 2 is set.

Let's fix the issue by ignoring empty dlci->modem_rx for ADM mode. In
the AMD mode, CMD_MSC will never respond and gsm_process_modem() won't
get called to set dlci->modem_rx.

And according to ts_127010v140000p.pdf, MSC is only relevant if basic
option is chosen, so let's test for that too.

Fixes: ea3d8465ab9b ("tty: n_gsm: Allow ADM response in addition to UA for control dlci")
Cc: Alan Cox <>
Cc: Dan Williams <>
Cc: Jiri Prchal <>
Cc: Jiri Slaby <>
Cc: Marcel Partap <>
Cc: Merlijn Wajer <>
Cc: Michael Nazzareno Trimarchi <>
Cc: Michael Scott <>
Cc: Pavel Machek <>
Cc: Peter Hurley <>
Cc: Russ Gorby <>
Cc: Sascha Hauer <>
Cc: Sebastian Reichel <>
Signed-off-by: Tony Lindgren <>
Cc: stable <>
Signed-off-by: Greg Kroah-Hartman <>