Skip to contentSkip to conversation

atmodem/sms: no mms support for Gemalto

PATCH In review Giacinto Cifelli 2 participants 2 messages Sep 27, 2018

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 atmodem/sms: no mms support for Gemalto

---
 drivers/atmodem/sms.c | 17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)
drivers/atmodem/sms.c +14 -3
index 64169086..db04cb1e 100644
@@ -220,9 +220,20 @@ static void at_cmgs(struct ofono_sms *sms, const unsigned char *pdu,
 	int len;
 
 	if (mms) {
-		snprintf(buf, sizeof(buf), "AT+CMMS=%d", mms);
-		g_at_chat_send(data->chat, buf, none_prefix,
-				NULL, NULL, NULL);
+
+		switch (data->vendor) {
+
+		case OFONO_VENDOR_GEMALTO:
+			/* no mms support */
+			break;
+
+		default:
+			snprintf(buf, sizeof(buf), "AT+CMMS=%d", mms);
+			g_at_chat_send(data->chat, buf, none_prefix,
+					NULL, NULL, NULL);
+			break;
+
+		}
 	}
 
 	len = snprintf(buf, sizeof(buf), "AT+CMGS=%d\r", tpdu_len);
Signature · 1 line
2.17.1
#1 lore raw

Hi Giacinto,

Quoted 4 lines · Giacinto Cifelli
---
  drivers/atmodem/sms.c | 17 ++++++++++++++---
  1 file changed, 14 insertions(+), 3 deletions(-)

Applied after tweaking the coding style.

Regards,
-Denis