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

GPS UART fix for RPi 3 #394

Merged
merged 3 commits into from Apr 22, 2016
Merged

GPS UART fix for RPi 3 #394

merged 3 commits into from Apr 22, 2016

Conversation

ghost
Copy link

@ghost ghost commented Apr 20, 2016

  • Resolves RPi3: GPS cannot connect via UART #393 by removing /dev/ttyS0 from GPS initialization sequence. On the RPi 3, this port does not connect to GPIO pins 8 and 10; receivers connected to these pins are therefore not detected.
  • Add a interactive script for configuring MTK3339 GPS receivers from the command line

@peepsnet
Copy link
Contributor

See comments in #393 .

@ghost
Copy link
Author

ghost commented Apr 20, 2016

@peepsnet -- I added the fix for GPIO UART to config.txt

@ghost ghost mentioned this pull request Apr 20, 2016
@peepsnet
Copy link
Contributor

great. Do we know if there are any ill effects with RPi2's and RPi0's???

@peepsnet
Copy link
Contributor

peepsnet commented Apr 20, 2016

I applied the changes in this PR(for the RY835AI) and compiled locally on my pi2 and pi3 and there were no ill effects.

This is what I did:
nano /boot/config.txt
add dtoverlay=pi3-miniuart-bt

and removed the /dev/ttyS0 reference from ry835ai.go

I do have my 835 GPS functioning now on my pi3!!!!!!!

@ghost
Copy link
Author

ghost commented Apr 21, 2016

@peepsnet -- thanks for the test report.

I see the same good results on my Pi2 (ry835ai over GPIO serial) with both an upgraded v0.8r2 Jessie image, and with an older Wheezy image that I've been using.

@cyoung cyoung merged commit 6e5d449 into cyoung:master Apr 22, 2016
@ghost ghost deleted the gps-uart-fix branch April 23, 2016 04:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants