Skip to content

Conversation

khoa-nguyen-18
Copy link
Contributor

Add invisible RENESAS_PN_ config to reflect the SoC hardware information and provide input for hal_renesas

Update hal_renesas revision to get the update_renesas_soc_info

Signed-off-by: Khoa Nguyen <[email protected]>
Add invisible RENESAS_PN config to reflect the SoC hardware
information and provide input for hal_renesas

Signed-off-by: Khoa Nguyen <[email protected]>
@khoa-nguyen-18 khoa-nguyen-18 added the platform: Renesas RA Renesas Electronics Corporation, RA label Oct 1, 2025
Copy link

github-actions bot commented Oct 1, 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@d8ee5f1 zephyrproject-rtos/hal_renesas#150 zephyrproject-rtos/hal_renesas#150/files

DNM label due to: 1 project with PR revision

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

Copy link

sonarqubecloud bot commented Oct 1, 2025

- J (0x4a): Dual Core, MIPI DSI/CSI is not available
- K (0x4b): Dual Core, MIPI DSI/CSI is available

config RENESAS_PN_NUMBER_OF_CORES
Copy link
Contributor

Choose a reason for hiding this comment

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

Because now we have the config for PN number core, the existing config of second core boot should be updated?

https://github.com/renesas/zephyr/blob/update_renesas_soc_info/soc/renesas/ra/Kconfig#L27-L45

select XIP
select SOC_EARLY_INIT_HOOK

if SOC_SERIES_RA4E2
Copy link
Contributor

Choose a reason for hiding this comment

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

some MCUs (such as RA4E2) only have one flash size across all part numbers, but should we define a default RENESAS_PN_ROM_SIZE similar to other MCUs? This could help us maintain from HAL more effectively rather than keeping it fixed as it is now? https://github.com/renesas/hal_renesas/blob/update_renesas_soc_info/zephyr/ra/ra_cfg/fsp_cfg/bsp/ra4e2/bsp_mcu_device_pn_cfg.h#L11

Copy link
Contributor

Choose a reason for hiding this comment

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

or should we use directly from flash size config?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Boards/SoCs DNM (manifest) This PR should not be merged (controlled by action-manifest) manifest manifest-hal_renesas platform: Renesas RA Renesas Electronics Corporation, RA platform: Renesas Renesas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants