Skip to content
Commit 95500f8a authored by Nathan Harold's avatar Nathan Harold
Browse files

Fix CDMA Range Checks for SignalStrength

-Allow zero as a valid value for CDMA ECIO. Zero
 is allowed for EVDO ECIO and is equally valid for
 CDMA. Making them consistent by allowing zero here.
-Set EVDO ECIO to -160 if unreported rather than
 setting it to -1. The "unreported" value is
 undocumented, and since -1 is well within the range
 of valid values, makes no sense. Since CDMA ECIO was
 setting an unreported value to a very low number,
 again making them the same.
-Allow 0 for EVDO SNR. This value has a range that is
 documented both in the RIL and in SignalStrength to
 include zero, but we were previously disallowing 0.
 Making the range check inclusive in line with the
 existing documentation, which was self-consistent.

Bug: 32364031
Test: runtest frameworks-telephony
Change-Id: Ie0ca5abb4998d1b0b5abdbff9d51f364fe6db858
parent 701421c6
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment