Looks to me like you are going to have to tell us how you have the LCD module interface connected to the external memory bus in order to be able to help you.

At first glance your code looks substantially OK.

Michael Karas