Skip to content

Conversation

@khoa-nguyen-18
Copy link
Contributor

Currently, the SPI pinctrl is incorrect for the spi0 node.
In this PR (#77850), we changed the pinctrl number for SPI from spi0 to spi1, but we forgot to update the node name to spi1.
This PR aims to fix this issue.

@zephyrbot zephyrbot added platform: Renesas RA Renesas Electronics Corporation, RA area: SPI SPI bus labels Jan 3, 2025
@duynguyenxa duynguyenxa added the bug The issue is a bug, or the PR is fixing a bug label Jan 3, 2025
@khoa-nguyen-18
Copy link
Contributor Author

The current push is just to correct the commit message.

duynguyenxa
duynguyenxa previously approved these changes Jan 3, 2025
thaoluonguw
thaoluonguw previously approved these changes Jan 3, 2025
Copy link
Contributor

@thaoluonguw thaoluonguw left a comment

Choose a reason for hiding this comment

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

@khoa-nguyen-18 : Please update copyright year for modified files.

Currently, the SPI pinctrl is incorrect for the spi0 node.
In this PR (zephyrproject-rtos#77850), we changed the pinctrl number for SPI
from spi0 to spi1, but we forgot to update the node name
to spi1.
This commit aims to fix:
- Correct the pinctrl name for spi
- Correct the spi node on ek_ra8d1.dts

Signed-off-by: Khoa Nguyen <[email protected]>
Currently, the SPI pinctrl is incorrect for the spi0 node.
In this PR (zephyrproject-rtos#77850), we changed the pinctrl number for SPI
from spi0 to spi1, but we forgot to update the node name
to spi1.
This commit aims to:
- Correct the spi node in test app spi_loopback for ek_ra8d1

Signed-off-by: Khoa Nguyen <[email protected]>
@khoa-nguyen-18
Copy link
Contributor Author

@khoa-nguyen-18 : Please update copyright year for modified files.

I have updated. Thank you

@kartben kartben merged commit 436a6d8 into zephyrproject-rtos:main Jan 27, 2025
18 checks passed
@khoa-nguyen-18 khoa-nguyen-18 deleted the fix_spi_node branch April 23, 2025 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: SPI SPI bus bug The issue is a bug, or the PR is fixing a bug platform: Renesas RA Renesas Electronics Corporation, RA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants