Jackeagle | 0cc335a | 2018-10-19 00:06:49 -0400 | [diff] [blame^] | 1 | From 372ab41c59413ba81891195d72f6ebde33eb9b08 Mon Sep 17 00:00:00 2001 |
Jon West | 013ef58 | 2018-08-21 20:45:26 -0400 | [diff] [blame] | 2 | From: Pierre-Hugues Husson <phh@phh.me> |
| 3 | Date: Mon, 6 Aug 2018 20:01:44 +0200 |
Jackeagle | 0cc335a | 2018-10-19 00:06:49 -0400 | [diff] [blame^] | 4 | Subject: [PATCH 16/26] Change SignalStrentgh to change behaviour based on |
Jon West | 013ef58 | 2018-08-21 20:45:26 -0400 | [diff] [blame] | 5 | property |
| 6 | |
| 7 | Change-Id: I940ed724047567ec5195ac93ea04574c3d92f70b |
| 8 | --- |
| 9 | .../java/android/telephony/SignalStrength.java | 39 +++++++++++++++------- |
| 10 | 1 file changed, 27 insertions(+), 12 deletions(-) |
| 11 | |
| 12 | diff --git a/telephony/java/android/telephony/SignalStrength.java b/telephony/java/android/telephony/SignalStrength.java |
| 13 | index 4e56396..ff906d3 100644 |
| 14 | --- a/telephony/java/android/telephony/SignalStrength.java |
| 15 | +++ b/telephony/java/android/telephony/SignalStrength.java |
| 16 | @@ -854,6 +854,7 @@ public class SignalStrength implements Parcelable { |
| 17 | * RSRQ = quality of signal dB = Number of Resource blocks*RSRP/RSSI |
| 18 | * SNR = gain = signal/noise ratio = -10log P1/P2 dB |
| 19 | */ |
| 20 | + String method = android.os.SystemProperties.get("persist.sys.signal.level", "default"); |
| 21 | int rssiIconLevel = SIGNAL_STRENGTH_NONE_OR_UNKNOWN, rsrpIconLevel = -1, snrIconLevel = -1; |
| 22 | |
| 23 | if (mLteRsrp > MAX_LTE_RSRP || mLteRsrp < MIN_LTE_RSRP) { |
| 24 | @@ -897,6 +898,32 @@ public class SignalStrength implements Parcelable { |
| 25 | + rsrpIconLevel + " snrIconLevel:" + snrIconLevel |
| 26 | + " lteRsrpBoost:" + mLteRsrpBoost); |
| 27 | |
| 28 | + /* Valid values are (0-63, 99) as defined in TS 36.331 */ |
| 29 | + // TODO the range here is probably supposed to be (0..31, 99). It's unclear if anyone relies |
| 30 | + // on the current incorrect range check, so this will be fixed in a future release with more |
| 31 | + // soak time |
| 32 | + if (mLteSignalStrength > 63) rssiIconLevel = SIGNAL_STRENGTH_NONE_OR_UNKNOWN; |
| 33 | + else if (mLteSignalStrength >= 12) rssiIconLevel = SIGNAL_STRENGTH_GREAT; |
| 34 | + else if (mLteSignalStrength >= 8) rssiIconLevel = SIGNAL_STRENGTH_GOOD; |
| 35 | + else if (mLteSignalStrength >= 5) rssiIconLevel = SIGNAL_STRENGTH_MODERATE; |
| 36 | + else if (mLteSignalStrength >= 0) rssiIconLevel = SIGNAL_STRENGTH_POOR; |
| 37 | + |
| 38 | + if (DBG) log("getLTELevel - rssi:" + mLteSignalStrength + " rssiIconLevel:" |
| 39 | + + rssiIconLevel); |
| 40 | + |
| 41 | + if("rsrp".equals(method)) { |
| 42 | + if(rsrpIconLevel == -1) rsrpIconLevel = SIGNAL_STRENGTH_NONE_OR_UNKNOWN; |
| 43 | + return rsrpIconLevel; |
| 44 | + } |
| 45 | + if("rssnr".equals(method)) { |
| 46 | + if(snrIconLevel == -1) snrIconLevel = SIGNAL_STRENGTH_NONE_OR_UNKNOWN; |
| 47 | + return snrIconLevel; |
| 48 | + } |
| 49 | + if("rssi".equals(method)) { |
| 50 | + if(rssiIconLevel == -1) rssiIconLevel = SIGNAL_STRENGTH_NONE_OR_UNKNOWN; |
| 51 | + return rssiIconLevel; |
| 52 | + } |
| 53 | + |
| 54 | /* Choose a measurement type to use for notification */ |
| 55 | if (snrIconLevel != -1 && rsrpIconLevel != -1) { |
| 56 | /* |
| 57 | @@ -911,18 +938,6 @@ public class SignalStrength implements Parcelable { |
| 58 | |
| 59 | if (rsrpIconLevel != -1) return rsrpIconLevel; |
| 60 | |
| 61 | - /* Valid values are (0-63, 99) as defined in TS 36.331 */ |
| 62 | - // TODO the range here is probably supposed to be (0..31, 99). It's unclear if anyone relies |
| 63 | - // on the current incorrect range check, so this will be fixed in a future release with more |
| 64 | - // soak time |
| 65 | - if (mLteSignalStrength > 63) rssiIconLevel = SIGNAL_STRENGTH_NONE_OR_UNKNOWN; |
| 66 | - else if (mLteSignalStrength >= 12) rssiIconLevel = SIGNAL_STRENGTH_GREAT; |
| 67 | - else if (mLteSignalStrength >= 8) rssiIconLevel = SIGNAL_STRENGTH_GOOD; |
| 68 | - else if (mLteSignalStrength >= 5) rssiIconLevel = SIGNAL_STRENGTH_MODERATE; |
| 69 | - else if (mLteSignalStrength >= 0) rssiIconLevel = SIGNAL_STRENGTH_POOR; |
| 70 | - |
| 71 | - if (DBG) log("getLteLevel - rssi:" + mLteSignalStrength + " rssiIconLevel:" |
| 72 | - + rssiIconLevel); |
| 73 | return rssiIconLevel; |
| 74 | |
| 75 | } |
| 76 | -- |
| 77 | 2.7.4 |
| 78 | |