mcci-catena / arduino-lmic

LoraWAN-MAC-in-C library, adapted to run under the Arduino environment
https://forum.mcci.io/c/device-software/arduino-lmic/
MIT License
638 stars 208 forks source link

fix sanity checks for new CFG_sx1276_radio boards #790

Open croghostrider opened 3 years ago

terrillmoore commented 2 years ago

Since they changed the silicon ID, I conclude that there are other changes we have to take into account. The LMIC should record the silicon ID for easy reference at this point; and we must review the LMIC to determine whether other changes are needed. See, for example, #787, which is possibly related. I'd rather not make this change until the review has been done; and I have no time to review, nor hardware that has this value in the registers. Deferring.

terrillmoore commented 2 years ago

See, for example, #787, which is possibly related.

Turns out, not related. The datasheet with the change still shows 0x12 as the chip id. Google search indicates that the differences between 0x12 and 0x13 are not yet documented. More info is needed.

terrillmoore commented 2 years ago

I checked today on Semtech's site and there's still not an updated datasheet with the new chip ID.

terrillmoore commented 2 years ago

A detailed review and comparison of the latest datasheet shows a new footnote: image So it's possible that a chip with ID 0x13 simply denotes a wafer delivery of an SX1276.