From: Renat Zaripov <r.r.zaripov(a)gmail.com&gt; SIMCOM SIM900 modem module not support AT+CNMA command --- drivers/atmodem/sms.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/drivers/atmodem/sms.c b/drivers/atmodem/sms.c index c6fb8c4..c31eb39 100644 --- a/drivers/atmodem/sms.c +++ b/drivers/atmodem/sms.c @@ -389,6 +389,7 @@ static void at_cds_notify(GAtResult *result, gpointer user_data) static void at_cmt_notify(GAtResult *result, gpointer user_data) { struct ofono_sms *sms = user_data; + struct sms_data *data = ofono_sms_get_data(sms); const char *hexpdu; long pdu_len; int tpdu_len; @@ -409,7 +410,8 @@ static void at_cmt_notify(GAtResult *result, gpointer user_data) decode_hex_own_buf(hexpdu, -1, &pdu_len, 0, pdu); ofono_sms_deliver_notify(sms, pdu, pdu_len, tpdu_len); - at_ack_delivery(sms); + if (data->vendor != OFONO_VENDOR_SIMCOM) + at_ack_delivery(sms); } static void at_cmgr_notify(GAtResult *result, gpointer user_data) -- 1.7.7.3

Hi Renat, On 02/27/2012 01:19 AM, r.r.zaripov(a)gmail.com wrote: I had to fix up a trailing whitespace error at the end of this line. It might be a good idea to configure your editor to show these, or use checkpatch.pl from the Linux kernel to check the patches before submission. Patch has been applied, thanks. Regards, -Denis