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

MPU driver doesn't recognize device id #12855

Closed
BazookaJoe1900 opened this issue Aug 31, 2019 · 10 comments
Closed

MPU driver doesn't recognize device id #12855

BazookaJoe1900 opened this issue Aug 31, 2019 · 10 comments
Labels
bug Drivers 🔧 Sensors, Actuators, etc stale

Comments

@BazookaJoe1900
Copy link
Member

Describe the bug
Looking at the nsh of pixhawk4 boot I can see the next line:

WARN [mpu6000] unexpected ID 0xfd accepted

Looking at the code, there is check of valid known product ID. but it seems that its isn't valid.

https://github.com/PX4/Firmware/blob/78ef8aab2df33cd00e2c969bbff37ffd82bcf385/src/drivers/imu/mpu6000/MPU6000.cpp#L246-L278

Looking at the ICM20689, the MPU6000 datasheets I don't even see the register (MPUREG_PRODUCT_ID, 0x0c) documented. Who knows where the ID list came from?

Except for this warning I don't see any strange behavior, but if some of the pixhawk 4 has new device, that might behave different, its better to know, and remove the warning.

To Reproduce
Enable nsh (it can be done to one of the telemetry ports).
and look at the output

@BazookaJoe1900 BazookaJoe1900 changed the title MPU driver doesn't recognize device well MPU driver doesn't recognize device id Aug 31, 2019
@julianoes julianoes added bug Drivers 🔧 Sensors, Actuators, etc labels Sep 3, 2019
@stale
Copy link

stale bot commented Dec 2, 2019

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@BazookaJoe1900
Copy link
Member Author

Still relevant.

@stale stale bot removed the stale label Dec 2, 2019
@mhkabir
Copy link
Member

mhkabir commented Dec 2, 2019

@davids5?

@davids5
Copy link
Member

davids5 commented Dec 2, 2019

It was early IP shared with PX4. It is there to tell us there is a new version of the IC has arrived - it is a only a warning.

@BazookaJoe1900
Copy link
Member Author

BazookaJoe1900 commented Dec 5, 2019

I also tried to get the information about the register from TDK with no luck...
Can you be sure that they have no effect? And if so why do we test them at all?

@stale
Copy link

stale bot commented Mar 17, 2020

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@stale stale bot added the stale label Mar 17, 2020
@stale
Copy link

stale bot commented Mar 17, 2020

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@BazookaJoe1900
Copy link
Member Author

@davids5 , its look like dead end.
what do you say, maybe, at least to remove the warning? or say that 0xfd is unknown, but in the list?

@stale stale bot removed the stale label Mar 18, 2020
@stale
Copy link

stale bot commented Jun 17, 2020

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@stale stale bot added the stale label Jun 17, 2020
@BazookaJoe1900
Copy link
Member Author

dead end

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Drivers 🔧 Sensors, Actuators, etc stale
Projects
None yet
Development

No branches or pull requests

5 participants
@julianoes @davids5 @mhkabir @BazookaJoe1900 and others