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 reference_design.md #3428

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update reference_design.md #3428

wants to merge 3 commits into from

Conversation

abisop
Copy link

@abisop abisop commented Nov 4, 2024

these stm32h7 boards has 1mb ram, not 2mb

Copy link

No flaws found

Comment on lines 21 to 25
- FMUv5X: (Holybro Pixhawk 5X) \(STM32F7, 400 MHz, 512KB RAM, 2 MB flash\) ([standard](https://github.com/pixhawk/Pixhawk-Standards/blob/master/DS-011%20Pixhawk%20Autopilot%20v5X%20Standard.pdf))
- FMUv6X: (Holybro Pixhawk 6X) \(STM32H7, 480 MHz, 1 MB RAM, 2 MB flash\) and variant 6i \(i.MX RT1050, 600 MHz, 512 KB RAM, external flash\) ([standard](https://github.com/pixhawk/Pixhawk-Standards/blob/master/DS-012%20Pixhawk%20Autopilot%20v6X%20Standard.pdf))
- FMUv6c: (Holybro Pixhawk 6C) \(STM32H7, 480 MHz, 1 MB RAM, 2 MB flash\) ([standard](https://github.com/pixhawk/Pixhawk-Standards/blob/master/DS-018%20Pixhawk%20Autopilot%20v6C%20Standard.pdf))
- FMUv6u: (?REFBOARDNAME) \(STM32H7, 400 MHz, 1 MB RAM, 2 MB flash\) ([standard](https://github.com/pixhawk/Pixhawk-Standards/blob/master/DS-016%20Pixhawk%20Autopilot%20v6U%20Standard.pdf))
- FMUv6X-RT: (Holybro Pixhawk 6X-RT) \(NXP i.MX RT1176, 32 Bit Arm® Cortex®-M7, 1GHz 32 Bit Arm® Cortex®-M4, 400MHz secondary core, 2 MB RAM, 64 MB flash\) and variant 6i \(i.MX RT1050, 600 MHz, 512 KB RAM, external flash\) ([standard](https://github.com/pixhawk/Pixhawk-Standards/blob/master/DS-020%20Pixhawk%20Autopilot%20v6X-RT%20Standard.pdf))
Copy link
Collaborator

Choose a reason for hiding this comment

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

@mrpollo I have attempted to update this to match our docs.

  • Can you sanity check the reference board names - e.g. I have assumed that the Holybro variants in each case are the reference
  • What is 6U? I can't see a refboard for that? Is the processor info OK?

Copy link
Contributor

@mrpollo mrpollo left a comment

Choose a reason for hiding this comment

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

Thanks for helping keep this updated, I went ahead and marked a few critical changes that we need before merging. If you can help us get those I would truly appreciate it.

en/hardware/reference_design.md Outdated Show resolved Hide resolved
en/hardware/reference_design.md Outdated Show resolved Hide resolved
en/hardware/reference_design.md Outdated Show resolved Hide resolved
en/hardware/reference_design.md Outdated Show resolved Hide resolved
en/hardware/reference_design.md Outdated Show resolved Hide resolved
en/hardware/reference_design.md Show resolved Hide resolved
Copy link
Collaborator

@hamishwillee hamishwillee left a comment

Choose a reason for hiding this comment

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

Thanks @mrpollo - I merged all those fixes - much appreciated! @abisop Thanks for kicking this off.

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.

3 participants