[PATCH 3/3] xrandr: compute gamma-correction in [0,2^sigbits)

Andy Ritger aritger at nvidia.com
Mon Aug 13 16:25:12 PDT 2012


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 <aritger at nvidia.com>
Reviewed-by: Aaron Plattner <aplattner at nvidia.com>
---
 xrandr.c |   20 ++++++++++++--------
 1 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/xrandr.c b/xrandr.c
index 5709ded..9bea655 100644
--- a/xrandr.c
+++ b/xrandr.c
@@ -1349,8 +1349,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);
 
@@ -1373,25 +1374,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



More information about the xorg-devel mailing list