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

mcfg.rssi_aux_channel over pwm input if PPM is enabled #47

Open
KipK opened this issue Feb 28, 2014 · 12 comments
Open

mcfg.rssi_aux_channel over pwm input if PPM is enabled #47

KipK opened this issue Feb 28, 2014 · 12 comments

Comments

@KipK
Copy link
Contributor

KipK commented Feb 28, 2014

This is more a feature request than an issue.
the mcfg.rssi_aux_channel only works for a ppm channel if PPM is enabled. We can't use channel 2 as pwm input ( this ones remains free to use ) for frsky rssi.

It only works if we use PWM input for radio channels( feature PPM not set ).

@didgydont
Copy link

any news on this ?

@kiteretro
Copy link
Contributor

Just want to chime in with this request... something like using the RX pin 2 for this, I'm pretty sure this is the only unused one in that area that isn't GPS serial, or the softserials?

@hydra
Copy link
Contributor

hydra commented May 22, 2014

that's correct. RC3/4 are GPS, RC5-8 are softserial.
What kinds of RSSI input do you need? There's a few i'm aware of, PWM RSSI, Analog RSSI, any more?

@vbabich
Copy link

vbabich commented May 22, 2014

Frsky PWM RSSI input would be nice, duty cycle 1%-99% iirc

@KipK
Copy link
Contributor Author

KipK commented May 22, 2014

channel 2 can already be used for pwm rssi reading if radio input use separated channels.
The actual problem is it just doesn't work if ppm is enabled.

@hydra
Copy link
Contributor

hydra commented May 22, 2014

ok, there's a few things that would need to change then.

  1. the pwm mapping code, see pwmInit
  2. the storage of the rssi value after the irq handler reads the value - if it was enabled then the data would be overwritten by the ppm irq handler - see 'captures' variable an ppm/pwm callback handlers
  3. the code that extracts the RSSI value from rcData.

it seems that any solution that involves that code would be somewhat cumbersome or hacky. I would suggest splitting the mapping code from the PPM code and the PWM code and then adding PWM RSSI as a seperate feature or perhaps extracting what you need from the existing code and adding it as a seperate feature.

@hydra
Copy link
Contributor

hydra commented May 22, 2014

also, i think timecop has some new pwm code, according to commit b7fce19

@trollcop
Copy link
Member

Feel free to fuck off and stop spamming my project.

@carballada
Copy link

Sorry, but I don't understand, this feature or change will be available soon? still is open

@trollcop
Copy link
Member

Feel free to contribute code that makes it work.

@carballada
Copy link

@trollcop, yes, you are right, sorry if you feel like I demand something and if I am not able to contribute, was not my intention, only was a problem with my low language skills

@xeno010
Copy link

xeno010 commented Sep 26, 2014

The problem is, the FrSky PWM-RSSI singal is to fast.
e.g. D4R-II use a ~110 kHz PWM singal.

The best way is to use an RC-filter (convert a PWM in a analog singal (0-3.3V))
See #165 for more infos.

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

No branches or pull requests

8 participants