vServo1 is a byte and cant go above 255.

I will try declaring cServoOffTime as a variable instead. I think something similar to this was the reason for my thread about the datatypes of constants