Re: radeonfb: strange blue lines after 'setterm -inversescreen on'

From: Geert Uytterhoeven <[email protected]>
Date: Fri, Apr 09, 2004 at 10:46:54AM +0200
> 
> So why can't you just use pseudo_palette[region->color], for truecolor and
> directcolor visuals? Saves some switches and bit manipulations.
> 
Ok, new try:


diff -Br -b -U 3 -N linux-2.6.5/drivers/video/aty/radeon_accel.c linux-2.6.5-new/drivers/video/aty/radeon_accel.c
--- linux-2.6.5/drivers/video/aty/radeon_accel.c	2004-02-18 04:58:34.000000000 +0100
+++ linux-2.6.5-new/drivers/video/aty/radeon_accel.c	2004-04-09 19:26:43.000000000 +0200
@@ -13,6 +13,9 @@
 		rinfo->dp_gui_master_cntl  /* contains, like GMC_DST_32BPP */
                 | GMC_BRUSH_SOLID_COLOR
                 | ROP3_P);
+        if (radeon_get_dstbpp(rinfo->depth) != DST_8BPP)
+		OUTREG(DP_BRUSH_FRGD_CLR, rinfo->pseudo_palette[region->color]);
+	else
 	OUTREG(DP_BRUSH_FRGD_CLR, region->color);
 	OUTREG(DP_WRITE_MSK, 0xffffffff);
 	OUTREG(DP_CNTL, (DST_X_LEFT_TO_RIGHT | DST_Y_TOP_TO_BOTTOM));

Everybody happy with this? Then please push it into the kernel!

Thanks,
Jurriaan
-- 
This Vintage Whine was fermented from two years worth of sour grapes harvested
in the northern regions of The British Isles. Its bitter aftertaste makes it
the ideal accompaniment for hard cheese and humble pie. Perfect for divorces,
suicides and all other antisocial occasions. Serve chilled.
	Inlay from Skyclad - Vintage Whine
Debian (Unstable) GNU/Linux 2.6.5-mm1 2x6062 bogomips load av: 1.79 3.65 3.06


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click