hi anoop,
u didnt declare the sub-routines in the begining of the prgm. you should declare the sub-routine header following with semi colon.
ERROR:
void delay(unsigned int msec)
void lcdcmd(unsigned char item)
void lcddata(unsigned char item)
CORRECT:
void delay(unsigned int msec);
void lcdcmd(unsigned char item);
void lcddata(unsigned char item);
Note: declare every sub-routine like this in the beginning of prgm after the port pins are defined.