Subject: Re: [PATCH v3 03/12] drm/rockchip: return a true clock rate to
 adjusted_mode
To: Mark yao <mark.yao@rock-chips.com>, heiko@sntech.de,
        linux-rockchip@lists.infradead.org
References: <1447904131-29328-1-git-send-email-zyw@rock-chips.com>
 <1447904131-29328-4-git-send-email-zyw@rock-chips.com>
 <564D756D.9050000@rock-chips.com>
Cc: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
        David Airlie <airlied@linux.ie>, linux-arm-kernel@lists.infradead.org
From: Chris Zhong <zyw@rock-chips.com>
Message-ID: <564D83DB.9040705@rock-chips.com>
Date: Thu, 19 Nov 2015 16:10:03 +0800
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.3.0
MIME-Version: 1.0
In-Reply-To: <564D756D.9050000@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: 901
Lines: 32


On 11/19/2015 03:08 PM, Mark yao wrote:
> 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.
Hi Mark
     Nice advice, I'll modify it in next version. Thanks.
>
> 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/