Skip to content
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

NVSTORE test issue #8555

Closed
jeromecoutant opened this issue Oct 26, 2018 · 10 comments · Fixed by #8951
Closed

NVSTORE test issue #8555

jeromecoutant opened this issue Oct 26, 2018 · 10 comments · Fixed by #8951

Comments

@jeromecoutant
Copy link
Collaborator

jeromecoutant commented Oct 26, 2018

Description

Hi

features-storage-nvstore-tests-nvstore-functionality is failed for NUCLEO_F410RB.

Issue is the same as in #7212
Fix seems to be not sufficient.
I checked my validation status, but it seems that I checked only flashiap tests at that time.
I forgot nvstore test...
Sorry to reopen this point.

@davidsaada

Issue request type

[ ] Question
[ ] Enhancement
[x] Bug

@jeromecoutant
Copy link
Collaborator Author

#define NVSTORE_NUM_AREAS 2

I think the problem is that NVSTORE needs 2 FLASH area.
As explained in #7149, due to the memory mapping, some targets can't used 2 areas.

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 26, 2018

@ARMmbed/mbed-os-storage

@ciarmcom
Copy link
Member

Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-109

@dannybenor
Copy link

This cannot be fixed. NVStore requires 2 sectors

@dannybenor
Copy link

I think this can be closed

@jeromecoutant
Copy link
Collaborator Author

So how can I undefine NVSTORE for NUCLEO_F410RB ?

@dannybenor
Copy link

Yes

@dannybenor
Copy link

@jeromecoutant you can remove "FLASH" from "device_has_add": in NUCLEO_F410RB

@jeromecoutant
Copy link
Collaborator Author

No...
NUCLEO_F410RB supports FLASH!
It doesn't support NVSTORE

@dannybenor
Copy link

OK
You can use https://github.com/ARMmbed/mbed-os/blob/master/features/storage/nvstore/mbed_lib.json to disable NVStore for the relevant targets.
Just for my curiosity, what will be the effect of removing "FLASH" for this target?

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

Successfully merging a pull request may close this issue.

4 participants