Skip to content

Microchip sama7g5 gpio driver #92494

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 3 commits into from
Jul 31, 2025
Merged

Microchip sama7g5 gpio driver #92494

merged 3 commits into from
Jul 31, 2025

Conversation

@MCHP-MPU-Solutions-SHA
Copy link
Contributor

Add driver for sama7g5 GPIO controller (PIO4)
Update device-tree, add led & button devices related with GPIO Verified by blinky & button samples
@github-actions github-actions bot added area: GPIO platform: Microchip MEC Microchip MEC Platform platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) labels Jul 2, 2025
@MCHP-MPU-Solutions-SHA
Copy link
Contributor Author

Hello, guys.

Please could you help to review this PR.

Many thanks.
Xing.
@pdgendt
Copy link
Contributor

pdgendt commented Jul 15, 2025

Not really needed to create a new PR, in the future just force push to existing PR.
pdgendt
pdgendt reviewed Jul 15, 2025
View reviewed changes
Copy link
Contributor

@pdgendt pdgendt left a comment

Choose a reason for hiding this comment

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

A few more implicit boolean conversions.
@MCHP-MPU-Solutions-SHA
drivers: gpio: add gpio for sama7g5
33c35da 
Add driver for sama7g5 GPIO controller (PIO4) Signed-off-by: Xing Chen <xing.chen@microchip.com>
@MCHP-MPU-Solutions-SHA
dts: microchip: sam: add pio device to sama7g5
39deee1 
Add pioa, piob, pioc, piod and pioe devices to sama7g5 Signed-off-by: Xing Chen <xing.chen@microchip.com>
@MCHP-MPU-Solutions-SHA
boards: microchip: sam: add led and button device for sama7g54-ek
5d23d0c 
Devices has been verified by samples/basic/button Signed-off-by: Xing Chen <xing.chen@microchip.com>
@MCHP-MPU-Solutions-SHA
Copy link
Contributor Author

Not really needed to create a new PR, in the future just force push to existing PR.

Understand, thanks.
@MCHP-MPU-Solutions-SHA
Copy link
Contributor Author

A few more implicit boolean conversions.

All corrected, please help to review, thanks.
@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
16.2% Duplication on New Code

See analysis details on SonarQube Cloud
nandojve
nandojve approved these changes Jul 24, 2025
View reviewed changes
Copy link
Member

@nandojve nandojve left a comment

Choose a reason for hiding this comment

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

Ping @jvasanth1
@MCHP-MPU-Solutions-SHA
Copy link
Contributor Author

Hi Experts,
I was told that jvasanth1 does not have merge rights on zephyrproject-rtos.
Please who will help to merge this PR?

Many thanks.
Xing.
@pdgendt
Copy link
Contributor

pdgendt commented Jul 31, 2025

Hi Experts, I was told that jvasanth1 does not have merge rights on zephyrproject-rtos. Please who will help to merge this PR?

He doesn't need to merge it, but does need to approve it.
@MCHP-MPU-Solutions-SHA
Copy link
Contributor Author

Hi Experts, I was told that jvasanth1 does not have merge rights on zephyrproject-rtos. Please who will help to merge this PR?

He doesn't need to merge it, but does need to approve it.

Understand & thanks, I will check it with jvasanth1.
@cfriedt cfriedt merged commit 366d938 into zephyrproject-rtos:main Jul 31, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: GPIO platform: Microchip MEC Microchip MEC Platform platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM)

5 participants

@MCHP-MPU-Solutions-SHA @pdgendt @nandojve @jvasanth1 @cfriedt