Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

variant(wb55): add STeaMi board#2526

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
fpistm merged 4 commits intostm32duino:mainfromnedseb:main
Oct 28, 2024
Merged

variant(wb55): add STeaMi board#2526

fpistm merged 4 commits intostm32duino:mainfromnedseb:main
Oct 28, 2024

Conversation

nedseb
Copy link
Contributor

Add support to the STeaMi board developed for the project I-NOVMICRO with help from ST Rousset.

I tried to follow all the steps of the Wiki. Let me know if something is missing.

@fpistm
Copy link
Member

Hi@nedseb
Thanks for the PR.
Have you any documentations (User manual, schematics,...) helping to review the PR?

@fpistmfpistm added this to the2.9.0 milestoneSep 26, 2024
@fpistmfpistm added the new variantAdd support of new bard labelSep 26, 2024
@nedseb
Copy link
ContributorAuthor

nedseb commentedSep 26, 2024
edited
Loading

Hi@nedseb Thanks for the PR. Have you any documentations (User manual, schematics,...) helping to review the PR?

Hi@fpistm,
You can find the reference design of the boards here :https://github.com/steamicc/steami-reference-design
I hope it will help you to review this PR. I will extract the pinout into a more human-readable version soon.

fpistm reacted with thumbs up emoji

nedseb added a commit to nedseb/Arduino_Core_STM32 that referenced this pull requestSep 27, 2024
… 64 boards- Modified `platform.txt` to enable programming via OpenOCD, which was already used for debugging.- Added an example demonstrating how to program all Nucleo-64 boards, whether they use STLink or DAPLink.To avoid conflict, the support for STeaMi will be added after the merge of the PRstm32duino#2526*Benefits:*- Enables programming of STeaMi boards.- Supports P_NUCLEO boards that use DAPLink instead of STLink to leverage WebUSB functionality.
nedseb added a commit to nedseb/Arduino_Core_STM32 that referenced this pull requestSep 27, 2024
… 64 boards- Modified `platform.txt` to enable programming via OpenOCD, which was already used for debugging.- Added an example demonstrating how to program all Nucleo-64 boards, whether they use STLink or DAPLink.To avoid conflict, the support for STeaMi will be added after the merge of the PRstm32duino#2526*Benefits:*- Enables programming of STeaMi boards.- Supports P_NUCLEO boards that use DAPLink instead of STLink to leverage WebUSB functionality.
@nedseb
Copy link
ContributorAuthor

nedseb commentedOct 1, 2024
edited
Loading

I rebased this PR on top of the merge commit of PR#2528.

To be compatible with DAPLink, I need to replacetransport select hla_swd withtransport select swd (or at least remove it, because the default transport option on DAPLink is SWD).

To achieve this, I changed the default configuration of OpenOCD by modifying the value ofdebug.server.openocd.scripts.1. Previously, I used the target script to do this, but with the refactoring of the OpenOCD command, I broke (again) the CMake generation step of the CI.

I see 2 options, the first one is to modify the scriptupdate_boarddb.py to handle the configurationSTeaMi.menu.pnum.STEAM32_WB55RG.debug.server.openocd.scripts.1={runtime.platform.path}/debugger/select_swd.cfg or simply removethis line as the stlink interface configuration seems to don't need it anymore (I tested this option only with my P_NUCLEO).

What do you think about these options ?

Copy link
Member

@fpistmfpistm left a comment

Choose a reason for hiding this comment

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

First round.

Signed-off-by: Sébastien NEDJAR <sebastien.nedjar@univ-amu.fr>Co-authored-by: Frederic Pillon <frederic.pillon@st.com>
Copy link
Member

@fpistmfpistm left a comment

Choose a reason for hiding this comment

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

I've squashed and rebased on top of the main with some fix and formatting.
Please fetch your repo befopre any further change.

About the GPIO expander definition, I don't think it is a good idea to give them a number.

Finally, for pins not on a connector, you could add a custom peripheral pins to save space.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm
Copy link
Member

I've made a commit to fix the cmake issue.

nedseb reacted with thumbs up emoji

Copy link
Member

@fpistmfpistm left a comment

Choose a reason for hiding this comment

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

LGTM

@fpistmfpistm merged commitc35b8ef intostm32duino:mainOct 28, 2024
25 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@fpistmfpistmfpistm approved these changes

Assignees
No one assigned
Labels
new variantAdd support of new bard
Projects
Milestone
2.9.0
Development

Successfully merging this pull request may close these issues.

2 participants
@nedseb@fpistm

[8]ページ先頭

©2009-2025 Movatter.jp