-
Notifications
You must be signed in to change notification settings - Fork 82
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
Adding support for FIFO buffer #14
Comments
Note: I only have an MP6050 to work with, so for now that's all the code supports (as far as I know). |
What are the intended applications for this? Much IMU use is as an element in a control system where delays are anathema: the application must read the device at a high rate with minimal latency. |
I can see a few uses. First, Invensense suggests it can be used to reduce power consumption by letting the main board sleep while the MPU is gathering data. In my case - I'm working on a system that needs to report data leading up to an event (a crash) and it's easier to use the MPU's built-in buffer than program my own to run on the microcontroller. |
Thanks for that, I imagine that's exactly the kind of scenario for which it was designed. @turbinenreiter What's your view on this? My feeling is that it's rather specialist. This is the first time a user has wanted this, so I'd be reluctant to increase the complexity of our code. But if @jcalbert is happy that it's working and is prepared to maintain and document it, we could put a link to his fork in the docs. @jcalbert Would you be happy with this approach? |
Easiest way is to send a merge request. That way, the changes become obvious and whether we merge it or not, it's then part of this GitLab repository. Generally, I also err on the side of merging. At some point I will have a day off to streamline again. |
I'm fine with either a link or a PR once I test a bit more. As for keeping the code clean, I now think that it might make more sense to move the FIFO code into another another module and import from |
@jcalbert were you able to make the FIFO code work? |
The MPU family has a built-in 1024 FIFO which can be set up to read in data from the thermometer, gyroscopes, and accelerometer.
I've been working on building support for that feature, but don't know the ins and outs of memory management or this module yet, so I'd figured I'd mention the project here in case anyone wants to help.
Working branch here.
The text was updated successfully, but these errors were encountered: