drivers: pinmux: 👋 #55025
Merged
drivers: pinmux: 👋 #55025
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
gmarull
requested review from
nashif,
mnkp,
carlescufi,
galak,
MaureenHelm,
mmahadevan108,
dleach02,
albertofloyd,
erwango,
simonguinot,
ABOSTM,
FRASTM and
arnopo
as code owners
zephyrbot
added
platform: NXP
gmarull
force-pushed
the
drop-pinmux
branch
3 times, most recently
from
61c8614
to
509a7f2
Compare
simonguinot
previously approved these changes
Feb 21, 2023
|
let's give driver/soc/board maintainers/owners a couple of days to review and aknowledge before merging this. |
Drop RV32M1 pinmux driver in favor of pinctrl. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Drop STM32 pinmux driver in favor of pinctrl. Some definitions located in pinmux headers were used by the pinctrl driver, so they have been moved there. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Drop MCUX LPC pinmux driver in favor of pinctrl. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Drop LPC11U6X pinmux driver in favor of pinctrl. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Drop the MCUX driver in favor of Kinetis pinctrl driver. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Drop Microchip XEC driver in favor of pinctrl. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Driver did not use pinmux API. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
The test was not using the pinmux.h API. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Pinctrl is the new pinmux. Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
gmarull
dismissed stale reviews from stephanosio, galak, erwango, simonguinot, and fabiobaltieri
via
de0c710
|
rebased to fix conflicts |
gmarull
requested review from
fabiobaltieri,
simonguinot,
stephanosio and
erwango
and removed request for
stephanosio
fabiobaltieri
approved these changes
Feb 22, 2023
simonguinot
approved these changes
Feb 22, 2023
erwango
approved these changes
Feb 23, 2023
albertofloyd
approved these changes
Feb 23, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Removal of pinmux is scheduled for 3.4 release (see #39740). Let's finally remove all pinmux drivers.
NOTE: many SoCs had custom pinmux implementations (e.g.
-pinproperties for Nordic). I'll work later on removing such legacy code.