-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Support SPI driver for Renesas RA6 devices #79285
base: main
Are you sure you want to change the base?
Support SPI driver for Renesas RA6 devices #79285
Conversation
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
3ea7407
to
c21060e
Compare
Does this work with RA2, 4, 8? |
Add SPI driver code and basic support for RA6 Signed-off-by: Tri Nguyen <tri.nguyen.wj@bp.renesas.com> Signed-off-by: Thao Luong <thao.luong.uw@renesas.com>
Update hal_renesas which support for spi on RA6 Signed-off-by: Thao Luong <thao.luong.uw@renesas.com>
c21060e
to
7b62c6c
Compare
Hello @soburi,
It expects to work with RA2 and RA4. We are updating device tree and verifying for RA4 and RA2. They will be update later or in another PR. For RA8, it already supported. It uses sci_b (spi_b_renesas_ra8.c). |
Understood. I was worried that the implementation would need to be tweaked for each model. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All seems quite fine to me (but for the tiny comment I made on the code part)
Aside of this, could you split your first commit in 3:
- one for the driver (its dts bindings and the actual driver)
- one for the boards
- and one for the tests
case SPI_EVENT_ERR_FRAMING: /* Framing error */ | ||
case SPI_EVENT_ERR_MODE_UNDERRUN: /* Underrun error */ | ||
spi_context_cs_control(&data->ctx, false); | ||
spi_context_complete(&data->ctx, dev, 0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this path seems to be the one for errors, but you still return 0 as code? (it should be -EIO)
Add support for SPI driver for RA6 devices