Do you really expect us to do all the work for you??

As the other guys have suggested - use a simulator.

At a guess, your delay routine might need a little work - I guess the delay routine causes a delay of 10 * 0.05s = 0.5s

In DELAY:
Load the timer with the required value first - it's bad practice to expect other code to do it for you. Also, you may need to reset the TF0 bit before use (time to read the manual!!!!). Why are you loading the timer for the baud rate in this routine??

You code is fairly well structured with lots of comments - but most of the comments just repeat the obvious - pop DPL ; pop data pointer
The comments have got to tell us something that may not be obvious (the DPTR might have a pointer to a data structure in which case the comment should say something like ;restore -> to hash table). Your delay routine could be named 'delay_half_sec' which gives us a hint as to what it does, or at least a comment above the subroutine saying ' Delays half a second, destroys A,B,DPTR uses Timer0'.

Your PARSE routine might need some looking at, R7 is used as the bit count, but you DEC R4.


Remember the micro does exactly what you told it to, which is maybe not what you want it to do!