Test Loom_TSL2561 - Winnie

Question

Test Loom_TSL2561 - Winnie

Closed this issue 3 years ago · 4 comments

winniiew commented 3 years ago

TSL2561 failed to initialize

Answer 1 · 2021-08-30T21:43:28.000Z

[Device] Adding Module: Analog
[TSL2561] Initialize failed
[Device] Cannot add inactive module

Works with Adafruit library but not with Loom

Answer 2 · 2021-08-31T16:51:34.000Z

The TSL 2561 is discontinued and no longer available to purchase anyways. We should retire this option in the code base.

Answer 3 · 2021-08-31T19:03:39.000Z

I would disagree about removing this sensor module (or other working code not causing any harm). Even if it can't be purchased directly anymore, any existing hardware should still work just fine. Additionally, as an open source project, there may (perhaps unlikely) be people using the module still. It doesn't hurt to leave the sensor support code in the code base, other than perhaps compiled code size, but based on the nRF + sensor compile issues seen, the granularity of module enable sets should probably be reevaluated - e.g. perhaps add a deprecated sensor factory option.

Answer 4 · 2021-09-07T19:42:02.000Z

We'll be removing this from the library, so no need to worry about this anymore.