Closed Bug 850996 Opened 11 years ago Closed 10 years ago

B2G Emulator: Add LTE signal report function.

Categories

(Firefox OS Graveyard :: Emulator, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(b2g-v1.4 fixed)

RESOLVED FIXED
1.4 S2 (28feb)
Tracking Status
b2g-v1.4 --- fixed

People

(Reporter: kchang, Assigned: sku)

References

Details

Attachments

(1 file, 2 obsolete files)

Current emulator can only send the GSM signal. We should add LTE signal report function for future LTE project.
Component: DOM → Emulator
Product: Core → Boot2Gecko
Assignee: nobody → sku
Comment on attachment 8380515 [details] [review]
Bug 850996 - B2G Emulator: Add LTE signal report function.

1. Add LTE signal strength (ie: rxlev, rsrp and rssnr) on emulator.
2. Fix range check for rssi/ber at handleSignalStrength@android_modem.c

"gsm lte_signal <rxlev> <rsrp> <rssnr>" can be used to simulate the
LTE siganl strength on emulator with this patch.
Attachment #8380515 - Flags: review?(vyang)
Comment on attachment 8380515 [details] [review]
Bug 850996 - B2G Emulator: Add LTE signal report function.

Have commented on Github.
Attachment #8380515 - Flags: review?(vyang)
Test case is not available currently.
Reason:
  ril_worker.js will only handle rssi, and map it to 0 ~ 100. it did not handle rxlev/rsrp/rssnr. Hence, there will not take any effect if we trigger "gsm lte_signal <rxlev> <rsrp> <rssnr>".

I will file a new bug for supporting LTE signal on Gecko (plus test cases) later.


// Major Change:
1. Add LTE signal strength (ie: rxlev, rsrp and rssnr) on emulator.
2. Fix range check for rssi/ber at handleSignalStrength@android_modem.c.
3. print rssi/ber if no argument for "gsm signal" command,
   print rxlev/rsrp/rssnr if no argument for "gsm lte_signal" command.

Ex:
1. "gsm signal" will print the current rssi/ber on console.
2. "gsm lte_signal" will print the current rxlev/rsrp/rssnr on console.
3. "gsm lte_signal <rxlev> <rsrp> <rssnr>" will be used to set the
   LTE siganl strength on emulator, and report RIL_UNSOL_SIGNAL_STRENGTH
   to upper layer.
Attachment #8380515 - Attachment is obsolete: true
Attachment #8381932 - Attachment is patch: true
Attachment #8381932 - Attachment mime type: text/x-github-pull-request → text/plain
Attachment #8381932 - Flags: review?(vyang)
Attachment #8381932 - Flags: review?(vyang) → review+
Keywords: checkin-needed
Bug 977433 is created for supporting LTE signal strength in ril_worker.js (Gecko side).
Master: 456499c44d1ef39b602ea02e9ed460b6aab85b44
Status: NEW → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.4 S2 (28feb)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: