Message-ID: <564D756D.9050000@rock-chips.com>
Date: Thu, 19 Nov 2015 15:08:29 +0800
From: Mark yao <mark.yao@rock-chips.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.8.0
MIME-Version: 1.0
To: Chris Zhong <zyw@rock-chips.com>, heiko@sntech.de,
        linux-rockchip@lists.infradead.org
CC: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
        David Airlie <airlied@linux.ie>, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v3 03/12] drm/rockchip: return a true clock rate to adjusted_mode
References: <1447904131-29328-1-git-send-email-zyw@rock-chips.com> <1447904131-29328-4-git-send-email-zyw@rock-chips.com>
In-Reply-To: <1447904131-29328-4-git-send-email-zyw@rock-chips.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 741
Lines: 22

On 2015年11月19日 11:35, Chris Zhong wrote:
> +
> +	/*
> +	 * Sometimes the clock driver can not set a accurate clock_rate for vop,
> +	 * get the true rate of vop_dclk and set it back to adjusted_mode.
> +	 */
> +	adjusted_mode->clock = clk_get_rate(vop->dclk) / 1000;
>   out:
>   	ret_clk = clk_enable(vop->dclk);
>   	if (ret_clk < 0) {
Hi Chris
      How about use clk_round_rate to check the clock rate first?
      I think we can check it on vop_crtc_mode_fixup.

Thanks.
- Ｍark Yao

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/