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.

Sign up for GitHub

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

Jump to bottom

Update Piksi driver #1349

Merged
merged 13 commits into from
Sep 13, 2015
Merged

Update Piksi driver #1349

merged 13 commits into from
Sep 13, 2015

Conversation

flixr
Copy link
Member

@flixr flixr commented Sep 13, 2015

Improved version of #1271 with additionally (split up the commits from @fvantienen on his piksi branch):

  • add DGPS and RTK fix modes to GPS
  • fix uart write on linux
  • add GPS_INJECT message
  • add sbp2ivy

gautierhattenberger and others added 13 commits September 12, 2015 21:36
@fvantienen
Copy link
Member

Good to merge for me :)

flixr added a commit that referenced this pull request Sep 13, 2015
@flixr
Merge pull request #1349 from paparazzi/piksi
f5f30ae 
Update Piksi driver
- better handling of the the baseline messages
- use the libsbp as a submodule
- compute the base "position" from the global position and the local baseline
- send the `BASE_POS` message to module so he can produce pseudo-global DGPS position
- send the antenna frontend mode via sbp message
- add DGPS and RTK fix modes to GPS
- fix uart write on linux
- add GPS_INJECT message
- add sbp2ivy
@flixr flixr merged commit f5f30ae into master Sep 13, 2015
@flixr flixr deleted the piksi branch September 13, 2015 08:43
@flixr flixr mentioned this pull request Sep 13, 2015
Closed
@simmis
Copy link

simmis commented Sep 14, 2015

@fvantienen, was that merged with all required commits included? I am seeing compilation errors when I make clean && make:
firmwares/rotorcraft/datalink.c: In function 'dl_parse_msg': firmwares/rotorcraft/datalink.c:169:10: error: 'DL_GPS_INJECT' undeclared (first use in this function) case DL_GPS_INJECT :

Or did I miss to update some other configuration?

@flixr
Copy link
Member Author

flixr commented Sep 14, 2015

@simmis did you really do a make clean && make?
DL_GPS_INJECT should be generated with make from the added GPS_INJECT datalink message.

@simmis
Copy link

simmis commented Sep 14, 2015

@flixr, yes I used the "clean" and "make" buttons.

I just also tried to make in the paparazzi root directory. But now I got another error (sth. with luftboot.c) that I need to investigate tomorrow...

@flixr
Copy link
Member Author

flixr commented Sep 14, 2015

@simmis you need to run make in the paparazzi root dir in order to update the generated code for messages and the ground segment...
The clean and make button in the Paparazzi Center only recompile the aircraft firmware...

@simmis
Copy link

simmis commented Sep 14, 2015

@flixr, ok, thanks. Got it. Will take a look into the other error tomorrow. Probably also sth. trivial.

@flixr flixr mentioned this pull request Oct 14, 2015
Closed
@flixr flixr added this to the v5.8 milestone Nov 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Airborne Ground Segment
Projects
None yet
Milestone
v5.8
Development

Successfully merging this pull request may close these issues.
4 participants
@flixr @fvantienen @simmis @gautierhattenberger