C30....dspic33f + LCD 4-bit mode .... help

c30 clear watchdog

is any one work with dspic33f family before ? I must use dspic33fj256mc710 in my project .
here is my code in C30, i use LCD 16x2, use 5V for LCD and 3.3V for dspic, the compiler ok, but nothing show on my LCD ! I cant figure out ! HELP me

#include <p33FJ256MC710.h>
#include "stdio.h" // Used "sprintf" Function


_FOSCSEL(FNOSC_PRIPLL & // Select Primary OSC = XT + PLL
IESO_OFF ) // Startup With User Select OSC
_FOSC(FCKSM_CSDCMD & // Clock Switch Disabled
OSCIOFNC_OFF & // OSC2 Pin = Clock O/P
POSCMD_XT) // Primary OSC = XT
_FWDT(FWDTEN_OFF & // Disable Watchdog Timer
WINDIS_OFF & // Disabled Windowed WDT
WDTPRE_PR128 & // Watchdog prescaler=1:128
WDTPOST_PS32768) // Watchdog postscaler=1:32,768
_FPOR(FPWRT_PWR128) // Power-on Reset Value = 128ms
_CONFIG3(JTAGEN_OFF); // Disable JTAG Interface


// Character LCD Interface Pins
#define TRIS_DATA_PIN_4 TRISDbits.TRISD1 // Direction D4
#define TRIS_DATA_PIN_5 TRISDbits.TRISD2 // Direction D5
#define TRIS_DATA_PIN_6 TRISDbits.TRISD3 // Direction D6
#define TRIS_DATA_PIN_7 TRISDbits.TRISD12 // Direction D7
#define TRIS_RS TRISDbits.TRISD5 // Direction RS
#define TRIS_RW TRISDbits.TRISD4 // Direction RW
#define TRIS_E TRISDbits.TRISD13 // Direction E

#define DATA_PIN_4 LATDbits.LATD1 // RD1 = D4 LCD
#define DATA_PIN_5 LATDbits.LATD2 // RD2 = D5 LCD
#define DATA_PIN_6 LATDbits.LATD3 // RD3 = D6 LCD
#define DATA_PIN_7 LATDbits.LATD12 // RD12 = D7 LCD
#define RS_PIN LATDbits.LATD5 // RD5 = RS LCD
#define RW_PIN LATDbits.LATD4 // RD4 = RW LCD
#define E_PIN LATDbits.LATD13 // RD13 = E LCD

#define READ_PIN_BUSY PORTDbits.RD12 // Read LCD Busy Pin

/* Display ON/OFF Control */
#define DON 0x0F // Display on
#define DOFF 0x0B // Display off
#define CURSOR_ON 0x0F // Cursor on
#define CURSOR_OFF 0x0D // Cursor off
#define BLINK_ON 0x0F // Cursor Blink
#define BLINK_OFF 0x0E // Cursor No Blink

/* Cursor or Display Shift */
#define SHIFT_CUR_LEFT 0x13 // Cursor shifts to the left
#define SHIFT_CUR_RIGHT 0x17 // Cursor shifts to the right
#define SHIFT_DISP_LEFT 0x1B // Display shifts to the left
#define SHIFT_DISP_RIGHT 0x1F // Display shifts to the right

char lcd_buf[16]; // "sprint" LCD Buffer

/* Function Prototypes */
void Initial_4bitLCD(void); // Initial LCD Interface
void SetCursor(unsigned char); // Set Cursor Address
void WriteCmdLCD(unsigned char); // Write Command
void WriteDataLCD(unsigned char); // Write Data
void Delay_tW_LCD(void); // Enable Pulse Delay
char Read_BusyLCD(void); // Read LCD Busy
void print_LCD(void); // Print String to LCD
void Delay(unsigned long int); // Delay Time Function

int main(void)
{
/* Start of PLL Config Fcy = 40MIPS */
CLKDIV &= 0xFFE0; // PLLPRE[4..0] = 00000 (N1=2)
CLKDIV &= 0xFF3F; // PLLPOST[1:0] = 00 (N2=2)
PLLFBD = 0x26; // PLLDIV[8..0] = 000100110(M=40)
while(!OSCCONbits.LOCK); // Wait PLL to Lock
OSCCONbits.CLKLOCK = 1; // Disable PLL Modify
/* End of PLL Config Fly = 40MIPS */

Initial_4bitLCD(); // Initial LCD 4 Bit Interface

// Loop Print Message to LCD16 x 2 //
while(1) // Loop Continue
{
SetCursor(0x00); // Start Cursor Line-1
sprintf(lcd_buf,"ET-dsPIC33WEB-V1");
print_LCD();
SetCursor(0x40); // Start Cursor Line-2
sprintf(lcd_buf,"Run Speed 40MIPS");
print_LCD();
Delay(10000000); // Delay Display

SetCursor(0x00); // Start Cursor Line-1
sprintf(lcd_buf,"24 Bit dsPIC MCU");
print_LCD();
SetCursor(0x40); // Start Cursor Line-2
sprintf(lcd_buf,"BY...ETT CO.,LTD");
print_LCD();
Delay(10000000); // Delay Display
}

}

/********************/
/* Initial 4Bit LCD */
/********************/
void Initial_4bitLCD(void)
{
TRIS_DATA_PIN_4 = 0; // Set Port Direction = Output
TRIS_DATA_PIN_5 = 0;
TRIS_DATA_PIN_6 = 0;
TRIS_DATA_PIN_7 = 0;
TRIS_RS = 0;
TRIS_RW = 0;
TRIS_E = 0;
DATA_PIN_4 = 1; // Start 4 Bit Initial
DATA_PIN_5 = 1;
DATA_PIN_6 = 0;
DATA_PIN_7 = 0;
RS_PIN = 0; // Instruction Select
RW_PIN = 0; // Write Select
E_PIN = 0; // Disable LCD Read/Write
// Wait 15mS
Delay(600000); // Power-ON Delay (minimum of 15ms)

DATA_PIN_4 = 1; // Start 4 Bit Initial
DATA_PIN_5 = 1;
DATA_PIN_6 = 0;
DATA_PIN_7 = 0;
RS_PIN = 0; // Instruction Select
RW_PIN = 0; // Write Select
E_PIN = 1; // Strobe High Nibble
Delay_tW_LCD(); // Enable Pulse Delay
E_PIN = 0;
// Wait 4.1mS
Delay(164000); // Wait 4.1 mS
DATA_PIN_4 = 1; // Start 4 Bit Initial
DATA_PIN_5 = 1;
DATA_PIN_6 = 0;
DATA_PIN_7 = 0;
RS_PIN = 0; // Instruction Select
RW_PIN = 0; // Write Select
E_PIN = 1; // Strobe High Nibble
Delay_tW_LCD(); // Enable Pulse Delay
E_PIN = 0;
// Wait 100uS
Delay(4000); // Wait 100uS

WriteCmdLCD(0x32); // Start Function Set 4Bit LCD Interface
WriteCmdLCD(0x28); // Function Set (DL=0 4-Bit,N=1 2 Line,F=0 5X7)
WriteCmdLCD(0x0C); // Display on/off Control (Entry Display,Cursor off,Cursor not Blink)
WriteCmdLCD(0x06); // Entry Mode Set (I/D=1 Increment,S=0 Cursor Shift)
WriteCmdLCD(0x01); // Clear Display (Clear Display,Set DD RAM Address=0)
}

/**************************/
/* Set Cursor Address LCD */
/**************************/
void SetCursor(unsigned char address)
{
RS_PIN = 0; // Select Instruction
RW_PIN = 0; // Write Select
DATA_PIN_7 = 1; // DB7 Must be 1 For Setting DDRam ADDR
DATA_PIN_6 = ((address & 0x40)>>6);
DATA_PIN_5 = ((address & 0x20)>>5);
DATA_PIN_4 = ((address & 0x10)>>4);
E_PIN = 1; // Strobe High Nibble
Delay_tW_LCD(); // Enable Pulse Delay
E_PIN = 0;
Delay_tW_LCD(); // Enable Pulse Delay
RS_PIN = 0; // Select Instruction
RW_PIN = 0; // Write Select
DATA_PIN_7 = ((address & 0x08)>>3);
DATA_PIN_6 = ((address & 0x04)>>2);
DATA_PIN_5 = ((address & 0x02)>>1);
DATA_PIN_4 = ((address & 0x01)>>0);
E_PIN = 1; // Strobe Low Nibble
Delay_tW_LCD(); // Enable Pulse Delay
E_PIN =0;

while(Read_BusyLCD()); // Wait LCD Ready
}

