diff options
author | Mitchell Wills <mwills@google.com> | 2015-10-29 01:02:51 +0000 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2015-10-29 01:02:51 +0000 |
commit | 06f83655f769d9649cfd51fda485cd553dfca7cf (patch) | |
tree | 1e447b6b372a22ba925df0e97605e2b9d417842f | |
parent | 9f4c81f7018c3c8a5af1e0784b3c7d0a583ec95b (diff) | |
parent | 78ca9b3d6fb28abc6d52a31791f4b4084af76e65 (diff) | |
download | android_frameworks_opt_net_wifi-06f83655f769d9649cfd51fda485cd553dfca7cf.tar.gz android_frameworks_opt_net_wifi-06f83655f769d9649cfd51fda485cd553dfca7cf.tar.bz2 android_frameworks_opt_net_wifi-06f83655f769d9649cfd51fda485cd553dfca7cf.zip |
Don\'t set RSSI to -1 when no value is returned
am: 78ca9b3d6f
* commit '78ca9b3d6fb28abc6d52a31791f4b4084af76e65':
Don't set RSSI to -1 when no value is returned
-rw-r--r-- | service/java/com/android/server/wifi/WifiStateMachine.java | 17 |
1 files changed, 8 insertions, 9 deletions
diff --git a/service/java/com/android/server/wifi/WifiStateMachine.java b/service/java/com/android/server/wifi/WifiStateMachine.java index 09f3d939f..7001a9adf 100644 --- a/service/java/com/android/server/wifi/WifiStateMachine.java +++ b/service/java/com/android/server/wifi/WifiStateMachine.java @@ -4070,9 +4070,9 @@ public class WifiStateMachine extends StateMachine implements WifiNative.WifiPno * Fetch RSSI, linkspeed, and frequency on current connection */ private void fetchRssiLinkSpeedAndFrequencyNative() { - int newRssi = -1; - int newLinkSpeed = -1; - int newFrequency = -1; + Integer newRssi = null; + Integer newLinkSpeed = null; + Integer newFrequency = null; String signalPoll = mWifiNative.signalPoll(); @@ -4096,12 +4096,11 @@ public class WifiStateMachine extends StateMachine implements WifiNative.WifiPno } if (PDBG) { - logd("fetchRssiLinkSpeedAndFrequencyNative rssi=" - + Integer.toString(newRssi) + " linkspeed=" - + Integer.toString(newLinkSpeed)); + logd("fetchRssiLinkSpeedAndFrequencyNative rssi=" + newRssi + + " linkspeed=" + newLinkSpeed + " freq=" + newFrequency); } - if (newRssi > WifiInfo.INVALID_RSSI && newRssi < WifiInfo.MAX_RSSI) { + if (newRssi != null && newRssi > WifiInfo.INVALID_RSSI && newRssi < WifiInfo.MAX_RSSI) { // screen out invalid values /* some implementations avoid negative values by adding 256 * so we need to adjust for that here. @@ -4129,10 +4128,10 @@ public class WifiStateMachine extends StateMachine implements WifiNative.WifiPno updateCapabilities(getCurrentWifiConfiguration()); } - if (newLinkSpeed != -1) { + if (newLinkSpeed != null) { mWifiInfo.setLinkSpeed(newLinkSpeed); } - if (newFrequency > 0) { + if (newFrequency != null && newFrequency > 0) { if (ScanResult.is5GHz(newFrequency)) { mWifiConnectionStatistics.num5GhzConnected++; } |