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

Update MCU requirements #477

Merged
merged 3 commits into from
Oct 24, 2024
Merged

Conversation

haslinghuis
Copy link
Member

No description provided.

Copy link

netlify bot commented Oct 22, 2024

Deploy Preview for origin-betaflight-com ready!

Name Link
🔨 Latest commit c8f6c1f
🔍 Latest deploy log https://app.netlify.com/sites/origin-betaflight-com/deploys/671829f7be959100085f0e6a
😎 Deploy Preview https://deploy-preview-477.dev.web.betaflight.com
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

sonarcloud bot commented Oct 22, 2024

@ot0tot
Copy link
Contributor

ot0tot commented Oct 22, 2024

I agree this should be updated, and I have an opinion on what the default recommendations should be, but I hesitate to make them official without some testing/data to back them up.

For example, I'm pretty sure the AT32F435 is ok at 8k, but would say never for the F411 at 8k. For F411/F405, the recommendation also depends on whether timer based DShot is being used. If the FC design requires bitbang or burst DMA to work (resulting in no gyro DMA), then a lower loop rate is almost certainly required.

| | BMI-270 | 3.2 kHz | Disabled (not recommended) | 3.2 kHz | DShot 300 |
| MCU | IMU | Sampling Rate | Bidirectional DShot Status | PID Loop Rate | DShot Protocol |
| :---------------------------- | :--------------------------- | :------------ | :------------------------- | :------------ | :------------- |
| H7XX | MPU60X0, ICM2060X, ICM42688P | 8 kHz | Enabled or Disabled | 8 kHz | DShot 600 |
Copy link
Member

@nerdCopter nerdCopter Oct 24, 2024

Choose a reason for hiding this comment

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

while i understand this cleans up the tables, does it make searching harder? (X)

Copy link
Member Author

Choose a reason for hiding this comment

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

Other rows already used - so made it consistent.
But we need to decide on other factors like @ot0tot mentioned.

@haslinghuis haslinghuis merged commit 70af28c into betaflight:master Oct 24, 2024
9 checks passed
@haslinghuis haslinghuis deleted the mcu-requirements branch October 24, 2024 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants