Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pixel 5 returns wrong carrier frequency for L5 in GnssAntennaInfo #483

Closed
barbeau opened this issue Feb 10, 2021 · 3 comments
Closed

Pixel 5 returns wrong carrier frequency for L5 in GnssAntennaInfo #483

barbeau opened this issue Feb 10, 2021 · 3 comments
Milestone

Comments

@barbeau
Copy link
Owner

barbeau commented Feb 10, 2021

Describe the bug

tl;dr - the Pixel 5 is returning the wrong carrier frequency for L5 in GnssAntennaInfo. I've confirmed via GPSTest and GnssLogger apps and also via the Android Studio debugger.

I've opened an issue on the AOSP tracker here with more details:
https://issuetracker.google.com/issues/179800215

This isn't an issue with GPSTest but I'll keep this open until it's fixed, as Pixel 5s are going to be reporting the wrong antenna CF values to the GPSTest Device Database until this is fixed.

To Reproduce

  1. Clone https://github.com/barbeau/gpstest or download GnssLogger
  2. Follow the steps at https://github.com/barbeau/gpstest/blob/master/LOGGING.md#data-output---csv to log the GnssAntennaInfo to CSV or JSON

For the code that registers the GnssAntennaInfo.Listener for callback in GPSTest, see https://github.com/barbeau/gpstest/blob/master/GPSTest/src/main/java/com/android/gpstest/GpsTestActivity.java#L1042.

Expected behavior
One GnssAntennaInfo element should have a carrier frequency of 1176.45 for it's support of L5 and E5a signals

Observed behavior
GnssAntennaInfo doesn't have a carrier frequency for L5/E5a, and instead has a carrier frequency of 1227.6, which is for L2, which the device doesn't support

App, Device and Android version:

GPSTest v3.9.7 on Pixel 5 (Android 11, build RQA.210105.003)

@barbeau barbeau added this to the v3.9 milestone Feb 10, 2021
@barbeau
Copy link
Owner Author

barbeau commented Feb 15, 2021

Looking at the GPSTest Device Database:
https://bit.ly/gpstest-device-database

...this same issue affects the Pixel 4a 5G and the Galaxy S21 Ultra (Snapdragon variant).

@stale
Copy link

stale bot commented Jun 3, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 3, 2021
@barbeau barbeau removed the stale label Jun 3, 2021
@barbeau
Copy link
Owner Author

barbeau commented Jun 7, 2021

Looks like this is fixed on the Pixel 5 RQ3A.210605.005 update that rolled out today - hasGnssAntennaInfo() now reports false instead of true, and the GnssAntennaInfo data isn't populated.

So this is better than before, in that bad data isn't reported, but we still don't have valid antenna data.

I'm going to close this as the Pixel 5 is now fixed, but #520 will remain open for the systemic issue across OEMs.

@barbeau barbeau closed this as completed Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant