Skip to content

dts: pinctrl: added mspm0gx51x support#83

Open
JFMSP wants to merge 1 commit into
zephyrproject-rtos:masterfrom
msp-ti:feature/mspm0gx51x-support
Open

dts: pinctrl: added mspm0gx51x support#83
JFMSP wants to merge 1 commit into
zephyrproject-rtos:masterfrom
msp-ti:feature/mspm0gx51x-support

Conversation

@JFMSP

@JFMSP JFMSP commented Mar 16, 2026

Copy link
Copy Markdown

added initial mspm0gx51x support in pinctrl for boards like LP_MSPM0G3519

Comment thread dts/ti/mspm0/g/mspm0gx51x-pinctrl.dtsi Outdated
@@ -0,0 +1,3520 @@
#include<dt-bindings/pinctrl/mspm0-pinctrl.h>

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Add a space after #include, seems the other DTSI files in this also get this wrong.

@glneo

glneo commented Mar 16, 2026

Copy link
Copy Markdown

Looks fine, but why did these .dtsi files end up here in the HAL, they seem like something that would go just the same into the regular Zephyr repo maybe here: https://github.com/zephyrproject-rtos/zephyr/tree/main/dts/arm/ti/mspm0/g

@JFMSP

JFMSP commented Mar 17, 2026

Copy link
Copy Markdown
Author

Looks fine, but why did these .dtsi files end up here in the HAL, they seem like something that would go just the same into the regular Zephyr repo maybe here: https://github.com/zephyrproject-rtos/zephyr/tree/main/dts/arm/ti/mspm0/g

Hi @glneo, we've had this discussion with a few different Zephyr maintainers and the decision was made to move them out of zephyr proper and into the hal. I've seen it both ways but there's not a strong preference from my side one way or the other, but based on remaining consistent I'd like to have it stay in the ha

@ssekar15

Copy link
Copy Markdown
Collaborator

PR is duplicate of #76.
I already started working on it.

added initial mspm0gx51x support in pinctrl for devices like
LP_MSPM0G3519

Signed-off-by: Jackson Farley <j-farley@ti.com>
@JFMSP JFMSP force-pushed the feature/mspm0gx51x-support branch from 465cc87 to b73c495 Compare March 17, 2026 15:57
@glneo

glneo commented Apr 17, 2026

Copy link
Copy Markdown

@JFMSP , is this PR a duplicate of #76? If so we should close this one.

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.

3 participants