On 11/19/2015 03:08 PM, Mark yao wrote: > On 2015?11?19? 11:35, Chris Zhong wrote: >> + >> + /* >> + * Sometimes the clock driver can not set a accurate clock_rate >> for vop, >> + * get the true rate of vop_dclk and set it back to adjusted_mode. >> + */ >> + adjusted_mode->clock = clk_get_rate(vop->dclk) / 1000; >> out: >> ret_clk = clk_enable(vop->dclk); >> if (ret_clk < 0) { > Hi Chris > How about use clk_round_rate to check the clock rate first? > I think we can check it on vop_crtc_mode_fixup. Hi Mark Nice advice, I'll modify it in next version. Thanks. > > Thanks. > - ?ark Yao > > >