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

[refactor] add shared datalink for all firmware #1522

Merged
merged 1 commit into from Jan 30, 2016

Conversation

kirkscheper
Copy link
Member

Trying to help the joining of fixedwing and rotorcraft. This way all shared datalink code and updates only have to be done once. Firmware specific code not yet shared can be kept separate in the original files.

Pull request also contains a start for drone to drone messaging but reading the messages is waiting for the dl macros in telemetry.

@flixr
Copy link
Member

flixr commented Jan 30, 2016

Nice :-)

Apart from the missing declarations it would probably make sense to rename the firmware specific datalink.c to rotorcraft_datalink.c and fixedwing_datalink.c...

@kirkscheper
Copy link
Member Author

ah yea, forgot to include the datalink.h.

Isn't it enough that the files are in their respective firmware folders? If not let me know and I will rename them.

@flixr
Copy link
Member

flixr commented Jan 30, 2016

IMHO it is cleaner if files that are meant to be compiled together have unique names...

[refactor] add shared datalink for all firmware

[refactor] add shared datalink for all firmware

[refactor] add shared datalink for all firmware
@kirkscheper
Copy link
Member Author

done

@flixr
Copy link
Member

flixr commented Jan 30, 2016

Thx!

@flixr flixr merged commit 10c4af0 into paparazzi:master Jan 30, 2016
@kirkscheper kirkscheper deleted the refactor_datalink branch January 30, 2016 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants