Hi Shunqian, On 2016?09?01? 07:06, Shunqian Zheng wrote: > We would prefer the 2016M as 2.0G than 1992M which seems odd, adding > it to big cpu clk rate table then we can set 2016M in dts. > > Signed-off-by: Shunqian Zheng <zhengsq at rock-chips.com> > --- > drivers/clk/rockchip/clk-rk3399.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/clk/rockchip/clk-rk3399.c b/drivers/clk/rockchip/clk-rk3399.c > index ede6c47..a6a15c6 100644 > --- a/drivers/clk/rockchip/clk-rk3399.c > +++ b/drivers/clk/rockchip/clk-rk3399.c > @@ -375,6 +375,7 @@ static struct rockchip_cpuclk_rate_table rk3399_cpuclkb_rates[] __initdata = { > RK3399_CPUCLKB_RATE(2184000000, 1, 11, 11), > RK3399_CPUCLKB_RATE(2088000000, 1, 10, 10), > RK3399_CPUCLKB_RATE(2040000000, 1, 10, 10), > + RK3399_CPUCLKB_RATE(2016000000, 1, 9, 9), > RK3399_CPUCLKB_RATE(1992000000, 1, 9, 9), > RK3399_CPUCLKB_RATE(1896000000, 1, 9, 9), > RK3399_CPUCLKB_RATE(1800000000, 1, 8, 8), It looks good to me. Reviewed-by: Xing Zheng <zhengxing at rock-chips.com> Thanks. -- - Xing Zheng