[PATCH] savage: Fix ambiguity in SavageLoadPaletteSavage4

Tormod Volden lists.tormod at gmail.com
Sun Jan 23 06:49:58 PST 2011


From: Tormod Volden <debian.tormod at gmail.com>

It looks like "!" was meant for logic negation and the parentheses
were wrong.

I don't have a Savage 4 so I can't check if this fixes or breaks
anything.
---
 src/savage_driver.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/savage_driver.c b/src/savage_driver.c
index b0b26b9..38db9f6 100644
--- a/src/savage_driver.c
+++ b/src/savage_driver.c
@@ -4295,7 +4295,7 @@ void SavageLoadPaletteSavage4(ScrnInfoPtr pScrn, int numColors, int *indicies,
     VerticalRetraceWait();
 
     for (i=0; i<numColors; i++) {
-          if (!(inStatus1()) & 0x08)
+          if (!(inStatus1() & 0x08))
   	    VerticalRetraceWait(); 
 	index = indicies[i];
 	VGAOUT8(0x3c8, index);
-- 
1.7.0.4



More information about the xorg-devel mailing list