From f62615054d51fcba7cd2d9faa95b7646de0f2819 Mon Sep 17 00:00:00 2001 From: Sebastien Bourdeauducq Date: Mon, 18 Nov 2013 20:43:30 +0100 Subject: [PATCH] videomixer: stick to 20x PLL configuration as 10x causes traditional spartan6 weirdness --- software/videomixer/pll.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/software/videomixer/pll.c b/software/videomixer/pll.c index fd49393f..1a8689e4 100644 --- a/software/videomixer/pll.c +++ b/software/videomixer/pll.c @@ -56,6 +56,15 @@ static void program_data(const unsigned short *data) void pll_config_for_clock(int freq) { + /* + * FIXME: + * 10x configuration causes random IDELAY lockups (at high frequencies it seems) + * 20x configuration seems to always work, even with overclocked VCO + * Reproducible both with DRP and initial reconfiguration. + * Until this spartan6 weirdness is sorted out, just stick to 20x. + */ + program_data(pll_config_20x); +#ifdef XILINX_SPARTAN6_WORKS_AMAZINGLY_WELL if(freq < 2000) printf("Frequency too low for PLLs\n"); else if(freq < 4500) @@ -64,6 +73,7 @@ void pll_config_for_clock(int freq) program_data(pll_config_10x); else printf("Frequency too high for PLLs\n"); +#endif } void pll_dump(void) -- 2.30.2