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

Allow configuring the i2c device at runtime #88

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dberlin
Copy link

@dberlin dberlin commented Sep 12, 2022

This patch adds the i2c device to the board configuration, so it can be configured at runtime, rather than just being a static macro.

It defaults to the value of the static macro for compatibility.

@smtc-bot
Copy link

Thank you for your inquiry.

Customers are encouraged to submit technical questions via our dedicated support portal at https://semtech.force.com/ldp/ldp_support.

We invite all users to visit the LoRa Developer Portal Forum at https://forum.lora-developers.semtech.com and to join the thriving LoRa development community!

@joelguittet
Copy link

Duplicated from my #63

@dberlin
Copy link
Author

dberlin commented Oct 19, 2022

It's actually not - yours tries to do a lot more complicated of a thing that involves separating out where various sensors can be accessed, etc.

Mine simply makes i2c configurable.

It is a subset of the functionality of yours, for sure, but not sure that means anything.

@joelguittet
Copy link

I agree with you. Hope you will have more chance to get this integrated in the repo. I guess unfortunately Semtech d'ont take public contribution... as indicated by the bot.

@dberlin
Copy link
Author

dberlin commented Oct 20, 2022

Yes, it seems unfortunate. I did get one accepted over on the loramac project (SX12XX code), so maybe there is hope.

If worst comes to worst, i will just start a version of this project that is a little more flexible/open.

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

Successfully merging this pull request may close these issues.

3 participants