Also ich mache das in meinen Programmen auch so:

Erst mal Pullup einstellen: z.B. PORTC |= (1 << PC4);
Dann abfragen mit: if(!(PINC & (1 << PC4)))

Mach auch mal ein Ausrufezeichen davor.