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

AP_HAL_ChibiOS:modify hwdef for V3 of ReaperF745 #22436

Merged
merged 4 commits into from Dec 23, 2022

Conversation

Hwurzburg
Copy link
Collaborator

@Hwurzburg Hwurzburg commented Dec 18, 2022

now either rename to just ReaperF745 or split changes into a new hwdef based off the v2...I prefer the former
tested on V3 board....has the same funky 45 deg vehicle forward as the V2...

I sent Andy P a link to the build to test on his V2 to make sure it still works

@Hwurzburg Hwurzburg added WIP WikiNeeded needs wiki update labels Dec 18, 2022
@Hwurzburg
Copy link
Collaborator Author

renamed rather than create a separate V3 hwdef

@Hwurzburg
Copy link
Collaborator Author

space was down to 2K, free space by minimizing GPS and Airspeed backends

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.

Please try 16Mhz

@Hwurzburg
Copy link
Collaborator Author

Please try 16Mhz

@andyp1per we discussed in devcall and it was felt that trying this on a single board does not prove the board layout over production is guaranteed to support it, hence the use of a more conservative clock on most boards...those boards that do use it had OEMs that verified on multiple boards....and there was the uncertainty of real benefit...

have you had a chance to test on a v2 board?

@andyp1per
Copy link
Collaborator

andyp1per commented Dec 20, 2022

we discussed in devcall and it was felt that trying this on a single board does not prove the board layout over production is guaranteed to support it, hence the use of a more conservative clock on most boards...those boards that do use it had OEMs that verified on multiple boards....and there was the uncertainty of real benefi

The issue is that because of the clock tree the actual speed on the boards I have tested come out significantly lower that 8Mhz. Setting to 16Mhz usually ends up with a speed of about 12Mhz, which is conservative enough but much better than the 6Mhz or so that you get otherwise. This is a small board with short traces - as far as I am concerned picking 8Mhz is just as much finger in the air as anything else. However, the reduction in CPU duty cycle when you go with higher clock speeds is very significant.

I've not had a chance to try with v2 as yet.

@tridge
Copy link
Contributor

tridge commented Dec 20, 2022

However, the reduction in CPU duty cycle when you go with higher clock speeds is very significant.

there is no change in CPU load. When waiting for the SPI completion interrupt the thread is suspended, so no CPU usage. Note that if you use the disable-interrupts method to try to measure CPU then you will see higher CPU, but that is because the CPU is artificially locked to the thread.

@tridge tridge merged commit 6348fd5 into ArduPilot:master Dec 23, 2022
@Hwurzburg Hwurzburg deleted the reaperv3 branch December 23, 2022 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WikiNeeded needs wiki update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants