/* Stan linii PD2 należy odczytać z rejestru PIND, nie z DDRD */
#define klawiszI PIND   

unsigned char klawiatura(void)
{
   unsigned char zwrot=0;

    /* Linia PD2 pracuje jako wejście "podciągnięte do jedynki" */
   cbi(klawiszD,pin);
   sbi(klawiszP,pin); 

        /* Odczyt stanu linii PD2 */
   if(bit_is_clear(klawiszI ,pin))
   {
      zwrot=zwrot+1;
   }
   return zwrot;
}

#include <avr/io.h>
#define F_CPU        1000000                                     
#define klawiszD DDRD
#define klawiszP PORTD
#define klawiszI PIND   
#define pin PD2

/* #define LED_ON sbi(DDRB,PB1);sbi(PORTB,PB1) */
#define LED_ON    DDRB|=_BV(PB1);PORTB|=_BV(PB1); 

unsigned char klawiatura(void)
{
   unsigned char zwrot=0;

   /* Linia PD2 pracuje jako wejście "podciągnięte do jedynki" */

   /*  cbi(klawiszD,pin); */
   klawiszD &= ~_BV(pin);
   /* sbi(klawiszP,pin);*/ 
   klawiszP |= _BV(pin);   

   /* Odczyt stanu linii PD2 */
   if(bit_is_clear(klawiszI ,pin))
   {
      zwrot=zwrot+1;
   }
   return zwrot;
}
/* Dalej bez zmian */

/* Czy na PD2 jest stan wysoki */
if( bit_is_set(klawiszI,pin) )
/* lub tak */ 
if( klawiszI & _BV(pin) )

/* Zeruje bit nr. 4 rejestru DDRC- linia PC4 ma pracować jako wejście*/
DDRC &= ~ _BV(PC4);
/* Odczyt wartość bitu nr.4 w rejestrze PINC */
/* Czy na PC4 jest stan wysoki? /
if( bit_is_set(PINC,PC4) )
/* Czy na PC4 jest stan niski? /
if( bis_is_clear(sfr,bit) )

DDRD &= ~ _BV(PD2); /* Zeruje bit nr 2 rejestru DDRD,  linia PD2 będzie wejściem */
PORTD |= _BV(PD2);  /* Ustawia  bit nr.2 rejestru PORTD, wyprowadzenie PD2 będzie wewnętrznie podciągnięte przez rezystor do zasilania */

#define F_CPU  1000000L              /* Częstotliwość kwarcu */
#include <avr/io.h>
#include <util/delay.h>              /* Żeby można było korzystać z funkcji  '_delay_ms'  */

int main(void)
{
   int i; /* zmienna przyda się w pętli 'for' */

   /* Ustawia bin nr. 1 rejestru DDRB, linia PB1 będzie wyjściem */
   DDRB  |= _BV(PB1);
 
   /* Ustawia  bit nr.1 rejestru PORTB, na wyprowadzeniu PB1 pojawi się stan wysoki  */
   PORTB |= _BV(PB1);

   /* Wykonanie tego fragmentu zajmie uC 10s */
   for (i = 0; i < 10*100; i++)
                _delay_ms(10);      /* Każdym razem wykonanie funkcji  "_delay_ms" zajmie uC  10ms */

   /* Kasuje bit nr.1 rejestru PORTB, na wyprowadzeniu PB1 pojawi się stan niski */
   PORTB &= ~_BV(PB1);
 
   while(1){};  /*  Nieskończona pętla "uwięzi" program w tym miejscu */

   return 0;
}

#define LED_OFF    DDRB|=_BV(PB1);PORTB&=~_BV(PB1); 

void zaswiec(void)
{
    if(klawiatura()==1)
    {
       LED_ON;
    }
    else
    {
       LED_OFF;
    }
}