2:11 a.m.
From: Richard Röjfors <richard(a)puffinpack.se>
It seems like the CREG reporting from the L2 modems are quite
buggy. An example for a L210 where CREG reports UTRAN while
COPS and UREG reports LTE. A manual poll also indicates LTE.
I also found that the technology mapping was incorrect,
probably confused with enum packet_bearer.
A commented log showing where CREG is not trustable:
UREG indicates LTE
21:59:29 : < \r\n+UREG: 7\r\n
21:59:29 : < \r\n+CIEV: 9,2\r\n
21:59:29 : < \r\n+CGEV: NW MODIFY 1,0,0\r\n
21:59:31 : < \r\n+CIEV: 2,2\r\n
21:59:39 : < \r\n+CIEV: 2,3\r\n
21:59:44 : < \r\n+CIEV: 2,2\r\n
22:01:38 : < \r\n+CIEV: 2,3\r\n
22:01:43 : < \r\n+CIEV: 2,2\r\n
A CREG indicating UTRAN with HSDPA and HSUPA
22:29:39 : < \r\n+CREG: 5,"0000","00000000",6\r\n
22:29:39 : > AT\r
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS=3,2\r
22:29:39 : < \r\n+CIEV: 9,2\r\n
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS?\r
An immediate cops indicating LTE
22:29:39 : < \r\n+COPS: 0,2,"24007",7\r\n
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+CSQ\r
22:29:39 : < \r\n+CIEV: 2,4\r\n
22:29:39 : < \r\n+CSQ: 26,4\r\n
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+CGATT=1\r
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS=3,0\r
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS?\r
Another cops also indicates LTE
22:29:39 : < \r\n+COPS: 0,0,"Tele2",7\r\n <- 7: LTE
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+CGREG?\r
CGREG indicates unknown -> normal on LTE
22:29:39 : < \r\n+CGREG: 2,4\r\n
22:29:39 : < \r\nOK\r\n
22:29:44 : < \r\n+CIEV: 9,2\r\n
22:29:46 : < \r\n+CIEV: 2,2\r\n
22:56:23 : < \r\n+CIEV: 2,3\r\n
22:56:28 : < \r\n+CIEV: 2,2\r\n
22:59:40 : < \r\n+CIEV: 2,4\r\n
Manual poll shows we are running LTE
at+creg?
+CREG: 2,5,"2AFC","01DB0206",7
OK
---
drivers/ubloxmodem/network-registration.c | 34 +++++++++++++++++++----
1 file changed, 28 insertions(+), 6 deletions(-)
diff --git a/drivers/ubloxmodem/network-registration.c
b/drivers/ubloxmodem/network-registration.c
index 64ef8076..69af4644 100644
--- a/drivers/ubloxmodem/network-registration.c
+++ b/drivers/ubloxmodem/network-registration.c
@@ -58,6 +58,7 @@ struct tech_query {
int status;
int lac;
int ci;
+ int tech;
struct ofono_netreg *netreg;
};
@@ -235,20 +236,40 @@ static void ublox_query_tech_cb(gboolean ok, GAtResult *result,
return;
switch (state) {
+ case 0:
+ /* Not registered for PS, then we have to trust CREG... */
+ tech = tq->tech;
+ break;
+ case 1:
+ tech = ACCESS_TECHNOLOGY_GSM;
+ break;
+ case 2:
+ tech = ACCESS_TECHNOLOGY_GSM_EGPRS;
+ break;
+ case 3:
+ tech = ACCESS_TECHNOLOGY_UTRAN;
+ break;
case 4:
- tech = 5;
+ tech = ACCESS_TECHNOLOGY_UTRAN_HSDPA;
break;
case 5:
- tech = 4;
+ tech = ACCESS_TECHNOLOGY_UTRAN_HSUPA;
+ break;
+ case 6:
+ tech = ACCESS_TECHNOLOGY_UTRAN_HSDPA_HSUPA;
+ break;
+ case 7:
+ tech = ACCESS_TECHNOLOGY_EUTRAN;
break;
case 8:
- tech = 1;
+ tech = ACCESS_TECHNOLOGY_GSM;
break;
case 9:
- tech = 2;
+ tech = ACCESS_TECHNOLOGY_GSM_EGPRS;
break;
default:
- tech = state;
+ /* Not registered for PS or something unknown, trust CREG... */
+ tech = tq->tech;
}
error:
@@ -270,12 +291,13 @@ static void creg_notify(GAtResult *result, gpointer user_data)
if (status != 1 && status != 5)
goto notify;
- if (ublox_is_toby_l4(nd->model)) {
+ if (ublox_is_toby_l4(nd->model) || ublox_is_toby_l2(nd->model)) {
tq = g_new0(struct tech_query, 1);
tq->status = status;
tq->lac = lac;
tq->ci = ci;
+ tq->tech = tech;
tq->netreg = netreg;
if (g_at_chat_send(nd->at_data.chat, "AT+UREG?", ureg_prefix,
--
2.20.1
2:05 p.m.
Hi Richard,
On 24/09/2019 18:11, richard.rojfors(a)gmail.com wrote:
From: Richard Röjfors <richard(a)puffinpack.se>
It seems like the CREG reporting from the L2 modems are quite
buggy. An example for a L210 where CREG reports UTRAN while
COPS and UREG reports LTE. A manual poll also indicates LTE.
I also found that the technology mapping was incorrect,
probably confused with enum packet_bearer.
A commented log showing where CREG is not trustable:
UREG indicates LTE
21:59:29 : < \r\n+UREG: 7\r\n
21:59:29 : < \r\n+CIEV: 9,2\r\n
21:59:29 : < \r\n+CGEV: NW MODIFY 1,0,0\r\n
21:59:31 : < \r\n+CIEV: 2,2\r\n
21:59:39 : < \r\n+CIEV: 2,3\r\n
21:59:44 : < \r\n+CIEV: 2,2\r\n
22:01:38 : < \r\n+CIEV: 2,3\r\n
22:01:43 : < \r\n+CIEV: 2,2\r\n
A CREG indicating UTRAN with HSDPA and HSUPA
22:29:39 : < \r\n+CREG: 5,"0000","00000000",6\r\n
22:29:39 : > AT\r
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS=3,2\r
22:29:39 : < \r\n+CIEV: 9,2\r\n
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS?\r
An immediate cops indicating LTE
22:29:39 : < \r\n+COPS: 0,2,"24007",7\r\n
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+CSQ\r
22:29:39 : < \r\n+CIEV: 2,4\r\n
22:29:39 : < \r\n+CSQ: 26,4\r\n
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+CGATT=1\r
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS=3,0\r
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS?\r
Another cops also indicates LTE
22:29:39 : < \r\n+COPS: 0,0,"Tele2",7\r\n <- 7: LTE
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+CGREG?\r
CGREG indicates unknown -> normal on LTE
22:29:39 : < \r\n+CGREG: 2,4\r\n
22:29:39 : < \r\nOK\r\n
22:29:44 : < \r\n+CIEV: 9,2\r\n
22:29:46 : < \r\n+CIEV: 2,2\r\n
22:56:23 : < \r\n+CIEV: 2,3\r\n
22:56:28 : < \r\n+CIEV: 2,2\r\n
22:59:40 : < \r\n+CIEV: 2,4\r\n
Manual poll shows we are running LTE
at+creg?
+CREG: 2,5,"2AFC","01DB0206",7
OK
For the TOBY-L4, RAT changes won't trigger a CREG URC... for whatever
reason, this command is so brittle that they needed to bypass it with a
UREG command. For the L2, they don't bother to document this, but it
appears to suffer from the same deficiencies as the L4. CGREG and
friends appear to suffer from the same ailment... they don't have a firm
grasp of the overall state so they return "unknown" as a fallback.
---
drivers/ubloxmodem/network-registration.c | 34 +++++++++++++++++++----
1 file changed, 28 insertions(+), 6 deletions(-)
diff --git a/drivers/ubloxmodem/network-registration.c
b/drivers/ubloxmodem/network-registration.c
index 64ef8076..69af4644 100644
--- a/drivers/ubloxmodem/network-registration.c
+++ b/drivers/ubloxmodem/network-registration.c
@@ -58,6 +58,7 @@ struct tech_query {
int status;
int lac;
int ci;
+ int tech;
struct ofono_netreg *netreg;
};
@@ -235,20 +236,40 @@ static void ublox_query_tech_cb(gboolean ok, GAtResult *result,
return;
switch (state) {
+ case 0:
+ /* Not registered for PS, then we have to trust CREG... */
+ tech = tq->tech;
+ break;
+ case 1:
+ tech = ACCESS_TECHNOLOGY_GSM;
+ break;
+ case 2:
+ tech = ACCESS_TECHNOLOGY_GSM_EGPRS;
+ break;
+ case 3:
+ tech = ACCESS_TECHNOLOGY_UTRAN;
+ break;
case 4:
- tech = 5;
+ tech = ACCESS_TECHNOLOGY_UTRAN_HSDPA;
break;
case 5:
- tech = 4;
+ tech = ACCESS_TECHNOLOGY_UTRAN_HSUPA;
+ break;
+ case 6:
+ tech = ACCESS_TECHNOLOGY_UTRAN_HSDPA_HSUPA;
+ break;
+ case 7:
+ tech = ACCESS_TECHNOLOGY_EUTRAN;
break;
case 8:
- tech = 1;
+ tech = ACCESS_TECHNOLOGY_GSM;
break;
case 9:
- tech = 2;
+ tech = ACCESS_TECHNOLOGY_GSM_EGPRS;
break;
default:
- tech = state;
+ /* Not registered for PS or something unknown, trust CREG... */
+ tech = tq->tech;
}
error:
@@ -270,12 +291,13 @@ static void creg_notify(GAtResult *result, gpointer user_data)
if (status != 1 && status != 5)
goto notify;
- if (ublox_is_toby_l4(nd->model)) {
+ if (ublox_is_toby_l4(nd->model) || ublox_is_toby_l2(nd->model)) {
tq = g_new0(struct tech_query, 1);
tq->status = status;
tq->lac = lac;
tq->ci = ci;
+ tq->tech = tech;
tq->netreg = netreg;
if (g_at_chat_send(nd->at_data.chat, "AT+UREG?", ureg_prefix,
+1, looks good. I'm even inclined to say we should do this for all
u-Blox modems, but since I don't have access to any other models it's
probably best to just add them to this list one by one as they become
available for test.
/Jonas
4:45 p.m.
Hi Jonas
Den ons 25 sep. 2019 kl 06:05 skrev Jonas Bonn <jonas(a)norrbonn.se>:
For the TOBY-L4, RAT changes won't trigger a CREG URC... for whatever
reason, this command is so brittle that they needed to bypass it with a
UREG command. For the L2, they don't bother to document this, but it
appears to suffer from the same deficiencies as the L4. CGREG and
friends appear to suffer from the same ailment... they don't have a firm
grasp of the overall state so they return "unknown" as a fallback.
For CGREG unknown seems to be used when the modem is running LTE,
at least for L2, which kind of make sense I guess. Since cereg is what to be
used at that time.
But I also have a bad feeling about CREG and L2, most often CREG URC's are
sent,
but not always. And sometimes like here the CREG URC has the wrong tech.
I'm starting to think if we should also subscribe to UREG in netreg for
ublox,
but thats another commit ;-)
--Richard
11:56 a.m.
Hi Richard,
On 9/24/19 11:11 AM, richard.rojfors(a)gmail.com wrote:
From: Richard Röjfors <richard(a)puffinpack.se>
It seems like the CREG reporting from the L2 modems are quite
buggy. An example for a L210 where CREG reports UTRAN while
COPS and UREG reports LTE. A manual poll also indicates LTE.
I also found that the technology mapping was incorrect,
probably confused with enum packet_bearer.
A commented log showing where CREG is not trustable:
UREG indicates LTE
21:59:29 : < \r\n+UREG: 7\r\n
21:59:29 : < \r\n+CIEV: 9,2\r\n
21:59:29 : < \r\n+CGEV: NW MODIFY 1,0,0\r\n
21:59:31 : < \r\n+CIEV: 2,2\r\n
21:59:39 : < \r\n+CIEV: 2,3\r\n
21:59:44 : < \r\n+CIEV: 2,2\r\n
22:01:38 : < \r\n+CIEV: 2,3\r\n
22:01:43 : < \r\n+CIEV: 2,2\r\n
A CREG indicating UTRAN with HSDPA and HSUPA
22:29:39 : < \r\n+CREG: 5,"0000","00000000",6\r\n
22:29:39 : > AT\r
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS=3,2\r
22:29:39 : < \r\n+CIEV: 9,2\r\n
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS?\r
An immediate cops indicating LTE
22:29:39 : < \r\n+COPS: 0,2,"24007",7\r\n
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+CSQ\r
22:29:39 : < \r\n+CIEV: 2,4\r\n
22:29:39 : < \r\n+CSQ: 26,4\r\n
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+CGATT=1\r
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS=3,0\r
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+COPS?\r
Another cops also indicates LTE
22:29:39 : < \r\n+COPS: 0,0,"Tele2",7\r\n <- 7: LTE
22:29:39 : < \r\nOK\r\n
22:29:39 : > AT+CGREG?\r
CGREG indicates unknown -> normal on LTE
22:29:39 : < \r\n+CGREG: 2,4\r\n
22:29:39 : < \r\nOK\r\n
22:29:44 : < \r\n+CIEV: 9,2\r\n
22:29:46 : < \r\n+CIEV: 2,2\r\n
22:56:23 : < \r\n+CIEV: 2,3\r\n
22:56:28 : < \r\n+CIEV: 2,2\r\n
22:59:40 : < \r\n+CIEV: 2,4\r\n
Manual poll shows we are running LTE
at+creg?
+CREG: 2,5,"2AFC","01DB0206",7
OK
---
drivers/ubloxmodem/network-registration.c | 34 +++++++++++++++++++----
1 file changed, 28 insertions(+), 6 deletions(-)
Applied, thanks.
Regards,
-Denis
1088
days inactive
1090
days old
3 comments
4 participants
participants (4)
-
Denis Kenzior -
Jonas Bonn -
Richard Röjfors -
richard.rojfors＠gmail.com