Hello Tulasi,

Thank you for posting your code. Here are the next steps for you to follow:

1. Please post your schematic. It is impossible to tell if the code is correct without the schematic also. For example, we don't know what kind of LCD you are using. We don't know the CPU clock frequency. We don't know if the connection between the LCD and the at89c52 is correct. Etc., etc., etc.


2. Please add comments to your code so that we can tell exactly why you wrote each part and exactly what you think it is supposed to do. Just as an example, you have some code to initialize Timer 1, but you don't use Timer 1 anywhere. Similarly, you have some code to initialize the UART, but you don't use the UART anywhere. Why did you bother to initialize these peripherals if you do not use them?


3. Please pay very close attention to the data sheet for the LCD controller that you are using. Some of them have very specific timing requirements that you must follow in order to make them work. Your program does not seem to take this into account.

-- Russ