From c5b3aa120bf96ee169d07c33a2698499944a1624 Mon Sep 17 00:00:00 2001
From: Ian Romanick <idr@localhost.localdomain>
Date: Wed, 7 Jun 2006 17:46:23 -0700
Subject: [PATCH] Added a comment about some suspicious code.

---
 hw/xfree86/int10/generic.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/hw/xfree86/int10/generic.c b/hw/xfree86/int10/generic.c
index 0c3f2878c..512277122 100644
--- a/hw/xfree86/int10/generic.c
+++ b/hw/xfree86/int10/generic.c
@@ -105,6 +105,9 @@ xf86ExtendedInitInt10(int entityIndex, int Flags)
     pInt->scrnIndex = screen;
     base = INTPriv(pInt)->base = xnfalloc(SYS_BIOS);
 
+    /* FIXME: Shouldn't this be a failure case?  Leaving Tag as 0 seems like
+     * FIXME: an error
+     */
     pvp = xf86GetPciInfoForEntity(entityIndex);
     if (pvp != NULL) {
 	pInt->Tag = PCI_MAKE_TAG(PCI_MAKE_BUS(pvp->domain, pvp->bus),