From 6bf44c5af57e08d7cc532e76a48b0bb6c9269928 Mon Sep 17 00:00:00 2001 From: Yong Cong Sin Date: Sun, 20 Feb 2022 17:19:58 +0800 Subject: [PATCH] drivers: modem: gsm: +CGATT shouldn't give sem_response & set error The +CGATT: is followed by 'OK' or 'ERROR', so its handler should not set error code and give sem_response. Signed-off-by: Yong Cong Sin --- drivers/modem/gsm_ppp.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/drivers/modem/gsm_ppp.c b/drivers/modem/gsm_ppp.c index a42da4980bb..0d330c0a868 100644 --- a/drivers/modem/gsm_ppp.c +++ b/drivers/modem/gsm_ppp.c @@ -449,17 +449,11 @@ static const struct setup_cmd setup_cmds[] = { MODEM_CMD_DEFINE(on_cmd_atcmdinfo_attached) { - int error = -EAGAIN; - /* Expected response is "+CGATT: 0|1" so simply look for '1' */ if (argc && atoi(argv[0]) == 1) { - error = 0; LOG_INF("Attached to packet service!"); } - modem_cmd_handler_set_error(data, error); - k_sem_give(&gsm.sem_response); - return 0; }