-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
Add new board PixFlamingo F7 #25714
Add new board PixFlamingo F7 #25714
Conversation
how to include this hw into 4.4.3 branches , Kindly help on this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs pinouts,bd photos,typical wriring diagram, where to buy in a readme file....see https://github.com/ArduPilot/ardupilot/blob/master/libraries/AP_HAL_ChibiOS/hwdef/KakuteH7-Wing/README.md for an example
also needs the bootloader files in the Tools/bootloaders directory
I have updated the changes .Kindly check this |
@tsubashchandar thanks, readme started but fairly incomplete...refer to example given..still needs board photo, where to buy,wiring/pinouts...also uart section needs dma capabilities, etc. |
6e478a0
to
a6954a0
Compare
I have rebased and fixedup the commit history. |
Added dev call tag so we won't forget to follow up. |
Thanks |
a6954a0
to
5f1c82d
Compare
@IamPete1 @Hwurzburg I have made the changes . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still needs DMA comments fixed...needs rebase, squash, library commits split into two commits wwith proper commit messages, see Tools/gittools/git-subsystems split utility
- SERIAL3 -> UART1 (GPS1) DMA Enabled | ||
- SERIAL4 -> EMPTY | ||
- SERIAL5 -> UART7 (GPS2) NODMA | ||
- SERIAL6 -> USART2 (User) DMA Enabled |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- SERIAL6 -> USART2 (User) DMA Enabled | |
- SERIAL6 -> USART2 (User) RX only is DMA Enabled |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
@Hwurzburg @IamPete1 I have fixed the review comments, only hwdef files I changed , other files need to be rebased , help me on that |
9553c53
to
333050d
Compare
Squashed and rebased. @tsubashchandar You should now pull this branch down to your local repo so any further changes are added on top. You should also prefix your commit messages with the subsystem they changing, in this case |
@Hwurzburg will check that the requested changes have been made |
Too late for 4.4. I have marked for 4.5. |
Added new board