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

SP Racing H7 extreme updates #20385

Merged
merged 2 commits into from
Mar 28, 2022
Merged

Conversation

andyp1per
Copy link
Collaborator

@andyp1per andyp1per commented Mar 27, 2022

Found a few bugs in the hwdef and one build bug

This gives the following CPU load in stabilize:

ThreadsV2
ISR           PRI=255 sp=0x24069DF0 STACK=1236/1536 LOAD= 2.2%
ArduCopter    PRI=182 sp=0x2406A3F0 STACK=4096/7168 LOAD=51.0%
idle          PRI=  1 sp=0x300158D0 STACK= 144/ 352 LOAD= 8.7%
UART_RX       PRI= 60 sp=0x3003EFA0 STACK= 760/1104 LOAD= 0.7%
OTG1          PRI= 60 sp=0x3003E0A8 STACK= 248/ 656 LOAD= 0.2%
monitor       PRI=183 sp=0x300113F8 STACK= 480/ 848 LOAD= 0.3%
timer         PRI=181 sp=0x30012678 STACK=1448/1872 LOAD= 2.1%
rcout         PRI=181 sp=0x30011D38 STACK= 432/ 848 LOAD= 8.9%*
rcin          PRI=177 sp=0x30011798 STACK= 416/1360 LOAD= 9.8%*
io            PRI= 58 sp=0x30010A58 STACK=1496/2384 LOAD= 0.5%*
storage       PRI= 59 sp=0x300120D8 STACK= 792/1360 LOAD= 0.1%
UART2         PRI= 60 sp=0x3003D058 STACK= 384/ 656 LOAD= 0.1%
UART3         PRI= 60 sp=0x3003C560 STACK= 264/ 656 LOAD= 0.1%
UART4         PRI= 60 sp=0x3003BA68 STACK= 384/ 656 LOAD= 0.1%
I2C0          PRI=176 sp=0x3003A810 STACK= 872/1360 LOAD= 1.0%*
OSD           PRI= 59 sp=0x30036590 STACK= 912/1616 LOAD= 0.6%*
log_io        PRI= 59 sp=0x30036D90 STACK= 264/1656 LOAD= 0.3%
UART1         PRI= 60 sp=0x30038580 STACK= 248/ 656 LOAD= 0.2%
SPI3          PRI=181 sp=0x30020C98 STACK= 712/1360 LOAD=12.5%
FTP           PRI= 58 sp=0x24002F30 STACK=1464/2896 LOAD= 0.1%

and the following in loiter:

ThreadsV2
ISR           PRI=255 sp=0x24069DF0 STACK=1236/1536 LOAD= 2.1%
ArduCopter    PRI=182 sp=0x2406A3F0 STACK=4096/7168 LOAD=58.0%
idle          PRI=  1 sp=0x300158D0 STACK= 144/ 352 LOAD= 2.8%
UART_RX       PRI= 60 sp=0x3003EFA0 STACK= 760/1104 LOAD= 0.5%
OTG1          PRI= 60 sp=0x3003E0A8 STACK= 248/ 656 LOAD= 0.1%
monitor       PRI=183 sp=0x300113F8 STACK= 480/ 848 LOAD= 0.3%
timer         PRI=181 sp=0x30012678 STACK=1456/1872 LOAD= 2.1%
rcout         PRI=181 sp=0x30011D38 STACK= 432/ 848 LOAD= 8.9%*
rcin          PRI=177 sp=0x30011798 STACK= 416/1360 LOAD= 9.5%*
io            PRI= 58 sp=0x30010A58 STACK=1496/2384 LOAD= 0.3%*
storage       PRI= 59 sp=0x300120D8 STACK= 792/1360 LOAD= 0.1%
UART2         PRI= 60 sp=0x3003D058 STACK= 384/ 656 LOAD= 0.1%
UART3         PRI= 60 sp=0x3003C560 STACK= 264/ 656 LOAD= 0.1%
UART4         PRI= 60 sp=0x3003BA68 STACK= 384/ 656 LOAD= 0.1%
I2C0          PRI=176 sp=0x3003A810 STACK= 872/1360 LOAD= 0.9%
OSD           PRI= 59 sp=0x30036590 STACK= 912/1616 LOAD= 0.6%*
log_io        PRI= 59 sp=0x30036D90 STACK= 264/1656 LOAD= 0.2%
UART1         PRI= 60 sp=0x30038580 STACK= 248/ 656 LOAD= 0.1%
SPI3          PRI=181 sp=0x30020C98 STACK= 712/1360 LOAD=12.8%
FTP           PRI= 58 sp=0x24002F30 STACK=1464/2896 LOAD= 0.1%

You can see that neither setup has quite enough idle CPU to accommodate the extra 12% of the second IMU

@peterbarker peterbarker merged commit 2e52496 into ArduPilot:master Mar 28, 2022
@peterbarker
Copy link
Contributor

Merged,thanks!

@andyp1per andyp1per deleted the pr-spracing-config branch March 29, 2022 07:30
@rmackay9
Copy link
Contributor

I guess we should backport this to 4.2?

@andyp1per
Copy link
Collaborator Author

@rmackay9 yes please!

@rmackay9 rmackay9 added this to Pending in Copter 4.2 Mar 29, 2022
@tridge tridge added this to pending in Plane 4.2 Apr 5, 2022
@tridge tridge moved this from pending to merged in Plane 4.2 Apr 5, 2022
@rmackay9 rmackay9 moved this from Pending to 4.2.0-beta4 in Copter 4.2 Apr 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Copter 4.2
4.2.0-rc1
Development

Successfully merging this pull request may close these issues.

None yet

4 participants