Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

driver: modem: sara-u2 error when connecting #22689

Closed
WilliamGFish opened this issue Feb 10, 2020 · 0 comments · Fixed by #22692
Closed

driver: modem: sara-u2 error when connecting #22689

WilliamGFish opened this issue Feb 10, 2020 · 0 comments · Fixed by #22692
Assignees
@mike-scott
Labels
bug The issue is a bug, or the PR is fixing a bug
Milestone
v2.2.0

Comments

@WilliamGFish
Copy link
Collaborator

Describe the bug
The U2 option of driver fails to connect giving a CME ERROR: 133 | Requested service option not subscribed

Turns out a typo which I think came from a cut-n-paste from an order version(ublox-sara-r4.c):

#if defined(CONFIG_MODEM_UBLOX_SARA_U2)
/* set the APN */
SETUP_CMD_NOHANDLE("AT+UPSD=0,1,""
CONFIG_MODEM_UBLOX_SARA_R4_MANUAL_MCCMNO """),

Should be:

#if defined(CONFIG_MODEM_UBLOX_SARA_U2)
/* set the APN */
SETUP_CMD_NOHANDLE("AT+UPSD=0,1,""
CONFIG_MODEM_UBLOX_SARA_R4_APN """),

To Reproduce
Steps to reproduce the behavior:
Compile with the sara-u2 option

Expected behavior
Connect to APN as specified

Impact
showstopper

Additional context
Once fix (highlighted above) is implemented all appears to work fine
@WilliamGFish WilliamGFish added the bug The issue is a bug, or the PR is fixing a bug label Feb 10, 2020
@WilliamGFish WilliamGFish mentioned this issue Feb 10, 2020
Merged
mike-scott added a commit to mike-scott/zephyr that referenced this issue Feb 10, 2020
@mike-scott
drivers: modem: sara-r4: fix APN setting for U2 modem
ba9234b 
usage of AT+UPSD command per UBX-17003787(R15) command reference is:
AT+UPSD=<profile_id>, <param_tag>, <param_val>

For AT+UPSD=0, 1, "<value>": we are setting value to MCCMNO, which
is technically incorrect.  <param_tag> of 1 means:

1: APN - <param_val> defines the APN text string, e.g.
"apn.provider.com"; the maximum length is 99. The factory-programmed
value is an empty string.

Let's use APN here instead.  This fixes a +CME ERROR: 113 when
connecting with U2 modem.

Fixes: zephyrproject-rtos#22689

Signed-off-by: Michael Scott <mike@foundries.io>
@mike-scott mike-scott mentioned this issue Feb 10, 2020
Merged
@mike-scott mike-scott self-assigned this Feb 10, 2020
@mike-scott mike-scott added this to the v2.2.0 milestone Feb 10, 2020
jukkar pushed a commit that referenced this issue Feb 10, 2020
@mike-scott @jukkar
drivers: modem: sara-r4: fix APN setting for U2 modem
e934825 
usage of AT+UPSD command per UBX-17003787(R15) command reference is:
AT+UPSD=<profile_id>, <param_tag>, <param_val>

For AT+UPSD=0, 1, "<value>": we are setting value to MCCMNO, which
is technically incorrect.  <param_tag> of 1 means:

1: APN - <param_val> defines the APN text string, e.g.
"apn.provider.com"; the maximum length is 99. The factory-programmed
value is an empty string.

Let's use APN here instead.  This fixes a +CME ERROR: 113 when
connecting with U2 modem.

Fixes: #22689

Signed-off-by: Michael Scott <mike@foundries.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mike-scott mike-scott

Labels
bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Milestone
v2.2.0
Development

Successfully merging a pull request may close this issue.

drivers: modem: sara-r4: fix APN setting for U2 modem mike-scott/zephyr
2 participants
@mike-scott @WilliamGFish