Skip to content

Modified esp_spiffs functions to forward the spiffs error code #208

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

Closed
wants to merge 3 commits into from
Closed

Conversation

AFellmett
Copy link
Contributor

@AFellmett AFellmett commented Jun 4, 2018

I like the idea to have a working version of spiffs out of the box, but if something goes wrong I want to be able to determine what happend and handle the fault.
Spiffs itself provide these information so they should be forwarded to the user.

Update: Change wrongly labeled ret to res

AFellmett added 3 commits June 4, 2018 21:30
Changed position of formating check to be able to format the fs even if its not loaded. 
Required if SPIFFS_MAGIC is used
@trygvis
Copy link
Contributor

trygvis commented Jun 5, 2018

This is good.

I've made some changes to the SPIFFS code that probably would be nice to look at at the same time this is reviewed in #210.

wujiangang pushed a commit that referenced this pull request Jun 8, 2018
Changed position of formating check to be able to format the fs even if its not loaded.
Required if SPIFFS_MAGIC is used

Merges #208
@27-301 27-301 closed this Jun 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants