On Fri, Dec 02, 2016 at 11:58:34AM -0600, Grygorii Strashko wrote: > or I missed smth? You are missing three important points. 1. Unlike the code you posted, no edges will be lost. 2. The solution using the PWM is implemented in USER SPACE. If people use this way, then they will be forced to understand the inherit limitations. In addition, the behavior of servo will be under their control. 3. The update rate of the PHC is not once per second. It can be any rate at all, like 16 Hz for the telecom profile. You can't just blindly pick out an adjustment value once per second. Using the feedback from the time stamped PWM and adjusting THAT at the PWM rate (also not necessarily 1 PPS) is the right way. The second reply in that thread is an even better solution, leaving the PHC free running and adjusting the timer input clock (probably they used a VCO). Just hacking in some kind of kernel PPS with unknown accuracy is just asking for trouble later, since people will expect HW accuracy. So just get the input time stamps working, and make PWM control available to userspace in mainline (not sure about this, I guess it isn't), and leave the PPS part to a userspace utility. Thanks, Richard -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
