Still not working, then modify the lcdcmd() function again as shown below.
void lcdcmd(unsigned char cmdout)
{
lcdport=cmdout; //Send command to lcdport=PORTB
rs=0;
rw=0;
Delay_ms(10); //add a delay here also
en=1;
Delay_ms(10);
en=0;
Delay_ms(10); //add a delay here also
}
Make sure that you are using the same kind of lcd module.
All you need is to get rid off that black boxes from lcd.
Still there is black boxes, then there is some mistake in your hardware.