Remove unneeded new RIL command.
Jake Hamby [Thu, 6 Feb 2014 22:47:14 +0000 (14:47 -0800)]
Remove the recently added RIL_REQUEST_SET_RADIO_MODE command and update
the definition of the RIL_REQUEST_NV_RESET_CONFIG parameter.
Also, space out the NV item enum values so we can add new items per category.

Bug: 12864208
Change-Id: I1201c92198a8cd7e169efe7055b58259d975309b

src/java/com/android/internal/telephony/CommandsInterface.java
src/java/com/android/internal/telephony/Phone.java
src/java/com/android/internal/telephony/PhoneBase.java
src/java/com/android/internal/telephony/PhoneProxy.java
src/java/com/android/internal/telephony/RIL.java
src/java/com/android/internal/telephony/RadioNVItems.java
src/java/com/android/internal/telephony/sip/SipCommandInterface.java
src/java/com/android/internal/telephony/test/SimulatedCommands.java
tests/telephonytests/src/com/android/internal/telephony/gsm/UsimDataDownloadCommands.java

index 91551e0..7577ccf 100644 (file)
@@ -1782,25 +1782,16 @@ public interface CommandsInterface {
     void nvWriteCdmaPrl(byte[] preferredRoamingList, Message response);
 
     /**
-     * Perform the specified type of NV config reset.
-     * Used for device configuration by some CDMA operators.
+     * Perform the specified type of NV config reset. The radio will be taken offline
+     * and the device must be rebooted after erasing the NV. Used for device
+     * configuration by some CDMA operators.
      *
-     * @param resetType type of reset to perform (1 == factory reset; 2 == NV-only reset)
+     * @param resetType reset type: 1: reload NV reset, 2: erase NV reset, 3: factory NV reset
      * @param response Callback message.
      */
     void nvResetConfig(int resetType, Message response);
 
     /**
-     * Change the radio to the specified mode.
-     * Used for device configuration by some CDMA operators.
-     *
-     * @param radioMode is 0 for offline mode, 1 for online mode, 2 for low-power mode,
-     *                  or 3 to reset the radio.
-     * @param response Callback message.
-     */
-    void setRadioMode(int radioMode, Message response);
-
-    /**
      * @return version of the ril.
      */
     int getRilVersion();
index aee30ba..2971771 100644 (file)
@@ -1753,21 +1753,12 @@ public interface Phone {
     void nvWriteCdmaPrl(byte[] preferredRoamingList, Message response);
 
     /**
-     * Perform the specified type of NV config reset.
-     * Used for device configuration by some CDMA operators.
+     * Perform the specified type of NV config reset. The radio will be taken offline
+     * and the device must be rebooted after erasing the NV. Used for device
+     * configuration by some CDMA operators.
      *
-     * @param resetType type of reset to perform (1 == factory reset; 2 == NV-only reset)
+     * @param resetType reset type: 1: reload NV reset, 2: erase NV reset, 3: factory NV reset
      * @param response Callback message.
      */
     void nvResetConfig(int resetType, Message response);
-
-    /**
-     * Change the radio to the specified mode.
-     * Used for device configuration by some CDMA operators.
-     *
-     * @param radioMode is 0 for offline mode, 1 for online mode, 2 for low-power mode,
-     *                  or 3 to reset the radio.
-     * @param response Callback message.
-     */
-    void setRadioMode(int radioMode, Message response);
 }
index 1c5e441..8869f34 100644 (file)
@@ -981,11 +981,6 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     @Override
-    public void setRadioMode(int radioMode, Message response) {
-        mCi.setRadioMode(radioMode, response);
-    }
-
-    @Override
     public void notifyDataActivity() {
         mNotifier.notifyDataActivity(this);
     }
index 9a900e8..f5739b5 100644 (file)
@@ -1193,11 +1193,6 @@ public class PhoneProxy extends Handler implements Phone {
     }
 
     @Override
-    public void setRadioMode(int radioMode, Message response) {
-        mActivePhone.setRadioMode(radioMode, response);
-    }
-
-    @Override
     public void dispose() {
         mCommandsInterface.unregisterForOn(this);
         mCommandsInterface.unregisterForVoiceRadioTechChanged(this);
index 1b36e9f..18646f2 100644 (file)
@@ -2432,7 +2432,6 @@ public final class RIL extends BaseCommands implements CommandsInterface {
             case RIL_REQUEST_NV_WRITE_ITEM: ret = responseVoid(p); break;
             case RIL_REQUEST_NV_WRITE_CDMA_PRL: ret = responseVoid(p); break;
             case RIL_REQUEST_NV_RESET_CONFIG: ret = responseVoid(p); break;
-            case RIL_REQUEST_SET_RADIO_MODE: ret = responseVoid(p); break;
             default:
                 throw new RuntimeException("Unrecognized solicited response: " + rr.mRequest);
             //break;
@@ -3746,7 +3745,6 @@ public final class RIL extends BaseCommands implements CommandsInterface {
             case RIL_REQUEST_NV_WRITE_ITEM: return "RIL_REQUEST_NV_WRITE_ITEM";
             case RIL_REQUEST_NV_WRITE_CDMA_PRL: return "RIL_REQUEST_NV_WRITE_CDMA_PRL";
             case RIL_REQUEST_NV_RESET_CONFIG: return "RIL_REQUEST_NV_RESET_CONFIG";
-            case RIL_REQUEST_SET_RADIO_MODE: return "RIL_REQUEST_SET_RADIO_MODE";
             default: return "<unknown request>";
         }
     }
@@ -4241,17 +4239,4 @@ public final class RIL extends BaseCommands implements CommandsInterface {
 
         send(rr);
     }
-
-    @Override
-    public void setRadioMode(int radioMode, Message response) {
-        RILRequest rr = RILRequest.obtain(RIL_REQUEST_SET_RADIO_MODE, response);
-
-        rr.mParcel.writeInt(1);
-        rr.mParcel.writeInt(radioMode);
-
-        if (RILJ_LOGD) riljLog(rr.serialString() + "> " + requestToString(rr.mRequest)
-                + ' ' + radioMode);
-
-        send(rr);
-    }
 }
index b71aeca..fcdbfbc 100644 (file)
@@ -25,57 +25,56 @@ package com.android.internal.telephony;
  */
 public interface RadioNVItems {
 
-    // CDMA radio information
-    int RIL_NV_CDMA_MEID = 1;                   // hex MEID
-    int RIL_NV_CDMA_PRL_VERSION = 2;            // CDMA PRL version
-
-    // CDMA mobile account information
+    // CDMA radio and account information (items 1-10)
+    int RIL_NV_CDMA_MEID = 1;                   // CDMA MEID (hex)
+    int RIL_NV_CDMA_MIN = 2;                    // CDMA MIN (MSID)
     int RIL_NV_CDMA_MDN = 3;                    // CDMA MDN
-    int RIL_NV_CDMA_MIN = 4;                    // CDMA MIN (MSID)
+    int RIL_NV_CDMA_ACCOLC = 4;                 // CDMA access overload control
 
-    // Carrier device provisioning
-    int RIL_NV_DEVICE_MSL = 5;                  // device MSL
-    int RIL_NV_RTN_RECONDITIONED_STATUS = 6;    // RTN reconditioned status
-    int RIL_NV_RTN_ACTIVATION_DATE = 7;         // RTN activation date
-    int RIL_NV_RTN_LIFE_TIMER = 8;              // RTN life timer
-    int RIL_NV_RTN_LIFE_CALLS = 9;              // RTN life calls
-    int RIL_NV_RTN_LIFE_DATA_TX = 10;           // RTN life data TX
-    int RIL_NV_RTN_LIFE_DATA_RX = 11;           // RTN life data RX
-    int RIL_NV_OMADM_HFA_LEVEL = 12;            // HFA in progress
+    // Carrier device provisioning (items 11-30)
+    int RIL_NV_DEVICE_MSL = 11;                 // device MSL
+    int RIL_NV_RTN_RECONDITIONED_STATUS = 12;   // RTN reconditioned status
+    int RIL_NV_RTN_ACTIVATION_DATE = 13;        // RTN activation date
+    int RIL_NV_RTN_LIFE_TIMER = 14;             // RTN life timer
+    int RIL_NV_RTN_LIFE_CALLS = 15;             // RTN life calls
+    int RIL_NV_RTN_LIFE_DATA_TX = 16;           // RTN life data TX
+    int RIL_NV_RTN_LIFE_DATA_RX = 17;           // RTN life data RX
+    int RIL_NV_OMADM_HFA_LEVEL = 18;            // HFA in progress
 
-    // Mobile IP profile information
-    int RIL_NV_MIP_PROFILE_NAI = 13;            // NAI realm
-    int RIL_NV_MIP_PROFILE_HOME_ADDRESS = 14;   // MIP home address
-    int RIL_NV_MIP_PROFILE_AAA_AUTH = 15;       // AAA auth
-    int RIL_NV_MIP_PROFILE_HA_AUTH = 16;        // HA auth
-    int RIL_NV_MIP_PROFILE_PRI_HA_ADDR = 17;    // primary HA address
-    int RIL_NV_MIP_PROFILE_SEC_HA_ADDR = 18;    // secondary HA address
-    int RIL_NV_MIP_PROFILE_REV_TUN_PREF = 19;   // reverse TUN pref
-    int RIL_NV_MIP_PROFILE_HA_SPI = 20;         // HA SPI
-    int RIL_NV_MIP_PROFILE_AAA_SPI = 21;        // AAA SPI
-    int RIL_NV_MIP_PROFILE_MN_HA_SS = 22;       // HA shared secret
-    int RIL_NV_MIP_PROFILE_MN_AAA_SS = 23;      // AAA shared secret
+    // Mobile IP profile information (items 31-50)
+    int RIL_NV_MIP_PROFILE_NAI = 31;            // NAI realm
+    int RIL_NV_MIP_PROFILE_HOME_ADDRESS = 32;   // MIP home address
+    int RIL_NV_MIP_PROFILE_AAA_AUTH = 33;       // AAA auth
+    int RIL_NV_MIP_PROFILE_HA_AUTH = 34;        // HA auth
+    int RIL_NV_MIP_PROFILE_PRI_HA_ADDR = 35;    // primary HA address
+    int RIL_NV_MIP_PROFILE_SEC_HA_ADDR = 36;    // secondary HA address
+    int RIL_NV_MIP_PROFILE_REV_TUN_PREF = 37;   // reverse TUN pref
+    int RIL_NV_MIP_PROFILE_HA_SPI = 38;         // HA SPI
+    int RIL_NV_MIP_PROFILE_AAA_SPI = 39;        // AAA SPI
+    int RIL_NV_MIP_PROFILE_MN_HA_SS = 40;       // HA shared secret
+    int RIL_NV_MIP_PROFILE_MN_AAA_SS = 41;      // AAA shared secret
 
-    // CDMA network and band config
-    int RIL_NV_CDMA_BC10 = 24;                  // CDMA band class 10
-    int RIL_NV_CDMA_BC14 = 25;                  // CDMA band class 14
-    int RIL_NV_CDMA_SO68 = 26;                  // CDMA SO68
-    int RIL_NV_CDMA_SO73_COP0 = 27;             // CDMA SO73 COP0
-    int RIL_NV_CDMA_SO73_COP1TO7 = 28;          // CDMA SO73 COP1-7
-    int RIL_NV_CDMA_1X_ADVANCED_ENABLED = 29;   // CDMA 1X Advanced enabled
-    int RIL_NV_CDMA_EHRPD_ENABLED = 30;         // CDMA eHRPD enabled
-    int RIL_NV_CDMA_EHRPD_FORCED = 31;          // CDMA eHRPD forced
+    // CDMA network and band config (items 51-70)
+    int RIL_NV_CDMA_PRL_VERSION = 51;           // CDMA PRL version
+    int RIL_NV_CDMA_BC10 = 52;                  // CDMA band class 10
+    int RIL_NV_CDMA_BC14 = 53;                  // CDMA band class 14
+    int RIL_NV_CDMA_SO68 = 54;                  // CDMA SO68
+    int RIL_NV_CDMA_SO73_COP0 = 55;             // CDMA SO73 COP0
+    int RIL_NV_CDMA_SO73_COP1TO7 = 56;          // CDMA SO73 COP1-7
+    int RIL_NV_CDMA_1X_ADVANCED_ENABLED = 57;   // CDMA 1X Advanced enabled
+    int RIL_NV_CDMA_EHRPD_ENABLED = 58;         // CDMA eHRPD enabled
+    int RIL_NV_CDMA_EHRPD_FORCED = 59;          // CDMA eHRPD forced
 
-    // LTE network and band config
-    int RIL_NV_LTE_BAND_ENABLE_25 = 32;         // LTE band 25 enable
-    int RIL_NV_LTE_BAND_ENABLE_26 = 33;         // LTE band 26 enable
-    int RIL_NV_LTE_BAND_ENABLE_41 = 34;         // LTE band 41 enable
+    // LTE network and band config (items 71-90)
+    int RIL_NV_LTE_BAND_ENABLE_25 = 71;         // LTE band 25 enable
+    int RIL_NV_LTE_BAND_ENABLE_26 = 72;         // LTE band 26 enable
+    int RIL_NV_LTE_BAND_ENABLE_41 = 73;         // LTE band 41 enable
 
-    int RIL_NV_LTE_SCAN_PRIORITY_25 = 35;       // LTE band 25 scan priority
-    int RIL_NV_LTE_SCAN_PRIORITY_26 = 36;       // LTE band 26 scan priority
-    int RIL_NV_LTE_SCAN_PRIORITY_41 = 37;       // LTE band 41 scan priority
+    int RIL_NV_LTE_SCAN_PRIORITY_25 = 74;       // LTE band 25 scan priority
+    int RIL_NV_LTE_SCAN_PRIORITY_26 = 75;       // LTE band 26 scan priority
+    int RIL_NV_LTE_SCAN_PRIORITY_41 = 76;       // LTE band 41 scan priority
 
-    int RIL_NV_LTE_HIDDEN_BAND_PRIORITY_25 = 38;    // LTE hidden band 25 priority
-    int RIL_NV_LTE_HIDDEN_BAND_PRIORITY_26 = 39;    // LTE hidden band 26 priority
-    int RIL_NV_LTE_HIDDEN_BAND_PRIORITY_41 = 40;    // LTE hidden band 41 priority
+    int RIL_NV_LTE_HIDDEN_BAND_PRIORITY_25 = 77;    // LTE hidden band 25 priority
+    int RIL_NV_LTE_HIDDEN_BAND_PRIORITY_26 = 78;    // LTE hidden band 26 priority
+    int RIL_NV_LTE_HIDDEN_BAND_PRIORITY_41 = 79;    // LTE hidden band 41 priority
 }
index 9574825..07945c8 100644 (file)
@@ -589,8 +589,4 @@ class SipCommandInterface extends BaseCommands implements CommandsInterface {
     @Override
     public void nvResetConfig(int resetType, Message response) {
     }
-
-    @Override
-    public void setRadioMode(int radioMode, Message response) {
-    }
 }
index 3de32dc..d7f039c 100644 (file)
@@ -1719,9 +1719,4 @@ public final class SimulatedCommands extends BaseCommands
     public void nvResetConfig(int resetType, Message response) {
         unimplemented(response);
     }
-
-    @Override
-    public void setRadioMode(int radioMode, Message response) {
-        unimplemented(response);
-    }
 }
index 859a2ee..f179468 100644 (file)
@@ -683,8 +683,4 @@ class UsimDataDownloadCommands extends BaseCommands {
     @Override
     public void nvResetConfig(int resetType, Message response) {
     }
-
-    @Override
-    public void setRadioMode(int radioMode, Message response) {
-    }
 }