Skip to content
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

'const lmic_pinmap' has no non-static data member named 'rxtx_rx_polarity' #56

Closed
BoelD opened this issue Mar 24, 2018 · 5 comments
Closed
Assignees
Labels

Comments

@BoelD
Copy link

BoelD commented Mar 24, 2018

When I try to compile an LMIC example sketch with the suggested settings for the Catena 4551 board, I am getting the following error:

const lmic_pinmap' has no non-static data member named 'rxtx_rx_polarity

I have read in the Catena Arduino Lora Wan lib that I should install LMIC 2.1 version of the library to avoid this issue.

But when I followed the link there, I ended up in this repo, which does not seem to contain the correct version? Or at least the error is (still) there when I try to compile an example sketch.

@BoelD
Copy link
Author

BoelD commented Mar 24, 2018

Update: I searched back and installed exactly the specified version: 2.1.0. The error is still there and same for later versions.

@terrillmoore
Copy link
Member

Sorry you're having problems. Let me take a look; I don't usually use the examples, and we don't have CI set up, so it's possible that we missed updating this during the revision that completed the support for hte Murata module.

@terrillmoore terrillmoore self-assigned this Mar 24, 2018
@terrillmoore
Copy link
Member

Hi again,

I researched this. Very sorry, this is a documentation error.

The file libraries/arduino-lmic/src/hal/hal.h defines a field named named "rxtx_rx_active", not "rxtx_rx_polarity". It has the documented function.

I strongly suggest you update to 2.1.4, because there are some important minor changes. If you're using our board support package, also update to v1.0.8 (if using Linux or macOS -- v1.07 will work well for Windows).

Sorry for the inconvenience. I'll update the docs now.

--Terry

terrillmoore added a commit that referenced this issue Mar 24, 2018
Fix #56: erroneous pinmap name in README.md
@terrillmoore
Copy link
Member

Thanks very much for your report. Docs are updated and new release posted.

@BoelD
Copy link
Author

BoelD commented Mar 25, 2018

Thanks for getting into this and solving it so quick! Now it all makes sense. I already wondered why I couldn't find "rxtx_rx_polarity" in the source code at all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants