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

fatfs: Fixed initialization of block device in mount/unmount functions #4562

Merged
merged 1 commit into from Jun 19, 2017

Conversation

Projects
None yet
6 participants
@geky
Member

geky commented Jun 14, 2017

At some point the "mount" parameter for "f_mount" was name "force". This led to a bit of confusion that ended with the default mount function never calling block device init.

This is fine, since the block device can be manually initialized, but a better user experience is where the filesystem initializes the block device for the user.

This is backwards compatible due to the repeatability of the block device init functions.

related issues #4526, #4407

fatfs: Fixed initialization of block device in mount/unmount functions
At some point the "mount" parameter for "f_mount" was name "force". This
led to a bit of confusion that ended with the default mount function
never calling block device init.

This is fine, since the block device can be manually initialized, but
a better user experience is where the filesystem initializes the block
device for the user.

This is backwards compatible due to the repeatability of the block
device init functions.
@sg-

This comment has been minimized.

Member

sg- commented Jun 15, 2017

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Jun 16, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 565

All builds and test passed!

@adbridge adbridge added ready for merge and removed needs: CI labels Jun 16, 2017

@theotherjimmy theotherjimmy merged commit e4bdedc into ARMmbed:master Jun 19, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment