Skip to content

drivers: flash: Add support Flash-LP driver for Renesas RA #89471

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

khoa-nguyen-18
Copy link
Contributor

  • Added support for the Flash-LP driver on Renesas RA.
  • Removed unused .code-in-ram section for RA4M1 and RA2A1.
  • Updated flash-controller to support Flash-LP on ek_ra4m1, ek_ra4w1, ek_ra2a1, and ek_ra2l1.
  • Enabled testing of flash/common on ek_ra4m1, ek_ra4w1, ek_ra2a1, and ek_ra2l1.

Copy link

github-actions bot commented May 5, 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@9d68ee7 zephyrproject-rtos/hal_renesas@9c29341 (main) zephyrproject-rtos/hal_renesas@9d68ee71..9c29341a

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 May 5, 2025
@khoa-nguyen-18
Copy link
Contributor Author

Hi @soburi, @petejohanson, this PR may be relevant to your board. Could you please have a look?

@khoa-nguyen-18 khoa-nguyen-18 force-pushed the support_renesas_ra4_flash branch from 64eef1a to f2bf94a Compare May 5, 2025 11:14
This commit is to update renesas commit ID to support Flash-LP

Signed-off-by: Khoa Nguyen <khoa.nguyen.xh@renesas.com>
Initial support Flash-LP driver for Renesas RA

Signed-off-by: Khoa Nguyen <khoa.nguyen.xh@renesas.com>
Signed-off-by: Phi Tran <phi.tran.jg@bp.renesas.com>
Remove code_in_ram section in linker file of RA4M1, RA2A1

Signed-off-by: Khoa Nguyen <khoa.nguyen.xh@renesas.com>
Add dts node to support Flash LP for:
- RA4: RA4M1, RA4W1
- RA2: RA2A1, RA2L1

Signed-off-by: Khoa Nguyen <khoa.nguyen.xh@renesas.com>
Add support Flash-LP for Renesas boards:
- RA4: ek_ra4m1, ek_ra4w1
- RA2: ek_ra2a1, ek_ra2l1

Signed-off-by: Khoa Nguyen <khoa.nguyen.xh@renesas.com>
Add support test app `flash/common` for ek_ra4m1, ek_ra4w1,
ek_ra2a1, ek_ra2l1

Signed-off-by: Khoa Nguyen <khoa.nguyen.xh@renesas.com>
@khoa-nguyen-18 khoa-nguyen-18 force-pushed the support_renesas_ra4_flash branch from f2bf94a to 62de7bd Compare May 8, 2025 02:02
@github-actions github-actions bot removed the DNM (manifest) This PR should not be merged (controlled by action-manifest) label May 8, 2025
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@de-nordic de-nordic assigned KhiemNguyenT and unassigned de-nordic May 8, 2025

if (!len) {
return 0;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Move it into line 54, if the len = 0, no need to check the others


if (!len) {
return 0;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

move above line 80


if (!len) {
return 0;
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Move above line 174


#define DT_DRV_COMPAT renesas_ra_flash_lp_controller

#define UNIT_CODE_FLASH_PROGRAMMING 8
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you use this marco?

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

Successfully merging this pull request may close these issues.

4 participants