Dear Girish,

It is good that your code is now working by use of "volatile"

But, remember you are working with MCS51 fly. This fly supports a veriety of bit operation. Try to take advantage of it.

If the variable is going to assume only 2 values (True & False), I would prefer declaring it as "bit".

Using "int" or "char" (signed or unsigned) instead of bit is OK, but it increases code as well as RAM memory required by program. (Your current example may not need it, but in future this can verywell help you)

Also execution time increases due to integer operations.

just for example:

int my_int;
my_int = 0;

asm generated will be something like this

 mov dptr,#_my_int
clr a
movx @dptr,a
inc dptr
movx @dptr,a

on other hand if you use bit

bit my_bit
my_bit = 0;

the asm generated would be something like

 clr _my_bit

Regards,
Mahesh