The gamma-correction lookup table values are 16:16:16 X Colors, where the MSBs are programmed into the hardware lookup table. Rather than compute values over the entire range [0,65536) (where values below 2^(16 - sigbits) will receive the same hardware value), compute values over the range [0,2^sigbits) and left shift by (16 - sigbits) into the MSBs.
Signed-off-by: Andy Ritger <[email protected]> Reviewed-by: Aaron Plattner <[email protected]> --- xrandr.c | 20 ++++++++++++-------- 1 files changed, 12 insertions(+), 8 deletions(-) diff --git a/xrandr.c b/xrandr.c index 045fe29..46d133f 100644 --- a/xrandr.c +++ b/xrandr.c @@ -1360,8 +1360,9 @@ set_gamma(void) /* * The hardware color lookup table has a number of significant - * bits equal to ffs(size) - 1; shift values so that they - * occupy the MSBs of the 16-bit X Color. + * bits equal to ffs(size) - 1; compute all values so that + * they are in the range [0,size) then shift the values so + * that they occupy the MSBs of the 16-bit X Color. */ shift = 16 - (ffs(size) - 1); @@ -1384,25 +1385,28 @@ set_gamma(void) for (i = 0; i < size; i++) { if (gammaRed == 1.0 && output->brightness == 1.0) - gamma->red[i] = (i << shift); + gamma->red[i] = i; else gamma->red[i] = dmin(pow((double)i/(double)(size - 1), gammaRed) * output->brightness, - 1.0) * 65535.0; + 1.0) * (double)(size - 1); + gamma->red[i] <<= shift; if (gammaGreen == 1.0 && output->brightness == 1.0) - gamma->green[i] = (i << shift); + gamma->green[i] = i; else gamma->green[i] = dmin(pow((double)i/(double)(size - 1), gammaGreen) * output->brightness, - 1.0) * 65535.0; + 1.0) * (double)(size - 1); + gamma->green[i] <<= shift; if (gammaBlue == 1.0 && output->brightness == 1.0) - gamma->blue[i] = (i << shift); + gamma->blue[i] = i; else gamma->blue[i] = dmin(pow((double)i/(double)(size - 1), gammaBlue) * output->brightness, - 1.0) * 65535.0; + 1.0) * (double)(size - 1); + gamma->blue[i] <<= shift; } XRRSetCrtcGamma(dpy, crtc->crtc.xid, gamma); -- 1.7.2.5 _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
