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.

Sign up for GitHub

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

Jump to bottom

[H7] Initial cut on H7 MCU support #5827

Merged
merged 2 commits into from Dec 4, 2020
Merged

[H7] Initial cut on H7 MCU support #5827

merged 2 commits into from Dec 4, 2020

Conversation

digitalentity
Copy link
Member

@digitalentity digitalentity commented Jun 8, 2020 •
edited by DzikuVx

  • RCC
  • IO
  • TIMERS
  • DMA
  • ADC
  • EXTI
  • SPI
  • I2C
  • UART
  • USB VCP
  • USB MSC
  • SDIO
  • QSPI

@digitalentity digitalentity added this to the 2.6 milestone Jun 8, 2020
@digitalentity digitalentity marked this pull request as draft June 8, 2020 11:36
@digitalentity digitalentity mentioned this pull request Jun 8, 2020
Closed
@hydra
Copy link
Contributor

hydra commented Jun 9, 2020

Can you confirm what board have you been using for testing?

hydra
hydra reviewed Jun 9, 2020
View reviewed changes
Copy link
Contributor

@hydra hydra left a comment •
edited

Choose a reason for hiding this comment

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

just some basic review feedback that I noted when trying to compile for the SPRACINGH7EXTREME target that I'm working on. I'll create a PR with some fixes for the issues noted.

Stuff here fixed in #5834

make/mcu/STM32H7.mk Outdated Show resolved Hide resolved
make/mcu/STM32H7.mk Outdated Show resolved Hide resolved
make/mcu/STM32H7.mk Outdated Show resolved Hide resolved
make/mcu/STM32H7.mk Outdated Show resolved Hide resolved
src/main/target/link/stm32_flash_h743.ld Outdated Show resolved Hide resolved
@hydra
Copy link
Contributor

hydra commented Jul 30, 2020

@digitalentity What's the current status of the H7 work?

@digitalentity digitalentity force-pushed the de_h7_basic_support branch 2 times, most recently from cf7c6b0 to 6dc3ca0 Compare August 2, 2020 14:52
@digitalentity
Copy link
Member Author

Finally got this to compile properly.

@hydra
Copy link
Contributor

hydra commented Sep 12, 2020

@digitalentity Cool to see more progress on this!

This was referenced Sep 14, 2020
Closed
Closed
@OptimusTi OptimusTi mentioned this pull request Oct 19, 2020
Closed
@digitalentity
Copy link
Member Author

Rebased and squashed everything.

Tested correct booting on the following CPUs:

  • F427 - boots
  • F765 - doesn't boot
  • H743 - boots

@digitalentity
Copy link
Member Author

F427 - boots
F765 - boots
H743 - boots

@digitalentity
Copy link
Member Author

Currently focusing on making this non-breaking for existing targets in order to get this merged to avoid further merge conflicts.

@digitalentity digitalentity modified the milestones: 2.6, 2.7 Nov 14, 2020
@digitalentity
Copy link
Member Author

Once 2.6 is released, I'm going to merge this to avoid further rebasing and merge conflicts.

@digitalentity digitalentity marked this pull request as ready for review December 4, 2020 19:34
@digitalentity
Copy link
Member Author

Merging. Broken stuff and missing features will be addressed in future PRs

@digitalentity digitalentity merged commit 02aee7e into master Dec 4, 2020
@digitalentity digitalentity deleted the de_h7_basic_support branch December 4, 2020 19:35
@digitalentity digitalentity mentioned this pull request Dec 4, 2020
Closed
13 tasks
@digitalentity digitalentity added Release Notes Add this when a PR needs to be mentioned in the release notes and removed Don't merge labels Dec 5, 2020
@stronnag stronnag mentioned this pull request Dec 6, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hydra hydra hydra left review comments

Assignees
No one assigned
Labels
Release Notes Add this when a PR needs to be mentioned in the release notes
Projects
None yet
Milestone
3.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@digitalentity @hydra