search

almusil / rfm69

A generic rust driver to support RFM69 family wireless chips.
Apache License 2.0
9 stars 9 forks source link

Add support for High Power #4

Open almusil opened 4 years ago

almusil commented 4 years ago
Cightline commented 3 years ago

I've been working on the high power stuff for a couple days, I'll eventually send out a pull request. I also managed to get my rfm69 fork to to communicate with a Feather M0 (the Feather is using RadioHead).

Cightline commented 3 years ago

If you wanna take a look it's over here, I'm sure it'll need to be cleaned up a little. Just tell me what you want done and I'll correct it.

almusil commented 3 years ago

Hello,

thank you for working on this. I will definitely take a look. What is required in addition to have it communicate with Feather M0? If it is just matter of settings I was planning to have RadioHead defaults the same way as there is already low_power_labs_defaults.

Cightline commented 3 years ago

The spacing is off, but here's what I'm using. I believe the only difference is RadioHead sets the FIFO level to 15. Minus setting up the interrupt pin the Feather M0 communicates the same way the Raspberry Pi does.

almusil commented 3 years ago

Looks pretty good, if you don't mind would you open PR for that as well?

Cightline commented 3 years ago

Sure, it'll be a few days. I also saw a couple warnings that I need to fix.

Cightline commented 3 years ago

I haven't forgotten about this, just need a bit more time. Also what do you think about merging in the packet thing you wrote in one of the examples?, maybe we could add it as a feature?

almusil commented 3 years ago

I haven't forgotten about this, just need a bit more time.

No worries we don't have to rush it.

Also what do you think about merging in the packet thing you wrote in one of the examples?, maybe we could add it as a feature?

I haven't really thought about that, but if someone finds it useful why not.

Cightline commented 3 years ago

Once again I'm still gonna finish that pull request. Off-topic: I just ordered an RC airplane. I'm gonna see if I can use the RFM69HCW to relay telemetry messages. From what I've read I should be able to get 500m+ which would be great.

https://lowpowerlab.com/forum/moteino/rfm69hw-driverange-test/

Cightline commented 3 years ago

Ok I accidentally closed the pull request, but I am done minus actually testing the physical devices. I'll do another pull request once I confirm everything works.

whatisbyandby commented 2 weeks ago

Did this ever get merged? I'd be interested in leveraging the high power mode if it's already been implemented, or I could also help to finish the implementation, if it still needs some work.

almusil commented 2 weeks ago

Did this ever get merged? I'd be interested in leveraging the high power mode if it's already been implemented, or I could also help to finish the implementation, if it still needs some work.

Hi, there is no PR to merge. And the high power is not currently implemented.

whatisbyandby commented 1 week ago

I would be interested in adding this, if you're open to a contribution for this. I think it could work the same way the API for the radiohead library works. Like this

...
    fn set_tx_power(power_level: u8, is_high_power: bool) {
        // Validate if this is a high power model?
        // Set the power level
    }
...
almusil commented 1 week ago

I don't have a hw to test it out, however I'm open to any contribution.