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

Add initial FAT support #121

Merged
merged 5 commits into from
Nov 23, 2016
Merged

Add initial FAT support #121

merged 5 commits into from
Nov 23, 2016

Conversation

utzig
Copy link
Member

@utzig utzig commented Nov 17, 2016

This adds initial FAT support currently relying on hal_flash (tested only on native).

Added a simple test app to apps/fat2native which reads an image file and does some basic operations.

Testing was done with this image file:

http://utzig.org/files/fatfs.img

Sample output:

FAT type = FAT12
Number of FATs = 2
Root DIR entries = 512
Sectors/FAT = 2
Volume start = 0
FAT start = 1
DIR start = 5
Data start = 37

    1004 KiB total drive space.
     994 KiB available.

Listing 0:/
FSEVEN~1		       0 bytes
README.TXT		      26 bytes

README.txt found, showing contents:

------------------------------------------------
This is a FAT image file.

------------------------------------------------

@utzig
Copy link
Member Author

utzig commented Nov 17, 2016

This is not meant to be merged yet!

@utzig
Copy link
Member Author

utzig commented Nov 22, 2016

In general this is good for merging now.

Left FIXME/TODOs in some of the files to work on later. Most important issue would probably be making FAT safe for multi-threading and multi-file operation (remove globals, add locking, etc).

@ccollins476ad
Copy link
Contributor

Thanks Fabio, it looks great! I will merge it shortly.

P.S.: Should have called it fabfs :)

@asfgit asfgit merged commit 62d05db into apache:develop Nov 23, 2016
asfgit pushed a commit that referenced this pull request Nov 23, 2016
Merge remote-tracking branch 'utzig/initial-fat-support' into develop

* utzig/initial-fat-support:
  Fix comment style
  Remove unused config file
  Finished initial FS interface, test app uses fs_ops
  FAT fs with fs interface WIP
  Add initial FAT support
@utzig utzig deleted the initial-fat-support branch April 5, 2017 10:58
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.

None yet

3 participants