Let's take a step back and consider the design.  What you really want
        is a new synchronization state.  Something like:
        
        enum servo_state {
                SERVO_UNLOCKED,
                SERVO_JUMP,
                SERVO_LOCKED_ALPHA,
                SERVO_LOCKED_BETA,
        };
        
        The new code should either go into a new servo type or possibly right
        into servo.c.  (In either case it should wrap the functionality of the
        existing servos.)
        
        When the underlying servo becomes LOCKED, then you enter ALPHA and
        start your counter.  When the time series of the offset has entered
        the acceptable zone, you enter the BETA state.
        
        At the top level (clock.c), on the transition from ALPHA:BETA you send
        the signaling message to reduce the Sync rate.
        
Hi Richard, 

This makes sense to me. I will make the corresponding changes and send you the 
updated series.

I have also gone through all the feedback you provided for other patches in the 
series and incorporated the changes.

Thanks,
Vedang
        See?
        
        Thanks,
        Richard
        
        
        
        
        
    
    
    _______________________________________________
    Linuxptp-devel mailing list
    Linuxptp-devel@lists.sourceforge.net
    https://lists.sourceforge.net/lists/listinfo/linuxptp-devel
    


_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to