To comply the Bug 912365, we need to support ICC refresh event on emulator.

Comment on attachment 804271 [details] [diff] [review] Bug 915977: patch for "hardware/ril" part Review of attachment 804271 [details] [diff] [review]: ----------------------------------------------------------------- ::: reference-ril/reference-ril.c @@ +3556,4 @@ > RIL_onUnsolicitedResponse(RIL_UNSOL_CDMA_PRL_CHANGED, &version, sizeof(version)); > } else if (strStartsWith(s, "+CFUN: 0")) { > setRadioState(RADIO_STATE_OFF); > + } else if (strStartsWith(s, "ICC REFRESH:")) { Shouldn't this come from +CUSATP? Is there really a AT COMMAND call "ICC REFRESH"? If there is, please point it out in a comment line.

Hi Vicamo: I will mark this bug as INVALID. Reasons are listed as below: 1. ril_worker should get SIM Refresh notification via STK proactive command (and UNSOLICITED_SIM_REFRESH callback as well). AOSP handle this refresh notification via UNSOLICITED_SIM_REFRESH in each record (eq: SIMRecord/RUIMRecord etc...) but ignore it in CAT service. 2. ril_worker already handle SIM Refresh notification via STK proactive command, but just need to make up SIM_RESET case (Bug 912365). 3. currently, "stk pdu d009810301010482028182" is enough for ril_worker.js to get the proper refresh notification. Due to above reasons, there is no need to implement extra mechansim for SIM Refresh. Please correct me if anything is wrong. Thanks!! sku