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

add STM32L433RC, STM32L433RB and STM32L443RC support #2033

Merged
merged 5 commits into from
Jun 6, 2023

Conversation

emoninet2
Copy link
Contributor

Support for STM32L433RC, STM32L433RB and STM32L443RC

@emoninet2
Copy link
Contributor Author

I see there are a few Astyle errors. Do I create a new pull request with the errors fixed?

@fpistm
Copy link
Member

fpistm commented Jun 3, 2023

I see there are a few Astyle errors. Do I create a new pull request with the errors fixed?

Hi @emoninet2. Thanks for this PR. No, simply amend your commit with astyle fixes and force push on the same branch. Then this PR will be automatically updated.

Copy link
Member

@fpistm fpistm 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 the update.
Still one format issue.
I've also added the USB clock selection else USB will not work as it require 48MHz clock source.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
…ck.c

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
…ck.c

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
…ck.c

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm fpistm added this to In progress in STM32 core based on ST HAL via automation Jun 5, 2023
@fpistm fpistm added this to the 2.6.0 milestone Jun 5, 2023
STM32 core based on ST HAL automation moved this from In progress to Reviewer approved Jun 5, 2023
@fpistm fpistm merged commit 9fb5081 into stm32duino:main Jun 6, 2023
23 checks passed
STM32 core based on ST HAL automation moved this from Reviewer approved to Done Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants