Skip to content

drivers: eeprom: fm25xxx: add support for infineon fm25xxx FRAM #94557

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

drivers: eeprom: fm25xxx: add support for infineon fm25xxx FRAM #94557

wants to merge 1 commit into from
+296 −0

Conversation

Orange-Murker
Copy link

This driver adds support for the Infineon FM25XXX series of FRAM chips.

Has been tested on Infineon FM25CL64B-G.

@zephyrbot zephyrbot added area: EEPROM platform: Infineon Infineon Technologies AG labels Aug 15, 2025
@Orange-Murker Orange-Murker force-pushed the fm25xxx-fram-support branch 4 times, most recently from dc2c114 to 2472a8b Compare August 15, 2025 17:10
dsseng
dsseng reviewed Aug 15, 2025
View reviewed changes
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Orange-Murker Orange-Murker requested a review from dsseng August 18, 2025 14:36
henrikbrixandersen
henrikbrixandersen requested changes Oct 5, 2025
View reviewed changes
henrikbrixandersen
henrikbrixandersen requested changes Oct 6, 2025
View reviewed changes
Copy link
Member

@henrikbrixandersen henrikbrixandersen left a comment

Choose a reason for hiding this comment

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

Please remove the unnecessary locking around the single SPI transaction.

@Orange-Murker
Copy link
Author

Please remove the unnecessary locking around the single SPI transaction.

Done

@henrikbrixandersen
Copy link
Member

Please rebase on main to resolve the CI issues.

@Orange-Murker
Copy link
Author

Please rebase on main to resolve the CI issues.

Done. Also removed a now unused variable

henrikbrixandersen
henrikbrixandersen requested changes Oct 8, 2025
View reviewed changes
Copy link
Member

@henrikbrixandersen henrikbrixandersen left a comment

Choose a reason for hiding this comment

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

I just remembered - you need to add a build-only test to tests/drivers/build_all/eeprom/ as well.

@Orange-Murker
Copy link
Author

What exactly needs to be added there? I don't see any other tests for specific FRAM/EEPROM chips in that directory

@henrikbrixandersen
Copy link
Member

What exactly needs to be added there? I don't see any other tests for specific FRAM/EEPROM chips in that directory

You need to add an entry to https://github.com/zephyrproject-rtos/zephyr/blob/main/tests/drivers/build_all/eeprom/app.overlay to ensure the driver gets built in CI.

@zephyrbot zephyrbot added the area: Tests Issues related to a particular existing or missing test label Oct 8, 2025
@zephyrbot zephyrbot requested a review from nashif October 8, 2025 14:30
drivers: eeprom: fm25xxx: add support for infineon fm25xxx FRAM
79a8d65 
This driver adds support for the Infineon FM25XXX series of chips.
Has been tested on Infineon FM25CL64B-G.

Signed-off-by: Luna Pes <[email protected]>
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Oct 8, 2025

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

henrikbrixandersen
henrikbrixandersen approved these changes Oct 9, 2025
View reviewed changes
Copy link
Member

@henrikbrixandersen henrikbrixandersen left a comment

Choose a reason for hiding this comment

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

Thanks!

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

@henrikbrixandersen henrikbrixandersen henrikbrixandersen approved these changes

@sreeramIfx sreeramIfx Awaiting requested review from sreeramIfx

@talih0 talih0 Awaiting requested review from talih0

@dsseng dsseng Awaiting requested review from dsseng

@mcatee-infineon mcatee-infineon Awaiting requested review from mcatee-infineon

@nashif nashif Awaiting requested review from nashif

At least 2 approving reviews are required to merge this pull request.

Assignees

@henrikbrixandersen henrikbrixandersen

Labels
area: Devicetree Bindings area: EEPROM area: Tests Issues related to a particular existing or missing test platform: Infineon Infineon Technologies AG
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Orange-Murker @henrikbrixandersen @dsseng @zephyrbot