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

dts: arm: atmel: sam3x: add gpio header #86282

Merged

Conversation

everedero
Copy link
Contributor

The gpio header is missing, which will lead to failures when declaring GPIO down the line in board overlays.

The gpio header is missing, which will lead to failures
when declaring GPIO down the line in board overlays.

Signed-off-by: Eve Redero <eve.redero@gmail.com>
@zephyrbot zephyrbot added size: XS A PR changing only a single line of code platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) labels Feb 25, 2025
@nandojve
Copy link
Member

Hi @everedero ,

We are on release candidate phase at moment.
Could you create an issue so we can associate with your fix ?
This will allow us to merge the fix.

pdgendt
pdgendt previously approved these changes Feb 25, 2025
@pdgendt
Copy link
Collaborator

pdgendt commented Feb 25, 2025

The gpio header is missing, which will lead to failures when declaring GPIO down the line in board overlays.

The overlays can include the file, no?

@pdgendt pdgendt dismissed their stale review February 25, 2025 12:49

Pending question

@everedero
Copy link
Contributor Author

@pdgendt The other sam*.dtsi soc family files include it, like dts/arm/atmel/sam4e.dtsi, so it would not be as consistent to have it in overlay (and I like it more because it allows to keep the overlay level more end-user-friendly).
@nandojve I will create an issue, but it is not an emergency, more a like a nice-to-have kind of fix.

@fabiobaltieri fabiobaltieri added this to the v4.1.0 milestone Mar 3, 2025
@fabiobaltieri fabiobaltieri added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Mar 3, 2025
@fabiobaltieri fabiobaltieri merged commit af68151 into zephyrproject-rtos:main Mar 4, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) size: XS A PR changing only a single line of code Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants