helium / longfi-arduino

Apache License 2.0
63 stars 48 forks source link

Update to support LMIC v3.1.0 #10

Closed ppannuto closed 4 years ago

ppannuto commented 4 years ago

Looks like the upstream adruino library released v3.1.0 two weeks ago.

From the changeset, this includes Helium patches that add the pinmap for Disco_L072cz_Lrwan1. Unfortunately, that means that when this code compiles, it now throws duplicate symbol errors. This simply removes the now redundant definition and fixes the type where needed to match upstream.

Verified end-to-end data works with this fix:

image
kent-williams commented 4 years ago

@ppannuto Thank you for this!