Merge "Revert "Get rid of RIL version."" am: 6514905475 (c3990696) · Commits · e / os / android_frameworks_opt_telephony

src/java/com/android/internal/telephony/BaseCommands.java

+9 −4

Original line number	Diff line number	Diff line
		@@ -102,8 +102,8 @@ public abstract class BaseCommands implements CommandsInterface {
		protected int mCdmaSubscription;
		// Type of Phone, GSM or CDMA. Set by GsmCdmaPhone.
		protected int mPhoneType;
		// RIL connected
		protected boolean mRilConnected = false;
		// RIL Version
		protected int mRilVersion = -1;

		public BaseCommands(Context context) {
		mContext = context; // May be null (if so we won't log statistics)
		@@ -736,8 +736,8 @@ public abstract class BaseCommands implements CommandsInterface {
		public void registerForRilConnected(Handler h, int what, Object obj) {
		Registrant r = new Registrant (h, what, obj);
		mRilConnectedRegistrants.add(r);
		if (mRilConnected) {
		r.notifyRegistrant(new AsyncResult(null, null, null));
		if (mRilVersion != -1) {
		r.notifyRegistrant(new AsyncResult(null, new Integer(mRilVersion), null));
		}
		}

		@@ -836,6 +836,11 @@ public abstract class BaseCommands implements CommandsInterface {
		@Override
		public void testingEmergencyCall() {}

		@Override
		public int getRilVersion() {
		return mRilVersion;
		}

		public void setUiccSubscription(int slotId, int appIndex, int subId, int subStatus,
		Message response) {
		}

src/java/com/android/internal/telephony/CommandsInterface.java

+5 −0

Original line number	Diff line number	Diff line
		@@ -1886,6 +1886,11 @@ public interface CommandsInterface {
		*/
		void getHardwareConfig (Message result);

		/**
		* @return version of the ril.
		*/
		int getRilVersion();

		/**
		* Sets user selected subscription at Modem.
		*

src/java/com/android/internal/telephony/GsmCdmaPhone.java

+51 −20

Original line number	Diff line number	Diff line
		@@ -180,6 +180,7 @@ public class GsmCdmaPhone extends Phone {

		private boolean mResetModemOnRadioTechnologyChange = false;

		private int mRilVersion;
		private boolean mBroadcastEmergencyCallStateChanges = false;

		// Constructors
		@@ -254,6 +255,7 @@ public class GsmCdmaPhone extends Phone {
		mResetModemOnRadioTechnologyChange = SystemProperties.getBoolean(
		TelephonyProperties.PROPERTY_RESET_ON_RADIO_TECH_CHANGE, false);

		mCi.registerForRilConnected(this, EVENT_RIL_CONNECTED, null);
		mCi.registerForVoiceRadioTechChanged(this, EVENT_VOICE_RADIO_TECH_CHANGED, null);
		mContext.registerReceiver(mBroadcastReceiver, new IntentFilter(
		CarrierConfigManager.ACTION_CARRIER_CONFIG_CHANGED));
		@@ -2057,6 +2059,16 @@ public class GsmCdmaPhone extends Phone {
		handleRadioOn();
		break;

		case EVENT_RIL_CONNECTED:
		ar = (AsyncResult) msg.obj;
		if (ar.exception == null && ar.result != null) {
		mRilVersion = (Integer) ar.result;
		} else {
		logd("Unexpected exception on EVENT_RIL_CONNECTED");
		mRilVersion = -1;
		}
		break;

		case EVENT_VOICE_RADIO_TECH_CHANGED:
		case EVENT_REQUEST_VOICE_RADIO_TECH_DONE:
		String what = (msg.what == EVENT_VOICE_RADIO_TECH_CHANGED) ?
		@@ -3041,6 +3053,24 @@ public class GsmCdmaPhone extends Phone {
		}
		}

		if(mRilVersion == 6 && getLteOnCdmaMode() == PhoneConstants.LTE_ON_CDMA_TRUE) {
		/*
		* On v6 RIL, when LTE_ON_CDMA is TRUE, always create CDMALTEPhone
		* irrespective of the voice radio tech reported.
		*/
		if (getPhoneType() == PhoneConstants.PHONE_TYPE_CDMA) {
		logd("phoneObjectUpdater: LTE ON CDMA property is set. Use CDMA Phone" +
		" newVoiceRadioTech=" + newVoiceRadioTech +
		" mActivePhone=" + getPhoneName());
		return;
		} else {
		logd("phoneObjectUpdater: LTE ON CDMA property is set. Switch to CDMALTEPhone" +
		" newVoiceRadioTech=" + newVoiceRadioTech +
		" mActivePhone=" + getPhoneName());
		newVoiceRadioTech = ServiceState.RIL_RADIO_TECHNOLOGY_1xRTT;
		}
		} else {

		// If the device is shutting down, then there is no need to switch to the new phone
		// which might send unnecessary attach request to the modem.
		if (isShuttingDown()) {
		@@ -3063,6 +3093,7 @@ public class GsmCdmaPhone extends Phone {
		" doesn't match either CDMA or GSM - error! No phone change");
		return;
		}
		}

		if (newVoiceRadioTech == ServiceState.RIL_RADIO_TECHNOLOGY_UNKNOWN) {
		// We need some voice phone object to be active always, so never

src/java/com/android/internal/telephony/Phone.java

+1 −0

Original line number	Diff line number	Diff line
		@@ -186,6 +186,7 @@ public abstract class Phone extends Handler implements PhoneInternalInterface {
		private static final int EVENT_CHECK_FOR_NETWORK_AUTOMATIC = 38;
		protected static final int EVENT_VOICE_RADIO_TECH_CHANGED = 39;
		protected static final int EVENT_REQUEST_VOICE_RADIO_TECH_DONE = 40;
		protected static final int EVENT_RIL_CONNECTED = 41;
		protected static final int EVENT_UPDATE_PHONE_OBJECT = 42;
		protected static final int EVENT_CARRIER_CONFIG_CHANGED = 43;
		// Carrier's CDMA prefer mode setting

src/java/com/android/internal/telephony/RIL.java

+8 −2

Original line number	Diff line number	Diff line
		@@ -628,6 +628,9 @@ public final class RIL extends BaseCommands implements CommandsInterface {
		}} catch (Throwable tr) {
		Rlog.e(RILJ_LOG_TAG,"Uncaught exception", tr);
		}

		/* We're disconnected so we don't know the ril version */
		notifyRegistrantsRilConnectionChanged(-1);
		}
		}

		@@ -4353,11 +4356,14 @@ public final class RIL extends BaseCommands implements CommandsInterface {

		/**
		* Notify all registrants that the ril has connected or disconnected.
		*
		* @param rilVer is the version of the ril or -1 if disconnected.
		*/
		void notifyRegistrantsRilConnected() {
		void notifyRegistrantsRilConnectionChanged(int rilVer) {
		mRilVersion = rilVer;
		if (mRilConnectedRegistrants != null) {
		mRilConnectedRegistrants.notifyRegistrants(
		new AsyncResult(null, null, null));
		new AsyncResult(null, new Integer(rilVer), null));
		}
		}