From: Marek Olšák <maraeo@gmail.com>
Date: Sun, 11 Sep 2011 11:47:20 +0000 (+0200)
Subject: r600g: do not loop in radeon_family_from_device
X-Git-Tag: 062012170305~4524
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d26b49f87f533e8c3c2dab50c918b71ae45473a5;p=profile%2Fivi%2Fmesa.git

r600g: do not loop in radeon_family_from_device

Also move that function to r600_drm.c
---

diff --git a/src/gallium/winsys/r600/drm/Makefile.sources b/src/gallium/winsys/r600/drm/Makefile.sources
index 9798017..ce42ff6 100644
--- a/src/gallium/winsys/r600/drm/Makefile.sources
+++ b/src/gallium/winsys/r600/drm/Makefile.sources
@@ -1,6 +1,5 @@
 C_SOURCES := \
 	evergreen_hw_context.c \
-	radeon_pciid.c \
 	r600_bo.c \
 	r600_drm.c \
 	r600_hw_context.c
diff --git a/src/gallium/winsys/r600/drm/r600_drm.c b/src/gallium/winsys/r600/drm/r600_drm.c
index 7d5583f..fa90d96 100644
--- a/src/gallium/winsys/r600/drm/r600_drm.c
+++ b/src/gallium/winsys/r600/drm/r600_drm.c
@@ -183,6 +183,17 @@ static int radeon_drm_get_tiling(struct radeon *radeon)
 	}
 }
 
+static unsigned radeon_family_from_device(unsigned device)
+{
+	switch (device) {
+#define CHIPSET(pciid, name, family) case pciid: return CHIP_##family;
+#include "pci_ids/r600_pci_ids.h"
+#undef CHIPSET
+	default:
+		return CHIP_UNKNOWN;
+	}
+}
+
 struct radeon *radeon_create(struct radeon_winsys *ws)
 {
 	struct radeon *radeon = CALLOC_STRUCT(radeon);
diff --git a/src/gallium/winsys/r600/drm/r600_priv.h b/src/gallium/winsys/r600/drm/r600_priv.h
index 1e90189..2b8c85a 100644
--- a/src/gallium/winsys/r600/drm/r600_priv.h
+++ b/src/gallium/winsys/r600/drm/r600_priv.h
@@ -71,11 +71,6 @@ struct r600_bo {
 };
 
 /*
- * radeon_pciid.c
- */
-unsigned radeon_family_from_device(unsigned device);
-
-/*
  * r600_hw_context.c
  */
 void r600_context_bo_flush(struct r600_context *ctx, unsigned flush_flags,
diff --git a/src/gallium/winsys/r600/drm/radeon_pciid.c b/src/gallium/winsys/r600/drm/radeon_pciid.c
deleted file mode 100644
index 8757241..0000000
--- a/src/gallium/winsys/r600/drm/radeon_pciid.c
+++ /dev/null
@@ -1,51 +0,0 @@
-/*
- * Copyright 2010 Jerome Glisse <glisse@freedesktop.org>
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * on the rights to use, copy, modify, merge, publish, distribute, sub
- * license, and/or sell copies of the Software, and to permit persons to whom
- * the Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NON-INFRINGEMENT. IN NO EVENT SHALL
- * THE AUTHOR(S) AND/OR THEIR SUPPLIERS BE LIABLE FOR ANY CLAIM,
- * DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
- * OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE
- * USE OR OTHER DEALINGS IN THE SOFTWARE.
- *
- * Authors:
- *      Jerome Glisse
- */
-#include "r600_priv.h"
-
-struct pci_id {
-	unsigned	vendor;
-	unsigned	device;
-	unsigned	family;
-};
-
-static const struct pci_id radeon_pci_id[] = {
-#define CHIPSET(chip, name, family) { 0x1002, chip, CHIP_##family },
-#include "pci_ids/r600_pci_ids.h"
-	{0, 0},
-};
-
-unsigned radeon_family_from_device(unsigned device)
-{
-	unsigned i;
-
-	for (i = 0; ; i++) {
-		if (!radeon_pci_id[i].vendor)
-			return CHIP_UNKNOWN;
-		if (radeon_pci_id[i].device == device)
-			return radeon_pci_id[i].family;
-	}
-	return CHIP_UNKNOWN;
-}