ArduPilot
added radian f4 and h7 hardware definitions
Added Aerium Radian F4/H7 HW definitions https://www.aerium.co.il/product-page/radian
Test coverage Full compilation HW flashing Sensor calibration and testing Interface Testing Manual and automatic test flights
needs readme....see https://ardupilot.org/dev/docs/readme_file.html#readme-file for requirements
@Hwurzburg, the F405 is the one listed on the website. Both modules connect to the same carrier board (which doesn't utilize all FCUs pins and ports) so it might be a bit confusing. How about I'll drop the H743 from this PR and open a new PR for it for better separation. Assuming you agree to focus on the F405, anything else you would like me to add / edit?
I have pushed fixes to the F405 README according to the comments you made on the H743. The full pinout documentation and pinout exists here: https://aerium.bitbucket.io/documentation/Product%20Manuals/Radian/ For better clarity, I have fixed to images to include the pinout as well.
Added PR for board IDs https://github.com/ArduPilot/ardupilot/pull/30114
@Hwurzburg, @tridge and @peterbarker I believe I've made the necessary changes based on your comments. It now compiles for the plane as well.
Please review it at your convenience.
still waiting on changes...Please do not rebase on master! its not necessary and will make reviewing any changes you do more difficult....rebasing is done automatically when we merge...and is necessary only if something added to master affects the hwdefs or CI tests, which rarely happens....thanks
still waiting on changes...Please do not rebase on master! its not necessary and will make reviewing any changes you do more difficult....rebasing is done automatically when we merge...and is necessary only if something added to master affects the hwdefs or CI tests, which rarely happens....thanks
Apologies for that. I've reviewed everything and marked it as resolved. If anything else is needed, please let me know with more details.
@yaadch I was going to just fix this for you (quicker than typing the corrections as comments) and reformat to our standards, but ran into a question:
- what is the internal voltage divider for the power modules, and is it ,or external, the default? I will finish it up and push once I know that answer
@Hwurzburg
Hello, The default configuration uses an external voltage sense. The internal voltage divider has a ratio of 1/19, meaning the sense voltage is 1/19th of the input voltage. It is designed to scale a maximum input voltage of 60V down to 3.3V.
I have fixed the readme issues, and removed a few lines in hwdef that are not needed....squashed and library split.... @andyp1per still needs to review... @yaadch you do not have to rebase this any more at this point (do not use your master branch for PRs in the future..use a local fork of it)
I discovered also that you did not assign a board id...I created a PR for it #30316 and rebuilt the bootloaders...also changed to hwdef to drop VisODOM and Optical flow features....an F405 with CAN does not have room for those on Plane...
so when the bd id got merged it was changed by the merger....so I rebuilt the bootloaders and updated the PR...you should test a build from it on your board loaded by DFU to be sure, then I will mark for Devcal and merging
@Hwurzburg we did open a PR for the board ID here: https://github.com/ArduPilot/ardupilot/pull/30114 if possible, we prefer to reserve 10 IDs for future upcoming products.
assigne aerium a block starting at 4010.....I rebuilt bootloaders and pushed..you still need to address the HWDEF.DAT issues @andyp1per raised...
Ping @aerium-systems - are you still wanting to get these board definitions merged into ArduPilot?
Ping @aerium-systems - are you still wanting to get these board definitions merged into ArduPilot?
@peterbarker The F4 has already become outdated before we managed to merge this. The H7 is the only operational version now and will require a separate PR. I suggest we close this one and open a new PR for the H7.