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

Implement MCI emulation for native #4033

Closed
wants to merge 6 commits into from
Closed

Conversation

x3ro
Copy link
Contributor

@x3ro x3ro commented Oct 2, 2015

The MSBA2 has an MCI driver to talk to the on-board
SD-card reader, but until now there was no way of
running/testing an application using it under native.
This PR adds rudimentary emulation of the MCI interface
and provides a test application.

@x3ro x3ro added the Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation label Oct 2, 2015
@miri64
Copy link
Member

miri64 commented Oct 2, 2015

Would be cool if you can actually store to a file instead of an array :-)

@x3ro
Copy link
Contributor Author

x3ro commented Oct 2, 2015

@authmillenon This is just the low-level interface for talking to the storage medium. Everything else will be built on top of that, but I need this for testing :)

@miri64
Copy link
Member

miri64 commented Oct 2, 2015

Maybe I miss something, but cpu/native/mci/native-mci.c looks pretty much like an implementation and not like an interface for me. ;-) I meant, that you could store everything you throw in there into a file (with a fixed filename and size, e.g. cpu/native/mci8388608.dat), instead of the array native_mci_memory. Would also be far less RAM hungry ;-).

@x3ro
Copy link
Contributor Author

x3ro commented Oct 2, 2015

Oh :) Yeah, I misunderstood you there. Why would you want to store a file? That is, what would be your use case?

@miri64
Copy link
Member

miri64 commented Oct 2, 2015

The MCI interface is used for persistent data, right? A file is persistent. An array in RAM not.

@miri64
Copy link
Member

miri64 commented Oct 2, 2015

+ the module would block less RAM. And yes, when you start to use 8MB for one module I get grumpy, even on native ^^.

@LudwigKnuepfer
Copy link
Member

I already wrote a file based storage driver for native for some other interface.

@LudwigKnuepfer
Copy link
Member

#2239

@x3ro
Copy link
Contributor Author

x3ro commented Oct 13, 2015

@authmillenon Updated to use a file instead of RAM ✨

@miri64
Copy link
Member

miri64 commented Oct 13, 2015

💃

@miri64
Copy link
Member

miri64 commented Nov 26, 2015

Superseded by #4345

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants