From: zhangqing <zhangqing@rock-chips.com>
To: Heiko Stuebner <heiko@sntech.de>,
mturquette@baylibre.com, sboyd@codeaurora.org
Cc: linux-clk@vger.kernel.org, linux-rockchip@lists.infradead.org,
xf@rock-chips.com
Subject: Re: [PATCH 1/4] clk: rockchip: fix cpuclk mux bit of big cpu-cluster
Date: Thu, 21 Jan 2016 01:44:57 -0800 [thread overview]
Message-ID: <56A0A899.9080908@rock-chips.com> (raw)
In-Reply-To: <1453326560-20475-1-git-send-email-heiko@sntech.de>
hi:
On 01/20/2016 01:49 PM, Heiko Stuebner wrote:
> Both clusters have their mux bit in bit 7 of their respective register.
> For whatever reason the big cluster currently lists bit 15 which is
> definitly wrong.
>
> Fixes: 3536c97a52db ("clk: rockchip: add rk3368 clock controller")
> Reported-by: Zhang Qing <zhangqing@rock-chips.com>
> Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Reviewed-by: zhangqing <zhangqing@rock-chips.com>
> ---
> I plan to include them into my clk-fixes branch, so posted for reference
> and possible objections ;-)
>
> drivers/clk/rockchip/clk-rk3368.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/clk/rockchip/clk-rk3368.c b/drivers/clk/rockchip/clk-rk3368.c
> index 21f3ea9..f6667b8 100644
> --- a/drivers/clk/rockchip/clk-rk3368.c
> +++ b/drivers/clk/rockchip/clk-rk3368.c
> @@ -165,7 +165,7 @@ static const struct rockchip_cpuclk_reg_data rk3368_cpuclkb_data = {
> .core_reg = RK3368_CLKSEL_CON(0),
> .div_core_shift = 0,
> .div_core_mask = 0x1f,
> - .mux_core_shift = 15,
> + .mux_core_shift = 7,
> };
>
> static const struct rockchip_cpuclk_reg_data rk3368_cpuclkl_data = {
>
prev parent reply other threads:[~2016-01-21 1:48 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-20 21:49 [PATCH 1/4] clk: rockchip: fix cpuclk mux bit of big cpu-cluster Heiko Stuebner
2016-01-20 21:49 ` [PATCH 2/4] clk: rockchip: fix rk3368 cpuclk core dividers Heiko Stuebner
2016-01-21 9:42 ` zhangqing
2016-01-20 21:49 ` [PATCH 3/4] clk: rockchip: rk3368: fix parents of video encoder/decoder Heiko Stuebner
2016-01-21 9:43 ` zhangqing
2016-01-20 21:49 ` [PATCH 4/4] clk: rockchip: rk3368: fix hdmi_cec gate-register Heiko Stuebner
2016-01-21 9:44 ` zhangqing
2016-01-20 22:06 ` [PATCH 1/4] clk: rockchip: fix cpuclk mux bit of big cpu-cluster Stephen Boyd
2016-01-20 22:25 ` Heiko Stuebner
2016-01-20 22:25 ` Heiko Stuebner
2016-01-21 9:44 ` zhangqing [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56A0A899.9080908@rock-chips.com \
--to=zhangqing@rock-chips.com \
--cc=heiko@sntech.de \
--cc=linux-clk@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mturquette@baylibre.com \
--cc=sboyd@codeaurora.org \
--cc=xf@rock-chips.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.