Re: [PATCH 2/2][dizzy-next] linux-imx: add clock patch for T1.0 version of i.MX6Q

[Gary Bisson <bisson.gary@gmail.com>]

Hi Fabio,

On 12/03/2014 10:46 AM, Fabio Estevam wrote:
> Hi Gary,
>
>> @@ -0,0 +1,33 @@
>> +From ffcd8b98f6ba3a2b7a4aecc1e5e454de5fa5c4c3 Mon Sep 17 00:00:00 2001
>> +From: Gary Bisson <bisson.gary@gmail.com>
>> +Date: Tue, 2 Dec 2014 22:11:14 -0800
>> +Subject: [PATCH] ARM: clk-imx6q: fix video divider for revision 1.0 of i.MX6q
>> +
>> +As post dividers do not work on i.MX6Q revision 1.0 they must be fixed
>> +to 1. As the table index was wrong, a divider a of 4 could still be
>> +requested which implied the clock not to be set properly. This is the
>> +root cause of the HDMI not working at high resolution on T1.0 version of
>> +the SoC, giving the following error:
>> +mxc_sdc_fb fb.27: timeout when waiting for flip irq
>> +
>> +Signed-off-by: Gary Bisson <bisson.gary@gmail.com>
>> +---
>> + arch/arm/mach-imx/clk-imx6q.c | 2 +-
>> + 1 file changed, 1 insertion(+), 1 deletion(-)
>> +
>> +diff --git a/arch/arm/mach-imx/clk-imx6q.c b/arch/arm/mach-imx/clk-imx6q.c
>> +index e09b1e8..1e5c410 100644
>> +--- a/arch/arm/mach-imx/clk-imx6q.c
>> ++++ b/arch/arm/mach-imx/clk-imx6q.c
>> +@@ -279,7 +279,7 @@ static void __init imx6q_clocks_init(struct device_node *ccm_node)
>> +               post_div_table[1].div = 1;
>> +               post_div_table[2].div = 1;
>> +               video_div_table[1].div = 1;
>> +-              video_div_table[2].div = 1;
>> ++              video_div_table[3].div = 1;
> Could you please fix the mainline kernel and send a patch to the
> linux-arm-kernel mailing list?
>
> Please also Cc shawn.guo@linaro.org.
>
> Regards,
>
> Fabio Estevam
>
> Thanks
As explained on my coverletter, this patch fixes HDMI up to 1080p30 but 
doesn't work on 1080p60 which implies that I cannot really test it on 
mainline.
I plan on working on this issue later this week and only then plan on 
submitting it to the mainline.

Do you think it'd be better to add this one first today and later on add 
the one that allows the testing?

Thanks,
Gary