you have taken port1 for passing commands
and port2 for data but for lcd we consider 8/4 data pins …then how can you make use same 8 pins for port1 and for port2..so consider any one port.
so take port2/port1.use command 0x01 to clear the display before 0x80 and b/w every command in intializing lcd use delay;