Skip to content

Enable reception for ATtinyX4 and ATtinyX5#59

Open
cyosp wants to merge 2 commits intosui77:masterfrom
cyosp:master
Open

Enable reception for ATtinyX4 and ATtinyX5#59
cyosp wants to merge 2 commits intosui77:masterfrom
cyosp:master

Conversation

@cyosp
Copy link
Copy Markdown

@cyosp cyosp commented May 21, 2016

Hi Suat,

This pull request allows to enable reception for ATtinyX4 and ATtinyX5 using PinChangeInterrupt library of NicoHood.
I've tested with success on ATtiny85.

Best regards,
CYOSP

Comment thread RCSwitch.cpp
// according to discussion on issue #14 it might be more suitable to set the separation
// limit to the same time as the 'low' part of the sync signal for the current protocol.
unsigned int RCSwitch::timings[RCSWITCH_MAX_CHANGES];
#endif
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The above changes, and similar changes below, are not acceptable. These #ifdefs are there for an good reason: To keep code size low for people who do not need to receive data.

@vlad2005
Copy link
Copy Markdown

vlad2005 commented Sep 11, 2016

@cyosp
I try to use these modifications to be able to use RCSwitch on ordinary digital pin. My board is an arduino pro mini and use suggestion from @fingolfin to include necessary lib and code. But it's seem that not working.
Where can i have an discussion about that?
P.S. I think that is very important to make RCSwitch to work on regular pin because sometime external interrupts need for other tasks. In my case i need to precisely count revolution for two motors and using PIN CHANGE INTERRUPT is a bit slower because need to check witch pin got signal.
Thanks!

L.E. sorry, i'm a bit novice with github. I will continue discussion on RCSwitch fork make by @cyosp

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