Skip to content

Conversation

@quytranpzz
Copy link
Contributor

@quytranpzz quytranpzz commented Sep 24, 2025

  • Add group interrupt driver support
    Multiple peripheral interrupt requests (up to 32 requests) are grouped together as one interrupt request.
image
  • Add support interrupt driven for serial with group interrupt feature on RX26T
  • Add DTC support for async serial API

@github-actions
Copy link

github-actions bot commented Sep 24, 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@2a4f856 zephyrproject-rtos/hal_renesas#142 zephyrproject-rtos/hal_renesas#142/files

DNM label due to: 1 project with PR revision

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 Sep 24, 2025
@quytranpzz quytranpzz force-pushed the rx_zephyr_rx26t_grp_sci branch 2 times, most recently from 285af8b to 64a9d9c Compare October 2, 2025 04:21
@quytranpzz quytranpzz force-pushed the rx_zephyr_rx26t_grp_sci branch from 64a9d9c to b589038 Compare October 7, 2025 10:46
@quytranpzz quytranpzz marked this pull request as ready for review October 8, 2025 01:41
Comment on lines 11 to 14
int rx_grp_intc_set_callback(const struct device *dev, bsp_int_src_t vector, bsp_int_cb_t callback,
void *context);
int rx_grp_intc_set_grp_int(const struct device *dev, bsp_int_src_t vector, bool set);
int rx_grp_intc_set_gen(const struct device *dev, int is_number, bool set);
Copy link
Member

Choose a reason for hiding this comment

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

document these public functions

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks, I added document for these functions

const uint8_t prio;
};

static struct k_spinlock lock;
Copy link
Member

Choose a reason for hiding this comment

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

this means that all instances will have to grab this shared spinlock (instead of their own), is this intended?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It wasn’t intentional - I’ve moved it into the device data struct for each instance. Please take a look

@quytranpzz quytranpzz force-pushed the rx_zephyr_rx26t_grp_sci branch from a4fb234 to 02d8287 Compare October 15, 2025 07:29
@zephyrbot zephyrbot added the area: Tests Issues related to a particular existing or missing test label Oct 15, 2025
@quytranpzz
Copy link
Contributor Author

force-pushed to:

@quytranpzz quytranpzz added the platform: Renesas RX Renesas Electronics Corp, RX label Oct 15, 2025
@quytranpzz quytranpzz requested a review from ycsin October 15, 2025 07:31
@quytranpzz quytranpzz force-pushed the rx_zephyr_rx26t_grp_sci branch from 02d8287 to e8548bd Compare October 15, 2025 07:34
quytranpzz and others added 5 commits October 20, 2025 02:19
Update commit ID to latest for hal_renesas

Signed-off-by: Quy Tran <[email protected]>
Add support Group interrupt driver on RX

Signed-off-by: Tatsuya Ogawa <[email protected]>
Apply the grp irq support in serial driver for MCB-RX26T

Signed-off-by: Quy Tran <[email protected]>
Add dtc property node on RX26T dts, and ram section for
dtc_vector_table on RX26T SoC for dtc support

Signed-off-by: Quy Tran <[email protected]>
Add overlay for mcb_rx26t uart async test

Signed-off-by: Quy Tran <[email protected]>
@quytranpzz quytranpzz force-pushed the rx_zephyr_rx26t_grp_sci branch from e8548bd to f73af50 Compare October 20, 2025 06:05
@sonarqubecloud
Copy link

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

Labels

area: Boards/SoCs area: Devicetree Bindings area: Interrupt Controller area: RX area: Tests Issues related to a particular existing or missing test area: UART Universal Asynchronous Receiver-Transmitter DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_renesas platform: Renesas RX Renesas Electronics Corp, RX platform: Renesas Renesas

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants