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

HWDEF: support for MicoAir743v2 #28936

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Minderring
Copy link
Contributor

Dear ardupilot developers,

We are MicoAir Tech, and this is our new flight controller MicoAir743v2. It has been well tested with ardupilot and performed admirably. More detailed information about MicoAir743v2 is available in micoair web. Please check it and add it to the supported hardware.

MicoAir743v2

Best regard!
Minderring
MicoAir Tech

@ArduPilot ArduPilot deleted a comment from Lalbabu2001 Dec 28, 2024
Copy link
Collaborator

@Hwurzburg Hwurzburg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you might consider adding SD card support in the bootloader to allow firmware upload via SD card

libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/defaults.parm Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/defaults.parm Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/hwdef.dat Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/hwdef.dat Outdated Show resolved Hide resolved
@Minderring Minderring force-pushed the hwdef_add_MicoAir743v2_to_ardupilot branch 2 times, most recently from fc3e85d to 2a4efcb Compare December 31, 2024 11:37
@Minderring
Copy link
Contributor Author

you might consider adding SD card support in the bootloader to allow firmware upload via SD card

Thank you for your suggestion, but I believe updating firmware from the SD card is not convenient, so I didn't include that option in the bootloader.

@Minderring
Copy link
Contributor Author

I have made the changes. Please review the updates. @Hwurzburg

@Minderring Minderring requested a review from Hwurzburg December 31, 2024 11:44
Copy link
Collaborator

@Hwurzburg Hwurzburg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

close...a few more changes

libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/hwdef.dat Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/hwdef.dat Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/MicoAir743v2/README.md Outdated Show resolved Hide resolved
@Hwurzburg
Copy link
Collaborator

@andyp1per ping

@Minderring Minderring force-pushed the hwdef_add_MicoAir743v2_to_ardupilot branch from 22fb620 to f6277a0 Compare January 2, 2025 04:04
@Minderring Minderring requested a review from Hwurzburg January 2, 2025 04:10
Copy link
Collaborator

@andyp1per andyp1per left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@peterbarker
Copy link
Contributor

Needs a rebase.... and I'm not even sure we need those patches in the manifest generator...

Minderring added 2 commits January 2, 2025 20:05
Update README.md: add bluetooth introduction to features

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: fix description about SERIAL8

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: fix description about RC

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: add description about "LED" pin

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: fix description about Loading Firmware

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: fix description about update firmware

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

remove defaults.parm and defined default params in hwdef file

Update README.md: fix description about osd

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

remove parameter define about serial4

Update README.md: add a section about BlueTooth
Update README.md: add bluetooth introduction to features

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: fix description about SERIAL8

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: fix description about RC

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: add description about "LED" pin

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: fix description about Loading Firmware

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

Update README.md: fix description about update firmware

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

remove defaults.parm and defined default params in hwdef file

Update README.md: fix description about osd

Co-authored-by: Henry Wurzburg <hwurzburg@yahoo.com>

remove parameter define about serial4

Update README.md: add a section about BlueTooth
@Minderring Minderring force-pushed the hwdef_add_MicoAir743v2_to_ardupilot branch from f6277a0 to 95d5cb8 Compare January 2, 2025 12:21
@Minderring
Copy link
Contributor Author

Minderring commented Jan 2, 2025

Needs a rebase.... and I'm not even sure we need those patches in the manifest generator...

Rebased. Thank you! @peterbarker

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants