Skip to content

pinmux: stm32: remove pre-DT API #38275

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
Merged
merged 2 commits into from
Oct 5, 2021
Merged

pinmux: stm32: remove pre-DT API #38275

merged 2 commits into from
Oct 5, 2021

Conversation

@gmarull
Copy link
Member

@gmarull gmarull commented Sep 3, 2021 •
edited
Loading 

dt-bindings: pinctrl: stm32: remove unused pin macros

The pin macros were used in the pre-DT pinmux era to configure the
mapping between a pin and its configuration.

drivers: pinmux: stm32: remove pre-DT API

Remove APIs used in the pre-DT era. In case manual access is required,
`stm32_dt_pinctrl_configure` can still be used as a shortcut with data
initialized in the expected format.

@github-actions github-actions bot added area: API Changes to public APIs area: Boards area: Devicetree Binding PR modifies or adds a Device Tree binding platform: STM32 ST Micro STM32 labels Sep 3, 2021
@gmarull gmarull force-pushed the pinmux-some-stm32-cleanups branch from a07c934 to 81321be Compare September 3, 2021 08:15
@gmarull gmarull added the DNM This PR should not be merged (Do Not Merge) label Sep 3, 2021
@gmarull gmarull changed the title pinmux: stm32: some cleanups pinmux: stm32: remove pre-DT API Sep 3, 2021
@gmarull gmarull added this to the v3.0.0 milestone Sep 3, 2021
@zephyrbot zephyrbot requested a review from mnkp September 3, 2021 10:31
@gmarull
dt-bindings: pinctrl: stm32: remove unused pin macros
97b6bbc 
The pin macros were used in the pre-DT pinmux era to configure the
mapping between a pin and its configuration.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
@gmarull
drivers: pinmux: stm32: remove pre-DT API
a381a0e 
Remove APIs used in the pre-DT era. In case manual access is required,
`stm32_dt_pinctrl_configure` can still be used as a shortcut with data
initialized in the expected format.

Signed-off-by: Gerard Marull-Paretas <[email protected]>
@gmarull gmarull force-pushed the pinmux-some-stm32-cleanups branch from 81321be to a381a0e Compare September 3, 2021 13:25
@gmarull gmarull removed the DNM This PR should not be merged (Do Not Merge) label Sep 3, 2021
ABOSTM
ABOSTM approved these changes Sep 8, 2021
View reviewed changes
Copy link
Contributor

@ABOSTM ABOSTM left a comment

Choose a reason for hiding this comment

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

LGTM

@cfriedt cfriedt merged commit f371ae3 into zephyrproject-rtos:main Oct 5, 2021
@gmarull gmarull deleted the pinmux-some-stm32-cleanups branch January 18, 2022 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@erwango erwango erwango approved these changes

@carlescufi carlescufi Awaiting requested review from carlescufi

@FRASTM FRASTM Awaiting requested review from FRASTM

@galak galak Awaiting requested review from galak

@Mani-Sadhasivam Mani-Sadhasivam Awaiting requested review from Mani-Sadhasivam

@MaureenHelm MaureenHelm Awaiting requested review from MaureenHelm

@nashif nashif Awaiting requested review from nashif

@mnkp mnkp Awaiting requested review from mnkp

+1 more reviewer

@ABOSTM ABOSTM ABOSTM approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@erwango erwango

Labels

area: API Changes to public APIs area: Boards area: Devicetree Binding PR modifies or adds a Device Tree binding area: Pinmux platform: STM32 ST Micro STM32

Projects

None yet

Milestone

v3.0.0

Development

Successfully merging this pull request may close these issues.

5 participants

@gmarull @erwango @ABOSTM @cfriedt @zephyrbot