AVR: Why doesn't my if statement yield TRUE?
PINC5 and PORTC5 are defines that equate to 5. So the code does not read status of pins of IO port C at all to get the status of the bits, it is just comparing if number 5 is number 1 and it never is. Also, PORTC is the output data register. If you want to read status of a pushbutton, you must read PINC register.