[U-Boot] [PATCH] rockchip: video: fix mpixelclock in rockchip HDMI

Simon Glass sjg at chromium.org
Thu Feb 23 16:19:31 UTC 2017


Hi Nickey,

On 22 February 2017 at 23:56, Nickey.Yang <nickey.yang at rock-chips.com> wrote:
> Hi Simon,
>
>
>
> 在 2017年02月23日 11:52, Simon Glass 写道:
>>
>> Hi,
>>
>> On 11 January 2017 at 22:08, Simon Glass <sjg at chromium.org> wrote:
>>>
>>> On 28 December 2016 at 23:01, Nickey Yang <nickey.yang at rock-chips.com>
>>> wrote:
>>>>
>>>> Correct mpixelclock errors in rockchip_phy_config[] and
>>>> rockchip_mpll_cfg[].
>>>>
>>>> Signed-off-by: Nickey Yang <nickey.yang at rock-chips.com>
>>>> ---
>>>>   drivers/video/rockchip/rk_hdmi.c | 20 ++++++++++----------
>>>>   1 file changed, 10 insertions(+), 10 deletions(-)
>>>
>>> Applied to u-boot-rockchip, thanks!
>>
>> I only just noticed, but this patch breaks HDMI output on firefly. Can
>> you please take a look? What does this patch actually fix?
>>
>> Regards,
>> Simon
>>
>  You can add
>
>     printf("---YYS  mpll.cpce = %x \n",rockchip_mpll_cfg[i].cpce);
>     printf("---YYS  mpll.gmp = %x \n",rockchip_mpll_cfg[i].gmp);
>     printf("---YYS  mpll.curr = %x \n",rockchip_mpll_cfg[i].curr);
>
> in  hdmi_phy_configure(rk_hdmi.c   line 409), all of those value will be 0
> without this patch.
> We want to get those different value by near clock settings between
> rockchip_mpll_cfg[] in fact.

Yes it makes sense

>
> by the way,HDMI output on firefly will work well when reset this patch?

Yes it works when I revert.

Regards,
Simon


More information about the U-Boot mailing list