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

MPU6886 example not compilling #3

Closed
pgrawehr opened this issue Feb 7, 2022 · 5 comments
Closed

MPU6886 example not compilling #3

pgrawehr opened this issue Feb 7, 2022 · 5 comments

Comments

@pgrawehr
Copy link

pgrawehr commented Feb 7, 2022

In the examples folder, there's an example for the MPU6886 imu. Unfortunately, it does not compile, because 'class M5Tough' has no member named 'IMU'. Looking at the code, that is correct, there's no reference whatsoever to an IMU in the declaration of the M5 class. According to the documentation, I'm not even sure the M5Though does have an IMU/MPU.

Can you clearify whether that example is supposed to work? Is there an IMU in the M5Though, and if so, what type is it?

@felmue
Copy link

felmue commented Feb 20, 2022

Hello @pgrawehr

according to the specifications the M5Tough does not have an MPU.

Thanks
Felix

@pgrawehr
Copy link
Author

@felmue Looks like it, yes. But since I doubt a bit the quality of that documentation I really wasn't sure (the page on the very similar Core2 is much longer).

Bottom line is probably that this example should be deleted, as it is missleading.

@felmue
Copy link

felmue commented Feb 20, 2022

Hello @pgrawehr

I agree, the documentation isn't always up-to-date and often contains copy / paste errors. That said I confirmed the lack of an MPU by running an I2C scan on the internal I2C bus - only three responses (Touch IC, AXP, and RTC).

@imliubo - I agree with @pgrawehr the MPU6886 example should probably be deleted.

Thanks
Felix

@imliubo
Copy link
Member

imliubo commented Feb 21, 2022

Hi @Tinyu-Zhao , please check this, thanks.

@Tinyu-Zhao
Copy link
Collaborator

This is an example that shouldn't exist, sorry for being misleading, we've removed it

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

No branches or pull requests

4 participants