martin2250 / ADCTouch

touch sensing library for Arduino
MIT License
100 stars 24 forks source link

Support for ATTiny Processors #2

Closed gculik closed 6 years ago

martin2250 commented 6 years ago

Hi Jiri,

looks great, I'll merge it when I have some time on my hands.

Just one quick question: how does this line work: uint8_t digitalPin = analog_pin_to_digital_pin + ADCChannel;

with

const uint8_t PROGMEM analog_pin_to_digital_pin[] = {
    5, /* 0 */
    2,
    4,
    3,
    NOT_A_PIN, 
    NOT_A_PIN
};

, shouldn't this return a pointer to the value in program space?

Greetings Martin

gculik commented 6 years ago

Yes it should, I fixed the lookup.

martin2250 commented 6 years ago

Hi, just wanted to let you know that I've not forgotten about your pull request. I just want to test it beforehand which I don't have time for in the next few weeks. Thanks for being patient

gculik commented 6 years ago

No problem. I just needed it for a project with Adafruit Arduino Gemma, so I figured it would be useful to others.

martin2250 commented 6 years ago

thanks!