Skip to content

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