hwdef: Add a new hwdef param for AEROFOX H7 flight control #28533
+308
−0
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
AEROFOXtech
commented
Nov 5, 2024
AEROFOXtech
commented
- Add a new flight control board hwdef for AEROFOX_H7
- Create a new board ID for AEROFOX_H7 flight control board
- Add AEROFOX_H7 bootloader to Tools
andyp1per
requested changes
Nov 5, 2024
| @@ -404,6 +404,9 @@ AP_HW_CUAV-7-NANO 7000 | |||
| # IDs 7100-7109 reserved for VIEWPRO | |||
| AP_HW_VUAV-V7pro 7100 | |||
|
|
|||
| # IDs 7200-7209 reserved for AEROFOX | |||
| AP_HW_AEROFOX_H7 7200 | |||
There was a problem hiding this comment.
This needs to be in a separate commit
There was a problem hiding this comment.
I'll submit to modify it in the next commit
| BATT_MONITOR 3 | ||
| CAN_P1_DRIVER 1 | ||
|
|
||
| SERIAL1_BAUD 115 |
There was a problem hiding this comment.
You should not set parameters that are already the default as it consumes flash space. Ideally put these in the hwdef
| OSCILLATOR_HZ 24000000 | ||
|
|
||
| # board ID for firmware load | ||
| APJ_BOARD_ID 140 |
| OSCILLATOR_HZ 24000000 | ||
|
|
||
| # board ID for firmware load | ||
| APJ_BOARD_ID 7200 |
| IMU Invensensev3 SPI:42688_2 ROTATION_PITCH_180_YAW_270 | ||
|
|
||
| SPIDEV 16470 SPI1 DEVID1 16470_CS MODE3 1*MHZ 2*MHZ | ||
| SPIDEV 42688_1 SPI1 DEVID2 42688_1_CS MODE3 2*MHZ 8*MHZ |
There was a problem hiding this comment.
Make these 16Mhz if you can
| SPIDEV 42688_2 SPI4 DEVID3 42688_2_CS MODE3 2*MHZ 8*MHZ | ||
|
|
||
| SPIDEV ramtron SPI2 DEVID1 FRAM_CS MODE3 8*MHZ 8*MHZ | ||
| SPIDEV sdcard SPI2 DEVID2 FLASH_CS MODE0 1*MHZ 8*MHZ |
There was a problem hiding this comment.
Ideally you would use SDMMC here
There was a problem hiding this comment.
Thank you for your advice, but we don't have enough GPIO ports on our hardware to use SDMMC
|
|
||
| # GPIO | ||
| PA0 TIM2_CH1 TIM2 PWM(1) GPIO(50) | ||
| PA1 TIM2_CH2 TIM2 PWM(2) GPIO(51) |
There was a problem hiding this comment.
Should make these bdshot, should be easy
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.