linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Liu Ying <Ying.Liu@freescale.com>
To: <dri-devel@lists.freedesktop.org>
Cc: <devicetree@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>, <linux@arm.linux.org.uk>,
	<kernel@pengutronix.de>, <p.zabel@pengutronix.de>,
	<thierry.reding@gmail.com>, <shawn.guo@linaro.org>,
	<mturquette@linaro.org>, <airlied@linux.ie>, <andyshrk@gmail.com>,
	<stefan.wahren@i2se.com>, <a.hajda@samsung.com>,
	<linux-kernel@vger.kernel.org>
Subject: [PATCH RFC v8 01/21] clk: divider: Correct parent clk round rate if no bestdiv is normally found
Date: Wed, 31 Dec 2014 16:23:19 +0800	[thread overview]
Message-ID: <1420014219-915-2-git-send-email-Ying.Liu@freescale.com> (raw)
In-Reply-To: <1420014219-915-1-git-send-email-Ying.Liu@freescale.com>

If no best divider is normally found, we will try to use the maximum divider.
We should not set the parent clock rate to be 1Hz by force for being rounded.
Instead, we should take the maximum divider as a base and calculate a correct
parent clock rate for being rounded.

Signed-off-by: Liu Ying <Ying.Liu@freescale.com>
---
v7->v8:
 * None.

v6->v7:
 * None.

v5->v6:
 * None.

v4->v5:
 * None.

v3->v4:
 * None.

v2->v3:
 * None.

v1->v2:
 * None.

 drivers/clk/clk-divider.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/clk/clk-divider.c b/drivers/clk/clk-divider.c
index c0a842b..f641d4b 100644
--- a/drivers/clk/clk-divider.c
+++ b/drivers/clk/clk-divider.c
@@ -311,7 +311,8 @@ static int clk_divider_bestdiv(struct clk_hw *hw, unsigned long rate,
 
 	if (!bestdiv) {
 		bestdiv = _get_maxdiv(divider);
-		*best_parent_rate = __clk_round_rate(__clk_get_parent(hw->clk), 1);
+		*best_parent_rate = __clk_round_rate(__clk_get_parent(hw->clk),
+						MULT_ROUND_UP(rate, bestdiv));
 	}
 
 	return bestdiv;
-- 
2.1.0


  reply	other threads:[~2014-12-31  8:19 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-31  8:23 [PATCH RFC v8 00/21] Add support for i.MX MIPI DSI DRM driver Liu Ying
2014-12-31  8:23 ` Liu Ying [this message]
2014-12-31  8:23 ` [PATCH RFC v8 02/21] of: Add vendor prefix for Himax Technologies Inc Liu Ying
2015-02-04 16:02   ` Rob Herring
2015-02-05  2:20     ` Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 03/21] of: Add vendor prefix for Truly Semiconductors Limited Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 04/21] ARM: imx6q: Add GPR3 MIPI muxing control register field shift bits definition Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 05/21] ARM: imx6q: clk: Add the video_27m clock Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 06/21] ARM: imx6q: clk: Change hdmi_isfr clock's parent to be " Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 07/21] ARM: imx6q: clk: Change hsi_tx clock to be a shared clock gate Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 08/21] ARM: imx6q: clk: Add support for mipi_core_cfg clock as " Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 09/21] ARM: dts: imx6qdl: Move existing MIPI DSI ports into a new 'ports' node Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 10/21] drm/dsi: Add a helper to get bits per pixel of MIPI DSI pixel format Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 11/21] Documentation: dt-bindings: Add bindings for Synopsys DW MIPI DSI DRM bridge driver Liu Ying
2015-02-05 10:10   ` Philipp Zabel
2015-02-06  8:13     ` Liu Ying
2015-02-11  7:21       ` Liu Ying
2015-02-11 13:00         ` Philipp Zabel
2015-02-11 14:09           ` Liu Ying
2015-02-11 15:23             ` Philipp Zabel
2015-02-12  2:14               ` Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 12/21] drm/bridge: Add Synopsys DesignWare MIPI DSI host controller driver Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 13/21] Documentation: dt-bindings: Add bindings for i.MX specific Synopsys DW MIPI DSI driver Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 14/21] drm: imx: Support Synopsys DesignWare MIPI DSI host controller Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 15/21] Documentation: dt-bindings: Add bindings for Himax HX8369A DRM panel driver Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 16/21] drm: panel: Add support for Himax HX8369A MIPI DSI panel Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 17/21] ARM: dtsi: imx6qdl: Add support for MIPI DSI host controller Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 18/21] ARM: dts: imx6qdl-sabresd: Add support for TRULY TFT480800-16-E MIPI DSI panel Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 19/21] ARM: imx_v6_v7_defconfig: Cleanup for imx drm being moved out of staging Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 20/21] ARM: imx_v6_v7_defconfig: Add support for MIPI DSI host controller Liu Ying
2014-12-31  8:23 ` [PATCH RFC v8 21/21] ARM: imx_v6_v7_defconfig: Add support for Himax HX8369A panel Liu Ying
2015-02-02  2:17 ` [PATCH RFC v8 00/21] Add support for i.MX MIPI DSI DRM driver Liu Ying
2015-02-02  5:34   ` Liu Ying

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=1420014219-915-2-git-send-email-Ying.Liu@freescale.com \
    --to=ying.liu@freescale.com \
    --cc=a.hajda@samsung.com \
    --cc=airlied@linux.ie \
    --cc=andyshrk@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mturquette@linaro.org \
    --cc=p.zabel@pengutronix.de \
    --cc=shawn.guo@linaro.org \
    --cc=stefan.wahren@i2se.com \
    --cc=thierry.reding@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).