-
Notifications
You must be signed in to change notification settings - Fork 7.5k
"MPU6050_6Axis_MotionApps_V6_12.h" and "newly added calibration code using PID in MPU6050.cpp" make compilation error. #450
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
Comments
Having similar problem with Arduino IDE when trying to compile MPU6050_DMP_using_DMP_V6.12
|
As long as you have the latest code from the master branch (particularly the |
What is the date of the changelog on top of your MPU6050.cpp file ? Mine is this and it works:
I struggled a lot to make it work in a custom C++ class on a ESP32 but now it is pretty good. |
Thanks a lot for your fast reply @intensite ! I moved folders |
Got it working now with DMP6 demo instead. Had to do some changes like described here: #367 But instead of adapting |
Yes I recall seing this. I probably applied this change early on but just didn't remember.
|
It uses the default here for I2CDEV_IMPLEMENTATION == I2CDEV_ARDUINO_WIRE which is just a |
Uh oh!
There was an error while loading. Please reload this page.
There are quite a lot of mismatches for data type. For instance, in "MPU6050_6Axis_MotionApps_V6_12.h",
I2Cdev::writeBit(devAddr,0x6B, 7, &(val = 1));
--> Last parameter's data type is _uint8_t*
However, in the latest version of "I2Cdev.h",
static bool writeBit(uint8_t devAddr, uint8_t regAddr, uint8_t bitNum, uint8_t data);
--> It must be _uint8_t.
Is there any solution for this problem without changing corresponding data type?
right now, I've removed the code which was making error. Then, works!
I think those should be written with MPU6050.cpp functions.
The text was updated successfully, but these errors were encountered: