display 1234

Hi all,

I need to write a program which displays 1234, this is what I've done but its not working anybody could help me pls.?

/************************************************* ************************
* A Program for the 7-Segmen Display Module – Task 4 *
************************************************** ************************
* Developed By: Full Name of Member 1 & Member 2 *
************************************************** ************************
* Date: *
* Module: CT3011N – Embedded Systems and Interfacing *
************************************************** ***********************/
#include <reg51.h>
#include <string.h>
/* Declaration of the Data and Clock Bits for the M5450 */
sbit Data = P1^4 ;
sbit Clock = P1^6 ;
//sbit Enable = ? ; /* Only on some Modules but not all */
/* A very short Delay of about 10 microseconds */
/* This doesn’t have to be accurate as explained in the lecture classes */
void M5450_Delay(void)
{
int i;
for (i=0;i<10000; i++)
{
;
}
}
void M5450_Pulse(void)
{
/* This function sends a single clock pulse to the M5450 Driver */
Clock=0;
Clock=1;
M5450_Delay(); /* A short Delay */
Clock=0;
}
void M5450_Send (void)
{
int D0,D1,D2,D3;

/* This function sends the 8-bit (byte) D to the M5450 in serial form */
/* with the least significant bit first */
D0=0x66;
D1=0x4F;
D2=0x5B;
D3=0x06;

}
/* The following function sends a start pulse to the M5450 */
void M5450_Start(void)
{
//Enable = 0; /* Eliminate this line if there is no Enable pin defined */
Data = 0;
M5450_Pulse();
Data = 1;
M5450_Pulse();
}
/* The Main Program */
void main (void)
{
/* Declaration of all local variables in the main part of the program */
/* First Send a Start Pulse to the M5450 */
M5450_Start();
for(;;)
{
M5450_Send(0x66);
M5450_Send(0x4F);
M5450_Send(0x5B);
M5450_Send(0x06);
/* Send a further Three Clock Pulses to Complete the 35 Pulses */
M5450_Pulse();
M5450_Pulse();
M5450_Pulse();
}