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 97980170e6bf199730dd0b444ec8d4f379a9e233..ce42ff69bba9ab6be557c3476e58b616ae2be586 100644 (file)
--- 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 7d5583fd287d2aadacc1b91ee0b21fad0892897d..fa90d969f95d525ccd4ab83e4d05d213f510f5fe 100644 (file)
--- 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 1e901897efd03e313bfa2ba527e9077221d66d3a..2b8c85a1431ba6da64439d99dc573d2bb4cf0a38 100644 (file)
--- a/src/gallium/winsys/r600/drm/r600_priv.h
+++ b/src/gallium/winsys/r600/drm/r600_priv.h
@@ -70,11 +70,6 @@ struct r600_bo {
        unsigned                        binding;
 };
 
-/*
- * radeon_pciid.c
- */
-unsigned radeon_family_from_device(unsigned device);
-
 /*
  * r600_hw_context.c
  */

diff --git a/src/gallium/winsys/r600/drm/radeon_pciid.c b/src/gallium/winsys/r600/drm/radeon_pciid.c
deleted file mode 100644 (file)
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;
-}