16x2 LCD Working problem!

I have taken a code from a book to display characters on LCD. I ran this code in proteus and it is working perfectly but when I ran it on hardware but it doesn't display anything on LCD except that the back light is on. I know proteus is not 100% correct for hardware. I have been debugging it since morning but nothing got solved. I check voltages, remade the circuit, checked all connections too. Now it's getting so frustrating! I am using pic18f4520 microcontroller and 16x2 LCD and the compiler is MPLAB. I am attaching my schematic and code here.

P.S. I have read many forums here and did everything, installed potentiometer in the circuits, resistors etc, did reset on the board, tried every possible thing but nothing happens!

This is the schematic :
https://obrazki.elektroda.pl/3234481400_1367694148.png

and here's the code :

#include<p18f4520.h>
#define ldata PORTD
#define rs PORTCbits.RC0
#define rw PORTCbits.RC1
#define en PORTCbits.RC2
void lcdcmd(unsigned char value);
void lcddata(unsigned char value);
void MSDelay(unsigned int itime);

void main()
{
TRISD = 0;
TRISC = 0;
en = 0;
MSDelay(250);
lcdcmd(0x38);
MSDelay(250);
lcdcmd(0x0E);
MSDelay(15);
lcdcmd(0x01);
MSDelay(15);
lcdcmd(0x06);
MSDelay(15);
lcdcmd(0x80);
MSDelay(15);
lcddata('M');
MSDelay(15);
lcddata('D');
MSDelay(15);
lcddata('E');
}

void lcdcmd(unsigned char value)
{
ldata = value;
rs = 0;
rw = 0;
en = 1;
MSDelay(1);
en = 0;
}

void lcddata(unsigned char value)
{
ldata = value;
rs = 1;
rw = 0;
en = 1;
MSDelay(1);
en = 0;
}

void MSDelay(unsigned int itime)
{
unsigned i,j;
for(i = 0; i<itime; i++)
for(j = 0; j<135; j++);
}

Proteus ISIS is not considering the configuration bits values, instead, it's using a configuration word set in the device settings which should work fine most of times; the real PIC though has to be properly configured during programming.
Include the configuration bits in your code (__CONFIG(WDT_OFF & LVP_OFF ect..)) right after the #include<p18f4520.h> line and make sure the programmer doesn't have any overriding values for the ones you set.

T3STY said:

Proteus ISIS is not considering the configuration bits values, instead, it's using a configuration word set in the device settings which should work fine most of times; the real PIC though has to be properly configured during programming.
Include the configuration bits in your code (__CONFIG(WDT_OFF & LVP_OFF ect..)) right after the #include<p18f4520.h> line and make sure the programmer doesn't have any overriding values for the ones you set.

Click to expand...

Okay thanks I am going to try that What do you mean by overriding values? Sorry I am using LCD first time so I don't know much.

In the circuit you show VEE tied high, it should be tied low for you to see anything on the LCD or better still connected to a 5K pot to ground with its wiper initially set to ground.

pjmelect said:

In the circuit you show VEE tied high, it should be tied low for you to see anything on the LCD or better still connected to a 5K pot to ground with its wiper initially set to ground.

Click to expand...

I did that, as I mentioned earlier I tried everything, connected VEE to ground, Vcc, potentiometer, resistors etc but nothing happened...

Clock frequency used? If Clock frequency is > 4 MHz delay is not proper for LCD. Increase delay value.

with overriding values I mean that some programmers can write the configuration bits based on settings in the programmer itself, and totally ignoring the configuration bits set in the code.