By the way - spend some time picking up the wibes on using a busy-loop as delay:

void delay(unsigned int ms) {
    unsigned char n;
    unsigned int i;
    for(n=0;n<ms;n++) {
        for(i=0;i<count;i++) ;
    }
}

Exactly how do you know this function really gives a delay of "ms" milliseconds?
Why not use a delay that base the delay time on real, clocked hardware - like for example a hw timer? Wouldn't it be a huge advantage if the delay isn't greatly changed by compiler version or compiler options?