lcd requires some time to execute the commands so after a command we need to either give it a delay or we can check the busy flag.the crystal frequency is ok. in your program delay(250) is appropriate for lcd.
i have run the c code and assembly code for lcd, and both displays data on lcd. you also check your connections.