From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from regular2.263xmail.com ([211.157.152.3]:55398 "EHLO regular2.263xmail.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751327AbcAUBqO (ORCPT ); Wed, 20 Jan 2016 20:46:14 -0500 Received: from regular1.263xmail.com (unknown [192.168.165.231]) by regular2.263xmail.com (Postfix) with ESMTP id 460B71DD0C for ; Thu, 21 Jan 2016 09:46:10 +0800 (CST) Message-ID: <56A0A812.5090803@rock-chips.com> Date: Thu, 21 Jan 2016 01:42:42 -0800 From: zhangqing MIME-Version: 1.0 To: Heiko Stuebner , mturquette@baylibre.com, sboyd@codeaurora.org CC: linux-clk@vger.kernel.org, linux-rockchip@lists.infradead.org, xf@rock-chips.com Subject: Re: [PATCH 2/4] clk: rockchip: fix rk3368 cpuclk core dividers References: <1453326560-20475-1-git-send-email-heiko@sntech.de> <1453326560-20475-2-git-send-email-heiko@sntech.de> In-Reply-To: <1453326560-20475-2-git-send-email-heiko@sntech.de> Content-Type: text/plain; charset=windows-1252; format=flowed Sender: linux-clk-owner@vger.kernel.org List-ID: hi: On 01/20/2016 01:49 PM, Heiko Stuebner wrote: > Similar to commit 9880d4277f6a ("clk: rockchip: fix rk3288 cpuclk core > dividers") it seems the cpuclk dividers are one to high on the rk3368 > as well. > > And again similar to the previous fix, we opt to make the divider list > contain the values to be written to use the same paradigm for them on all > supported socs. > > Fixes: 3536c97a52db ("clk: rockchip: add rk3368 clock controller") > Reported-by: Zhang Qing > Signed-off-by: Heiko Stuebner Reviewed-by: zhangqing > --- > drivers/clk/rockchip/clk-rk3368.c | 40 +++++++++++++++++++-------------------- > 1 file changed, 20 insertions(+), 20 deletions(-) > > diff --git a/drivers/clk/rockchip/clk-rk3368.c b/drivers/clk/rockchip/clk-rk3368.c > index f6667b8..3c9733e 100644 > --- a/drivers/clk/rockchip/clk-rk3368.c > +++ b/drivers/clk/rockchip/clk-rk3368.c > @@ -218,29 +218,29 @@ static const struct rockchip_cpuclk_reg_data rk3368_cpuclkl_data = { > } > > static struct rockchip_cpuclk_rate_table rk3368_cpuclkb_rates[] __initdata = { > - RK3368_CPUCLKB_RATE(1512000000, 2, 6, 6), > - RK3368_CPUCLKB_RATE(1488000000, 2, 5, 5), > - RK3368_CPUCLKB_RATE(1416000000, 2, 5, 5), > - RK3368_CPUCLKB_RATE(1200000000, 2, 4, 4), > - RK3368_CPUCLKB_RATE(1008000000, 2, 4, 4), > - RK3368_CPUCLKB_RATE( 816000000, 2, 3, 3), > - RK3368_CPUCLKB_RATE( 696000000, 2, 3, 3), > - RK3368_CPUCLKB_RATE( 600000000, 2, 2, 2), > - RK3368_CPUCLKB_RATE( 408000000, 2, 2, 2), > - RK3368_CPUCLKB_RATE( 312000000, 2, 2, 2), > + RK3368_CPUCLKB_RATE(1512000000, 1, 5, 5), > + RK3368_CPUCLKB_RATE(1488000000, 1, 4, 4), > + RK3368_CPUCLKB_RATE(1416000000, 1, 4, 4), > + RK3368_CPUCLKB_RATE(1200000000, 1, 3, 3), > + RK3368_CPUCLKB_RATE(1008000000, 1, 3, 3), > + RK3368_CPUCLKB_RATE( 816000000, 1, 2, 2), > + RK3368_CPUCLKB_RATE( 696000000, 1, 2, 2), > + RK3368_CPUCLKB_RATE( 600000000, 1, 1, 1), > + RK3368_CPUCLKB_RATE( 408000000, 1, 1, 1), > + RK3368_CPUCLKB_RATE( 312000000, 1, 1, 1), > }; > > static struct rockchip_cpuclk_rate_table rk3368_cpuclkl_rates[] __initdata = { > - RK3368_CPUCLKL_RATE(1512000000, 2, 7, 7), > - RK3368_CPUCLKL_RATE(1488000000, 2, 6, 6), > - RK3368_CPUCLKL_RATE(1416000000, 2, 6, 6), > - RK3368_CPUCLKL_RATE(1200000000, 2, 5, 5), > - RK3368_CPUCLKL_RATE(1008000000, 2, 5, 5), > - RK3368_CPUCLKL_RATE( 816000000, 2, 4, 4), > - RK3368_CPUCLKL_RATE( 696000000, 2, 3, 3), > - RK3368_CPUCLKL_RATE( 600000000, 2, 3, 3), > - RK3368_CPUCLKL_RATE( 408000000, 2, 2, 2), > - RK3368_CPUCLKL_RATE( 312000000, 2, 2, 2), > + RK3368_CPUCLKL_RATE(1512000000, 1, 6, 6), > + RK3368_CPUCLKL_RATE(1488000000, 1, 5, 5), > + RK3368_CPUCLKL_RATE(1416000000, 1, 5, 5), > + RK3368_CPUCLKL_RATE(1200000000, 1, 4, 4), > + RK3368_CPUCLKL_RATE(1008000000, 1, 4, 4), > + RK3368_CPUCLKL_RATE( 816000000, 1, 3, 3), > + RK3368_CPUCLKL_RATE( 696000000, 1, 2, 2), > + RK3368_CPUCLKL_RATE( 600000000, 1, 2, 2), > + RK3368_CPUCLKL_RATE( 408000000, 1, 1, 1), > + RK3368_CPUCLKL_RATE( 312000000, 1, 1, 1), > }; > > static struct rockchip_clk_branch rk3368_clk_branches[] __initdata = { >