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

Add support for Mountain Eagle H743 #27044

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

Conversation

pegro66
Copy link
Contributor

@pegro66 pegro66 commented May 11, 2024

Reserve Board-ID for Mountain Eagle H743

board info on:
www.mountain-eagle-electronics.com

Copy link
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

This should be a single commit against board_types.txt, just adding the board type.

The commit should be titled board_types.txt: reservie ID for MountainEagleH743.

Let me know if you want me to do one of those for you.

@pegro66
Copy link
Contributor Author

pegro66 commented May 12, 2024

... maybe you do one for me please - I'm getting annoyed with GitHub-Desktop doing thins I do not want to include ...

//Peter

@peterbarker
Copy link
Contributor

... maybe you do one for me please - I'm getting annoyed with GitHub-Desktop doing thins I do not want to include ...

See #27046

Also see https://ardupilot.org/dev/docs/where-to-get-the-code.html and the documents linked from it ("rebasing" and "cleaning up the commits".

@pegro66
Copy link
Contributor Author

pegro66 commented May 12, 2024

@peterbarker Thanks - that helps !

Now - how to proceed to get the board into the wiki-page ?

I basically followed the process described here :
https://ardupilot.org/dev/docs/porting.html

//Peter

@MATEKSYS
Copy link
Contributor

looks like most of definitions are copied from MatekH743, even the APJ_BOARD_ID 1013 is copied. :-)

@pegro66
Copy link
Contributor Author

pegro66 commented May 13, 2024

... the MatekH743 was the basis for the hwdef - as proposed on that web page ...

Meanwhile I did some more changes to fit all the pins and buses plus I have the schematics in KiCAD to add it as an open source board

I also updated the board iD to the assigned "1444"

How can I send that package of files ?

//Peter

@pegro66
Copy link
Contributor Author

pegro66 commented May 13, 2024

@MATEKSYS I have a MatekH743 in my T1-Ranger and it works very well there (and I had some experience with its configuration). So thanks for your development.
For the MTX640 it was too big - so I was looking for something with a smaller footprint - here we go ...
//Peter

@pegro66
Copy link
Contributor Author

pegro66 commented May 13, 2024

@peterbarker ... the MatekH743 was the basis for the hwdef - as proposed on that web page ...

Meanwhile I did some more changes to fit all the pins and buses plus I have the schematics in KiCAD to add it as an open source board

I also updated the board iD to the assigned "1444"

How can I send that package of files ?

//Peter

@peterbarker
Copy link
Contributor

How can I send that package of files ?

@pegro66 not sure what you mean by "package of files". If you are referring to a KiCAD file and supporting files, a PR against https://github.com/ArduPilot/Schematics would be welcome!

@peterbarker
Copy link
Contributor

@pegro66 the commit list in here needs a good cleanup. That's rebasing and squashing commits, and putting them into a one-commit-per-directory format.

@Hwurzburg put together a nice Wiki page on how to do that here: https://ardupilot.org/dev/docs/git-interactive-rebase.html

... but I can do that for you if you would like.

@peterbarker peterbarker changed the title Reserving new board number for Mountain eagle h743 Add support for Mountain Eagle H743 Jul 25, 2024
@Hwurzburg
Copy link
Collaborator

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.

@Hwurzburg Hwurzburg added the WIP label Sep 10, 2024
@peterbarker
Copy link
Contributor

Ping @pegro66 - are you still interested in getting this work merged?

@pegro66
Copy link
Contributor Author

pegro66 commented Sep 16, 2024 via email

@peterbarker peterbarker self-assigned this Sep 25, 2024
@peterbarker
Copy link
Contributor

Hello Peter, absolutely - what do I have to do ? //Peter On 11. Sep 2024, at 09:01, Peter Barker @.***> wrote: Ping

Henry has asked you to supply a README.md and some images so he can populate the Wiki.

He has pointed you at a board which has the desired information already, so "do what it does".

Also note my request for a clean commit list; we have some notes on using git to do that here: https://ardupilot.org/dev/docs/git-interactive-rebase.html - but if you are going to struggle with that Henry or myself could clean that up for you.

If the board design is done you could also consider creating that PR against the schematics repo.

@pegro66
Copy link
Contributor Author

pegro66 commented Oct 5, 2024

Hello Peter,
I'm a bit slow and totally unfamiliar with git - I have used ClearCase for some 30 years and I'm struggling with this git ...
Anyway - I have created a repository in my GitHub containing all the files that are necessary:

https://github.com/pegro66/Mountain-Eagle-H743.git

If you can give me a pointer on how to "cleanup" the mess - I'm happy to give it a try (but it will be slow as I only have access to git in the early morning of a WE using a VPN as I'm currently working in China) - otherwise I would be happy if you can lend me a helping hand :)

Best regards
Peter

@Hwurzburg
Copy link
Collaborator

@pegro66 Peter, I had some time, so I transferred your files from your private repo to this PR, fixed an LED error that prevented compile, added the bootloader files, squashed, library split and pushed back to this PR...I haven't reviewed the README vs dat files yet, but I should be able to this week...

has this been tested?
can you provide the English translation for the Chinese notes in the image? I can fix those when this gets merged and I make a wiki page for it..

@pegro66
Copy link
Contributor Author

pegro66 commented Oct 16, 2024 via email

# LED
# LED0-orange
# LED1-green
PE9 LED0 OUTPUT LOW GPIO(90) # orange
Copy link
Collaborator

Choose a reason for hiding this comment

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

in addition to get it to compile, you will need to add this new define to get the bd leds to work```suggestion
define AP_NOTIFY_GPIO_LED_2_ENABLED 1
PE9 LED0 OUTPUT LOW GPIO(90) # orange

@pegro66
Copy link
Contributor Author

pegro66 commented Nov 23, 2024 via email

@Hwurzburg Hwurzburg self-requested a review December 1, 2024 23:20
@Hwurzburg
Copy link
Collaborator

@pegro66 are you going to push those changes to this PR?

@pegro66
Copy link
Contributor Author

pegro66 commented Dec 17, 2024 via email

@peterbarker
Copy link
Contributor

… absolutely … the question is how ? On 3. Dec 2024, at 00:06, Henry Wurzburg @.***> wrote: @pegro66 <https://

You need to force-push your PR branch. See https://ardupilot.org/dev/docs/git-rebase.html

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.

4 participants