-
Notifications
You must be signed in to change notification settings - Fork 203
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
sd
package: sdcard
package redesign
#639
base: dev
Are you sure you want to change the base?
Conversation
Is it possible to make this implement BlockDevice so that it is natively compatible with tinyfs? Skimming https://github.com/tinygo-org/tinyfs/blob/release/tinyfs.go#L43 |
@bgould Will do, but I'll do it through another type that uses the ReadBlocks/WriteBlocks interface set to implement the tinyfs.BlockDevice interface. Already had this in mind as the next addition to |
@bgould I've added a draft of what I mean: https://github.com/tinygo-org/drivers/pull/639/files#diff-92f519b8e01de4c93b473c245b482f67da4e20bfbfbe48aaca799f35e45afb43R38 |
@bgould OK, I've finished the sd.BlockDevice type, seems to be working OK. Check it out and let me know what you think :) |
@soypat would you perhaps be able to add a README with some small explanation about how this works? For example what is |
OK, added a Readme and switched some filenames around to make navigation easier. |
@bgould have you had a chance to try this out yet? |
Paging Dr. @bgould 😸 any chance to try this out? |
New Features:
Is still a WIP. CRC7 still does not work and I'm getting errors on ever other ReadBlock call.