Ticket #1971 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

keep retrieving all SMS list even there is no message in SIM

Reported by: erin_yueh Owned by: zecke
Priority: normal Milestone: Om2008.9
Component: Qtopia Version: Om2008.8
Severity: normal Keywords: sms
Cc: Blocked By:
Blocking: Estimated Completion (week):
HasPatchForReview: PatchReviewResult:
Reproducible: always

Description

check the attached log file and you can find it tries to send 'AT+CMGL=4' too many times. If there is no SMS in SIM card, we needn't send this AT command to modem.

please review this patch, it can avoid to keep sending this AT command.

Attachments

log.sms_list.error​ (63.2 KB) - added by erin_yueh 5 years ago.
log CMS error 321
0001-bug-fixed-for-retrieving-empty-SMS-list-Erin-Yueh.patch​ (1.6 KB) - added by erin_yueh 5 years ago.
patch
log.original.sms​ (63.1 KB) - added by erin_yueh 5 years ago.
original sms

Change History

Changed 5 years ago by erin_yueh

log CMS error 321

Changed 5 years ago by erin_yueh

patch

comment:1 Changed 5 years ago by erin_yueh

  • Component changed from unknown to Qtopia

comment:2 Changed 5 years ago by erin_yueh

related to bug 1880

comment:3 Changed 5 years ago by zecke

Also check #1757 which is a duplicate of this.

Changed 5 years ago by erin_yueh

original sms

comment:5 Changed 5 years ago by zecke

  • Status changed from new to in_testing

Tomorrow's Qtopia build should have it. To be honest I would have preferred to check the error code...

comment:6 Changed 5 years ago by erin_yueh

for testing reproduce steps:
after reboot FR, use 'logread' to check below these AT commands.
if there is no sms stored in SIM card, then we won't retrieve sms list.

eg.
AT+CPMS=0,15,0,15,0,15 (1st, 3rd, 5th number are 0, means EMPTY)
if it's empty, we wont send 'AT+CMGL=4'
AT+CMGL=4 (this command is to retrieve all SMS from SIM card)

comment:7 Changed 5 years ago by zecke

  • Status changed from in_testing to closed
  • Keywords sms added; sms, hasPatch removed
  • Resolution set to fixed

Closing it.

Note: See TracTickets for help on using tickets.