"CLR LCD_RW ; SELECTS WRITE MODE
SETB LCD_EN
CLR LCD_RS"

Better would have been

CLR LCD_RW ; SELECTS WRITE MODE
CLR LCD_RS
SETB LCD_EN

That will provide for enough address set up time. For instance this figure is 40ns minimum for HD44780 driver. Dont know about yours though.

Oleg is right about your WRITE and BOTHLINES calling each other endlessley.

"CLR P3.4
CPL P3.4"

Why ? Would not a simple SETB P3.4 do the job ? Maybe you want a pulse or there is some trick in it...

Raghu