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

fcb: start using errno codes #17481

Merged
merged 1 commit into from Sep 8, 2019

Conversation

nvlsianpu
Copy link
Collaborator

@nvlsianpu nvlsianpu commented Jul 11, 2019

Switch form using privater FCB error codes to
errno codes. FCB private codes convention were compatible
with <errno.h> codes:

  • 0 mean success
  • negative values mean errors
  • similar error types.
    There was no sense to kept private FCB error codes.

merge after #17425 as need to remove codes from fcb.h merged

Signed-off-by: Andrzej Puzdrowski andrzej.puzdrowski@nordicsemi.no

@nvlsianpu nvlsianpu added area: Storage Storage subsystem DNM This PR should not be merged (Do Not Merge) labels Jul 11, 2019
@zephyrbot zephyrbot added area: Settings Settings subsystem area: File System area: Tests Issues related to a particular existing or missing test labels Jul 11, 2019
@nvlsianpu nvlsianpu removed the DNM This PR should not be merged (Do Not Merge) label Aug 8, 2019
@nvlsianpu
Copy link
Collaborator Author

cc @carlescufi

@zephyrbot zephyrbot added the area: API Changes to public APIs label Aug 8, 2019
@ioannisg
Copy link
Member

ioannisg commented Aug 8, 2019

@nvlsianpu #17425 is merged.

@ioannisg
Copy link
Member

ioannisg commented Aug 8, 2019

@nvlsianpu is this needed for 2.0?

@nvlsianpu
Copy link
Collaborator Author

@ioannisg Not need - but still wort to merge, this is cleanup patch IMO.

@carlescufi
Copy link
Member

@nvlsianpu there's a typo ENSPACE

@nvlsianpu
Copy link
Collaborator Author

@carlescufi ^^ fixed

Switch form using privater FCB error codes to
errno codes. FCB private codes convention were compatible
with <errno.h> codes:
- 0 mean success
- negative values mean errors
- similar error types.
There was no sense to kept private FCB error codes.

Signed-off-by: Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>
@carlescufi carlescufi merged commit 94a022c into zephyrproject-rtos:master Sep 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: API Changes to public APIs area: File System area: Settings Settings subsystem area: Storage Storage subsystem area: Tests Issues related to a particular existing or missing test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants