-
Notifications
You must be signed in to change notification settings - Fork 34
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
Xtensa lx6 #22
Xtensa lx6 #22
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi,
yes, support for new devices is very welcome! I have one big question implementation wise, see comment below.
Co-authored-by: Rahix <rahix@rahix.de>
Currently CI is still failing because the xtensa crate does not compile on stable rust. I'll have to take a look... |
I would rather not try to add this... Can you check whether a |
Both seems to work fine: https://github.com/pyaillet/shared-bus/runs/3228754084?check_suite_focus=true However I'm not sure to understand how you want to split this. |
Thanks. I'll take it from here, you don't need to worry about it :) |
Hi !
Thank you for this crate I'm using it with a shared i2c bus on an esp32.
Would you be interested in adding support for this kind of device ?
If so, here is a PR.
I'm quite new to the Rust and embedded world so comments and suggestions are more than welcomed.