Skip to content

Conversation

khoa-nguyen-18
Copy link
Contributor

  • Update hal_renesas revision to get the FSP 6.2.0 migration commits
  • Update conditions for continuing data transfer for drivers/spi/spi_b_renesas_ra8.c
  • Remove redundant condition check causing transfer error for drivers/i2s/i2s_renesas_ra_ssie.c
  • Add pwm_test alias for Renesas devices
Copy link

github-actions bot commented Oct 18, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_renesas zephyrproject-rtos/hal_renesas@5ab2c84 zephyrproject-rtos/hal_renesas@f12f062 zephyrproject-rtos/hal_renesas@5ab2c84a..f12f062b

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hal_renesas DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Oct 18, 2025
@zephyrbot zephyrbot added platform: Renesas RA Renesas Electronics Corporation, RA platform: Renesas Renesas area: Tests Issues related to a particular existing or missing test area: SPI SPI bus area: I2S area: PWM Pulse Width Modulation labels Oct 18, 2025
thenguyenyf
thenguyenyf previously approved these changes Oct 20, 2025
Copy link
Contributor

@thenguyenyf thenguyenyf left a comment

Choose a reason for hiding this comment

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

LGTM. Please update the manifest revision

Update rev of hal_renesas to latest Signed-off-by: Khoa Tran <khoa.tran.yj@bp.renesas.com>
When performing polling-based data transfer without enabling interrupts, the current implementation stops transferring as soon as either the TX or RX buffer becomes NULL. This causes the transfer to stop prematurely, even if the other direction still has data to send or receive. This commit fixes the condition so that data transfer continues as long as one direction (TX or RX) still has data remaining. Signed-off-by: Khoa Tran <khoa.tran.yj@bp.renesas.com>
When enabling the DTC (Data Transfer Controller) for the I2S SSIE driver on Renesas RA8x2 boards, a redundant condition check prevented the interrupt service routines from entering during data transfer. This caused the transfer to fail when DTC was active. This commit removes the unnecessary condition check, allowing the DTC to operate correctly with I2S SSIE transfers on RA8x2 devices. Signed-off-by: Khoa Tran <khoa.tran.yj@bp.renesas.com>
Add pwm_test alias to support test pwm_api on Renesas RA devices Signed-off-by: Khoa Tran <khoa.tran.yj@bp.renesas.com>
@khoa-nguyen-18 khoa-nguyen-18 force-pushed the renesas_migrate_fsp_620 branch from 215f104 to 660c19e Compare October 21, 2025 02:57
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label Oct 21, 2025
@zephyrbot zephyrbot requested a review from nashif October 21, 2025 02:58
Copy link

@KhiemNguyenT KhiemNguyenT left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: I2S area: PWM Pulse Width Modulation area: SPI SPI bus area: Tests Issues related to a particular existing or missing test manifest manifest-hal_renesas platform: Renesas RA Renesas Electronics Corporation, RA platform: Renesas Renesas

7 participants