Bogachan,

The program, is looping through the servos using a pulse of 1ms for Servo1 and 2ms for Servo2.

On a standard servo this will mean that Servo1 is fully to the left, and Servo2 is fully to the right, and both servos will be held there.

Did you want the servos to move? If you do then you have to vary the pulse length to change the position of the servo.

you will also find it a bit easier to read if you use EQU in your code... like this (I have also removed the extra setb and clr code, you didn't need it.)

<pre>

SERVO1 EQU p3.3
SERVO2 EQU p3.4

loop: setb SERVO1
setb SERVO2
call 1ms
clr SERVO1
call 1ms
clr SERVO2
call 18ms
jmp loop
<pre>


I'm not actually sure what your question is, but I hope that helps.

Chris