keypad on PORTF - ATMEGA128

unsigned short kp, cnt, oldstate = 0;
char txt[6];
int k=1;

// Keypad module connections
char  keypadPort at PORTF;
char  keypadPort_Direction at DDRF;
// End Keypad module connections

// LCD module connections
sbit LCD_RS at PORTG1_bit;
sbit LCD_EN at PORTG2_bit;
sbit LCD_D4 at PORTA4_bit;
sbit LCD_D5 at PORTA5_bit;
sbit LCD_D6 at PORTA6_bit;
sbit LCD_D7 at PORTA7_bit;

sbit LCD_RS_Direction at DDG1_bit;
sbit LCD_EN_Direction at DDG2_bit;
sbit LCD_D4_Direction at DDA4_bit;
sbit LCD_D5_Direction at DDA5_bit;
sbit LCD_D6_Direction at DDA6_bit;
sbit LCD_D7_Direction at DDA7_bit;

void main()
{
  cnt = 0;
  Keypad_Init();
  Lcd_Init();
  Lcd_Cmd(_LCD_CLEAR);
  Lcd_Cmd(_LCD_CURSOR_OFF);
  Lcd_Out(1, 1, "Keys  :");
  do {
    kp = 0;
    do
      kp = Keypad_Key_Click();
    while (!kp);

    switch (kp) {
      case  1: kp = 49; break; // 1
      case  2: kp = 50; break; // 2
      case  3: kp = 51; break; // 3
      case  4: kp = 65; break; // A
      case  5: kp = 52; break; // 4
      case  6: kp = 53; break; // 5
      case  7: kp = 54; break; // 6
      case  8: kp = 66; break; // B
      case  9: kp = 55; break; // 7
      case 10: kp = 56; break; // 8
      case 11: kp = 57; break; // 9
      case 12: kp = 67; break; // C
      case 13: kp = 42; break; // *
      case 14: kp = 48; break; // 0
      case 15: kp = 35; break; // #
      case 16: kp = 68; break; // D
      default: kp += 48;
    }
if (kp != oldstate)
{
      cnt = 1;
      oldstate = kp;
}
   Lcd_Chr(2, k, kp);
   k++;
  } while (1);
}