From: bisson.gary@gmail.com (Gary Bisson)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH][imx/fixes] ARM: clk-imx6q: fix video divider for rev T0 1.0
Date: Wed, 3 Dec 2014 11:48:05 -0800 [thread overview]
Message-ID: <1417636085-14822-1-git-send-email-bisson.gary@gmail.com> (raw)
The post dividers do not work on i.MX6Q rev T0 1.0 so 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 rev T0 1.0 of
the SoC, giving the following error:
mxc_sdc_fb fb.27: timeout when waiting for flip irq
See this thread for details:
https://lists.yoctoproject.org/pipermail/meta-freescale/2014-December/thread.html#11776
Signed-off-by: Gary Bisson <bisson.gary@gmail.com>
---
Hi,
This patch has been tested with a Sabrelite rev D (5-9-12).
However I've realized this patch doesn't fix the HDMI at 1080p60 but only
resolutions up to 1080p30. As mainline kernel is forcing the highest
resolution supported by the monitor it might not work on 1080p60-enable
displays. This latter issue is being investigated.
This patch is being integrated into meta-fsl-arm. Fabio suggested to
submit this patch already as clearly the array index is currently
wrong and "Better to have some HDMI then no HDMI at all":
https://lists.yoctoproject.org/pipermail/meta-freescale/2014-December/011796.html
Thanks,
Gary
---
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 4e79da7..2aa1b67 100644
--- a/arch/arm/mach-imx/clk-imx6q.c
+++ b/arch/arm/mach-imx/clk-imx6q.c
@@ -144,7 +144,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;
};
clk[IMX6QDL_PLL1_BYPASS_SRC] = imx_clk_mux("pll1_bypass_src", base + 0x00, 14, 2, pll_bypass_src_sels, ARRAY_SIZE(pll_bypass_src_sels));
--
2.1.3
next reply other threads:[~2014-12-03 19:48 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-03 19:48 Gary Bisson [this message]
2014-12-03 22:06 ` [PATCH][imx/fixes] ARM: clk-imx6q: fix video divider for rev T0 1.0 Fabio Estevam
2014-12-03 23:00 ` Gary Bisson
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=1417636085-14822-1-git-send-email-bisson.gary@gmail.com \
--to=bisson.gary@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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).