On Wed, Sep 14, 2016 at 11:03:18PM +0300, Grygorii Strashko wrote: > On 09/14/2016 05:25 PM, Richard Cochran wrote: > > On Wed, Sep 14, 2016 at 04:02:30PM +0300, Grygorii Strashko wrote: > >> @@ -427,9 +427,6 @@ static void cpts_calc_mult_shift(struct cpts *cpts) > >> u64 ns; > >> u64 frac; > >> > >> - if (cpts->cc_mult || cpts->cc.shift) > >> - return; > >> - > >> freq = clk_get_rate(cpts->refclk); > >> > >> /* Calc the maximum number of seconds which we can run before > > > > This hunk has nothing to do with $subject. > > Sry, but I did not get your comment here :( > I'd happy to update patch according to your request, but could you provide more info here, pls? You added that code in patch #7. Then you moved it in patch #8. You could have made the code correct in patch #7 to begin with. > >> cpts->cc_mult = mult; > >> cpts->cc.mult = mult; > >> cpts->cc.shift = shift; > >> + If you want a blank line here, then put into the original patch #7. Thanks, Richard -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html