Hi Zhengxun, On Mon, Mar 15, 2021 at 12:55 PM Zhengxun Li <zhengxunli.mxic@xxxxxxxxx> wrote: > > Hi Shubhrajyoti, > > +static int clk_wzrd_dynamic_reconfig_f(struct clk_hw *hw, unsigned long rate, > + unsigned long parent_rate) > +{ > + int err; > + u32 value, pre; > + unsigned long rate_div, f, clockout0_div; > + struct clk_wzrd_divider *divider = to_clk_wzrd_divider(hw); > + void __iomem *div_addr = divider->base + divider->offset; > + > + rate_div = ((parent_rate * 1000) / rate); > > In some cases, the variable seems to overflow. Will fix in the next version. > > Thanks, > Zhengxun