just change the this
connect your lcd data pins to P2
void lcd_command(unsigned char value)
{
P2=value;
rs=0;
rw=0;
e=1;
delay_ms(100);
e=0;
}
void lcd_data(unsigned char value)
{
P2=value;
rs=1;
rw=0;
e=1;
delay_ms(100);
e=0;
}
or
if you use this connect ur lcd data pins to P1
void lcd_command(unsigned char value)
{
P1=value;
rs=0;
rw=0;
e=1;
delay_ms(100);
e=0;
}
void lcd_data(unsigned char value)
{
P1=value;
rs=1;
rw=0;
e=1;
delay_ms(100);
e=0;
}