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 pincrtl-dtsi files to add "very-high-speed" slew-rate value for MCO signal #224

Merged
merged 2 commits into from
Sep 12, 2024

Conversation

djiatsaf-st
Copy link
Contributor

Some important changes have been made

  1. In modules/hal/stm32/scripts/genpinctrl/stm32-pinctrl-config.yaml :
    Modify the yaml config file to add "very-high-speed" slew-rate for MCO signal

  2. In modules/hal/stm32/dts/st :
    Update pincrtl*;dtsi files for all stm32 family

@djiatsaf-st djiatsaf-st changed the title Update pincrtl-dtsi files for all stm32 family Update pincrtl-dtsi files to add "very-high-speed" slew-rate value for MCO signal Jul 30, 2024
@djiatsaf-st djiatsaf-st force-pushed the Update_pintcrl_with_MCO branch 5 times, most recently from c50098f to 1f19750 Compare August 1, 2024 14:47
dts/README.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@mathieuchopstm mathieuchopstm left a comment

Choose a reason for hiding this comment

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

Files appear to be correct - however, commits should IMO be reworded:

- scripts: genpinctrl: add "very-high-speed" slew rate
+ scripts: genpinctrl: make MCO pins use "very-high-speed" slew rate
- dts: st: update files after edition
+ dts: regenerate pinctrl.dtsi with "very-high-speed" MCO

Enable us to have frequency more than 16MHZ for MCO

Signed-off-by: Fabrice DJIATSA <[email protected]>
Re-generate all pincrtl.dtsi for each family with slew-rate added

Signed-off-by: Fabrice DJIATSA <[email protected]>
@erwango erwango merged commit 4804c11 into zephyrproject-rtos:main Sep 12, 2024
2 checks passed
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