From: Heiko Stuebner <heiko@sntech.de>
To: Chris Zhong <zyw@rock-chips.com>
Cc: dianders@chromium.org, linux-rockchip@lists.infradead.org,
zhengxing@rock-chips.com, ykk@rock-chips.com,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@codeaurora.org>,
linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [v2 PATCH 1/3] clk: rockchip: Fractional dividers can't set parent rates
Date: Sun, 04 Sep 2016 23:50:21 +0200 [thread overview]
Message-ID: <3040422.nFj5Be1ODN@phil> (raw)
In-Reply-To: <1472786785-2393-1-git-send-email-zyw@rock-chips.com>
Am Donnerstag, 1. September 2016, 20:26:23 CEST schrieb Chris Zhong:
> From: Douglas Anderson <dianders@chromium.org>
>
> Currently the fractional divider clock time can't handle the
> CLK_SET_RATE_PARENT flag. This is because, unlike normal dividers,
> there is no clk_divider_bestdiv() function to try speeding up the parent
> to see if it helps things.
>
> Eventually someone could try to figure out how to make fractional
> dividers able to use CLK_SET_RATE_PARENT, but until they do let's not
> confuse the common clock framework (and anyone using it) by setting the
> flag.
>
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
> Signed-off-by: Chris Zhong <zyw@rock-chips.com>
applied to my clk-branch for 4.9, after changing the subject to
clk: rockchip: drop CLK_SET_RATE_PARENT from rk3399 fractional dividers
Heiko
WARNING: multiple messages have this Message-ID (diff)
From: heiko@sntech.de (Heiko Stuebner)
To: linux-arm-kernel@lists.infradead.org
Subject: [v2 PATCH 1/3] clk: rockchip: Fractional dividers can't set parent rates
Date: Sun, 04 Sep 2016 23:50:21 +0200 [thread overview]
Message-ID: <3040422.nFj5Be1ODN@phil> (raw)
In-Reply-To: <1472786785-2393-1-git-send-email-zyw@rock-chips.com>
Am Donnerstag, 1. September 2016, 20:26:23 CEST schrieb Chris Zhong:
> From: Douglas Anderson <dianders@chromium.org>
>
> Currently the fractional divider clock time can't handle the
> CLK_SET_RATE_PARENT flag. This is because, unlike normal dividers,
> there is no clk_divider_bestdiv() function to try speeding up the parent
> to see if it helps things.
>
> Eventually someone could try to figure out how to make fractional
> dividers able to use CLK_SET_RATE_PARENT, but until they do let's not
> confuse the common clock framework (and anyone using it) by setting the
> flag.
>
> Signed-off-by: Douglas Anderson <dianders@chromium.org>
> Signed-off-by: Chris Zhong <zyw@rock-chips.com>
applied to my clk-branch for 4.9, after changing the subject to
clk: rockchip: drop CLK_SET_RATE_PARENT from rk3399 fractional dividers
Heiko
next prev parent reply other threads:[~2016-09-04 21:50 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-02 3:26 [v2 PATCH 1/3] clk: rockchip: Fractional dividers can't set parent rates Chris Zhong
2016-09-02 3:26 ` Chris Zhong
2016-09-02 3:26 ` [v2 PATCH 2/3] clk: rockchip: add dclk_vop_frac ids for vop Chris Zhong
2016-09-02 20:43 ` Heiko Stübner
2016-09-02 20:58 ` Chris Zhong
2016-09-02 20:58 ` Chris Zhong
[not found] ` <b2d57b28-19dc-c55c-39b7-4b01f9365967-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2016-09-02 21:14 ` Heiko Stübner
2016-09-02 21:14 ` Heiko Stübner
[not found] ` <1472786785-2393-2-git-send-email-zyw-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2016-09-04 21:50 ` Heiko Stuebner
2016-09-04 21:50 ` Heiko Stuebner
2016-09-02 3:26 ` [v2 PATCH 3/3] clk: rockchip: use the dclk_vop_frac clock ids Chris Zhong
2016-09-02 3:26 ` Chris Zhong
2016-09-04 21:51 ` Heiko Stuebner
2016-09-04 21:51 ` Heiko Stuebner
2016-09-04 21:50 ` Heiko Stuebner [this message]
2016-09-04 21:50 ` [v2 PATCH 1/3] clk: rockchip: Fractional dividers can't set parent rates Heiko Stuebner
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=3040422.nFj5Be1ODN@phil \
--to=heiko@sntech.de \
--cc=dianders@chromium.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mturquette@baylibre.com \
--cc=sboyd@codeaurora.org \
--cc=ykk@rock-chips.com \
--cc=zhengxing@rock-chips.com \
--cc=zyw@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.