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

[datalink] update bluegiga protocol and add rssi abi message #1507

Merged
merged 2 commits into from
Jan 11, 2016

Conversation

kirkscheper
Copy link
Member

Added abi message for rssi from bluegiga bluetooth module.
Simplified the bluetooth check and parse to pprz_check and parse.
Updated bluetooth the recent build of firmware (telemetry fully backwards compatible).
Update bluetooth ground code.

@flixr
Copy link
Member

flixr commented Jan 6, 2016

Hi Kirk,
Some questions:

  • What is the BLUEGIGA_RSSI message intended for? Doesn't seem to be used yet...
  • Why does it (need to) contain the ac_id?

Just wondering if it wouldn't make sense to make a more generic RSSI message...

@gautierhattenberger
Copy link
Member

I also think the RSSI message should be generic. It could be used with some xbee modules too

@kirkscheper
Copy link
Member Author

@flixr The rssi message should contain the signal strength and broadcast strength (if available) from the message received and an rssi was available. ac_id is useful for drone2drone coms that I and a student are working on. It is currently not used but will be shortly to help estimate the distance between a broadcaster and a receiver. This can of course also be used with the ground station (with ac_id 0).

I have no problem making that just RSSI...

@flixr
Copy link
Member

flixr commented Jan 11, 2016

Looks ok to me now, @gautierhattenberger if you are also happy now, feel free to merge it...

gautierhattenberger added a commit that referenced this pull request Jan 11, 2016
[datalink] update bluegiga protocol and add rssi abi message
@gautierhattenberger gautierhattenberger merged commit c000dd9 into paparazzi:master Jan 11, 2016
@kirkscheper kirkscheper deleted the bluegiga_update branch January 31, 2016 16:04
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.

None yet

3 participants