dummy driver obvious DacSpeed fix
Adam Jackson
ajax at nwnk.net
Thu Sep 17 07:57:09 PDT 2015
On Fri, 2015-09-11 at 21:15 +0700, Antoine Martin wrote:
> Hello,
>
> The dummy driver doesn't honour the "DacSpeed" config file option.
> It sets a local "maxClock" variable, but doesn't use it.
> This fixes it, and allowed me to run tests with resolutions up to
> 20000x20000:
>
> --- a/src/dummy_driver.c
> +++ b/src/dummy_driver.c
> @@ -391,7 +391,7 @@ DUMMYPreInit(ScrnInfoPtr pScrn, int flags)
> clockRanges->next = NULL;
> clockRanges->ClockMulFactor = 1;
> clockRanges->minClock = 11000; /* guessed ยงยงยง */
> - clockRanges->maxClock = 300000;
> + clockRanges->maxClock = maxClock;
> clockRanges->clockIndex = -1; /* programmable */
> clockRanges->interlaceAllowed = TRUE;
> clockRanges->doubleScanAllowed = TRUE;
>
> Note: this would change the default from 300000 to 230000 (different
> values hardcoded in two places).
> To keep the current default, just update the (currently unused) value
> defined at the top of the function to read:
> int maxClock = 300000;
>
> Please apply. (I would rather avoid the git pull dance for this
> trivial fix)
I pushed a version of this that also fixes the maxClock default.
Thanks!
- ajax
More information about the xorg-devel
mailing list