Skip to content

Add alternative 0xecaa ID #12

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

Closed
wants to merge 1 commit into from

Conversation

facchinm
Copy link

@facchinm facchinm commented Sep 2, 2024

Some sensors mounted on Plug And Play kit are reporting 0xecaa as ID (may be a new silicon revision not tracked by the datasheet?)

The issue was not discovered before d2c29d5 got merged, since the check was succeeded in any case.

Some sensors mounted on Plug And Play kit are reporting 0xecaa as ID (may be a new silicon revision not tracked by the datasheet?)

The issue was not discovered before stm32duino@d2c29d5 got merged, since the check was succeeded in any case.
@fpistm fpistm requested a review from cparata September 2, 2024 10:58
@fpistm
Copy link
Member

fpistm commented Sep 2, 2024

Hi @facchinm
Thanks fr the PR, anyway it seems not correct the 0xecaa is not an id of the VL53L4CD but it is the one of VL53L4ED

https://github.com/stm32duino/VL53L4ED/blob/a94cc0b687c9bab4562ed7f0e99ec0dba3a8b798/src/vl53l4ed_class.h#L138

So you should use this library:
https://github.com/stm32duino/VL53L4ED

@fpistm fpistm added the invalid This doesn't seem right label Sep 2, 2024
@facchinm facchinm closed this Sep 2, 2024
@facchinm
Copy link
Author

facchinm commented Sep 2, 2024

Thanks for the heads up, I'll fix the integration on our side of the library

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants