From fa3c59136e9dd788ee7d3689b6cb89dd27040a9e Mon Sep 17 00:00:00 2001
From: Corbin Simpson <MostAwesomeDude@gmail.com>
Date: Mon, 2 Feb 2009 16:13:41 -0800
Subject: [PATCH] r300: Take care of some XXXes.

---
 src/gallium/drivers/r300/r300_chipset.c |  3 ++-
 src/gallium/drivers/r300/r300_screen.c  | 11 +++++------
 src/gallium/drivers/r300/r300_winsys.h  |  1 -
 3 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/src/gallium/drivers/r300/r300_chipset.c b/src/gallium/drivers/r300/r300_chipset.c
index 4c84be26efa..b0a7fe7d213 100644
--- a/src/gallium/drivers/r300/r300_chipset.c
+++ b/src/gallium/drivers/r300/r300_chipset.c
@@ -337,7 +337,8 @@ void r300_parse_chipset(struct r300_capabilities* caps)
             break;
 
         default:
-            /* XXX not an r300?! */
+            debug_printf("r300: Warning: Unknown chipset 0x%x\n",
+                caps->pci_id);
             break;
     }
 
diff --git a/src/gallium/drivers/r300/r300_screen.c b/src/gallium/drivers/r300/r300_screen.c
index 99dcf38f431..8e77e0ddd97 100644
--- a/src/gallium/drivers/r300/r300_screen.c
+++ b/src/gallium/drivers/r300/r300_screen.c
@@ -81,12 +81,11 @@ static int r300_get_param(struct pipe_screen* pscreen, int param)
             /* IN THEORY */
             return 0;
         case PIPE_CAP_TWO_SIDED_STENCIL:
-            /* IN THEORY */
-            /* if (r300screen->is_r500) {
-             * return 1;
-             * } else {
-             * return 0;
-             * } */
+            if (r300screen->is_r500) {
+                return 1;
+            } else {
+                return 0;
+            }
             return 0;
         case PIPE_CAP_GLSL:
             /* IN THEORY */
diff --git a/src/gallium/drivers/r300/r300_winsys.h b/src/gallium/drivers/r300/r300_winsys.h
index 867d65b7de1..5a3a2128927 100644
--- a/src/gallium/drivers/r300/r300_winsys.h
+++ b/src/gallium/drivers/r300/r300_winsys.h
@@ -64,7 +64,6 @@ struct r300_winsys {
            int line);
 
     /* Write a dword to the command buffer. */
-    /* XXX is this an okay name for this handle? */
     void (*write_cs_dword)(struct radeon_cs* cs, uint32_t dword);
 
     /* Write a relocated dword to the command buffer. */
-- 
2.30.2