/************************/
/* Write Command to LCD */
/************************/
void WriteCmdLCD(unsigned char cmd)
{
RS_PIN = 0; // Select Instruction
RW_PIN = 0; // Write Select
DATA_PIN_7 = ((cmd & 0x80)>>7);
DATA_PIN_6 = ((cmd & 0x40)>>6);
DATA_PIN_5 = ((cmd & 0x20)>>5);
DATA_PIN_4 = ((cmd & 0x10)>>4);
E_PIN = 1; // Strobe High Nibble
Delay_tW_LCD(); // Enable Pulse Delay
E_PIN = 0;
Delay_tW_LCD(); // Enable Pulse Delay
RS_PIN = 0; // Select Instruction
RW_PIN = 0; // Write Select
DATA_PIN_7 = ((cmd & 0x08)>>3);
DATA_PIN_6 = ((cmd & 0x04)>>2);
DATA_PIN_5 = ((cmd & 0x02)>>1);
DATA_PIN_4 = (cmd & 0x01);
E_PIN = 1; // Strobe Low Nibble
Delay_tW_LCD(); // Enable Pulse Delay
E_PIN = 0;

while(Read_BusyLCD()); // Wait LCD Ready
}

/*********************/
/* Write Data to LCD */
/*********************/
void WriteDataLCD(unsigned char data)
{
RS_PIN = 1; // Select Data
RW_PIN = 0; // Write Select
DATA_PIN_7 = ((data & 0x80)>>7);
DATA_PIN_6 = ((data & 0x40)>>6);
DATA_PIN_5 = ((data & 0x20)>>5);
DATA_PIN_4 = ((data & 0x10)>>4);
E_PIN = 1; // Strobe High Nibble
Delay_tW_LCD(); // Enable Pulse Delay
E_PIN = 0;
Delay_tW_LCD(); // Enable Pulse Delay
RS_PIN = 1; // Select Data
RW_PIN = 0; // Write Select
DATA_PIN_7 = ((data & 0x08)>>3);
DATA_PIN_6 = ((data & 0x04)>>2);
DATA_PIN_5 = ((data & 0x02)>>1);
DATA_PIN_4 = (data & 0x01);
E_PIN = 1; // Strobe Low Nibble
Delay_tW_LCD(); // Enable Pulse Delay
E_PIN = 0;

while(Read_BusyLCD()); // Wait LCD Ready
}

/*************************/
/* Pulse Delay Write LCD */
/*************************/
void Delay_tW_LCD(void) // Enable Pulse Delay
{
int i;
for(i=0;i<500;i++); // Wait 10uS
}

/******************/
/* Wait LCD Ready */
/******************/
char Read_BusyLCD(void)
{
TRIS_DATA_PIN_7 = 1; // Set Port Direction = Input

RW_PIN = 1; // Read Function Select
RS_PIN = 0; // Instruction Select
Delay_tW_LCD(); // Enable Pulse Delay

E_PIN = 1; // Enable Read
Delay_tW_LCD(); // Enable Pulse Delay

if(READ_PIN_BUSY) // Verify Busy Flag
{
TRIS_DATA_PIN_7 = 0; // Set Port Direction = Output
E_PIN = 0; // Disable Read
return 1;
}
else
{
TRIS_DATA_PIN_7 = 0; // Set Port Direction = Output
E_PIN = 0; // Disable Read
return 0;
}
}

/***********************/
/* Print String to LCD */
/***********************/
void print_LCD(void)
{
char *p; // Pointer Buffer
p = lcd_buf; // LCD Buffer

do // Get char & Print Until null
{
WriteDataLCD(*p); // Write char to UART1
p++; // Next char
}
while(*p != '\0'); // End of ASCII (null)

return;
}

/***********************/
/* Delay Time Function */
/* 1-4294967296 */
/***********************/
void Delay(unsigned long int count1)
{
while(count1 > 0) {count1--;} // Loop Decrease Counter
}

c30 delay

Hi,
While partly going through the code, I found all the command values given during the software init routine appear to be incorrect. The first few bytes for initial functionset itself should have been '0010' instead of '0011' which specifies 8 bit interface. Please check the whole set of values used.

Will get back later if I find some more problems.

Regards,
Laktronics