-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add support for transmitting packets #14
Comments
I'm adding some wiggleroom in #21 to simplify experimenting with other ways of functioning. |
Sounds like a great idea, as I was using the crate in a project, I realised I was either going to have to make all the register functions public or make a public function configuring every single parameter. |
@Chocol4te As I see it, a low level module, or different crate will do what the constructor does today, and be responsible for, via public APIs, interacting with the registers. When creating the module like today, that inner SPI owner will not be exposed, nor the register interacting functions. So you would have either a simplified API for reception/transmission if you stay within a sweet spot, or you would be entirely on your own, but with the register names, fields, and defines for their possible values. |
@Chocol4te I'll be offline for the next 10 days or so, but I've pushed #21 which prepares the exposing thing, and created #22 for the second part of that. Ideally #10 would tie into that as well, but perhaps that can be added a bit later. |
@Chocol4te I've done said change now. Next up will be to change the macros to get proper type safety around the write/read methods. So now you chose between the limited high level cc1101::Cc1101 API, or the raw cc1101::lowlevel::Cc1101 API. Perhaps there should be some name cleanups in there at some point. I should have a second module within this month so I can play with transmission as well. |
I just wanted to say, great work on this library @dsvensson, I managed to get my custom cc1101 915MHz modules communicating thanks to your work. I had to expose the spi device to experiment since the library is unfinished, but I got ~100m through walls with max pa + crc + fec with no errors :) , next up I'll see if I can get better range by implementing retransmissions. Lmk if you still need a second module, I'll email you 3 of my own modules so you can keep going. |
@rubend056 would you mind emailing me those modules as well? [email protected] |
Figure out a good API for transmitting data. Blocked by me not having an extra CC1101 module to talk to, nor a personal use case.
The text was updated successfully, but these errors were encountered: