summaryrefslogtreecommitdiff
path: root/sys/dev/pci
diff options
context:
space:
mode:
authorOwain Ainsworth <oga@cvs.openbsd.org>2009-05-24 01:40:59 +0000
committerOwain Ainsworth <oga@cvs.openbsd.org>2009-05-24 01:40:59 +0000
commit666eaba82d7e6355f951dadbe98fbf44cb6e03c0 (patch)
tree3d30d26256c22c8161fa08130de33cb132c82fc4 /sys/dev/pci
parentb1b471866bd1bcf489ec3badc2924fb42b49e2e3 (diff)
Instead of using magic numbers, define the bits that we set in gtt ptes
and use those defines.
Diffstat (limited to 'sys/dev/pci')
-rw-r--r--sys/dev/pci/agp_i810.c19
1 files changed, 15 insertions, 4 deletions
diff --git a/sys/dev/pci/agp_i810.c b/sys/dev/pci/agp_i810.c
index 3b1a18346ee..5bf42fe13af 100644
--- a/sys/dev/pci/agp_i810.c
+++ b/sys/dev/pci/agp_i810.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: agp_i810.c,v 1.52 2009/05/10 16:57:44 oga Exp $ */
+/* $OpenBSD: agp_i810.c,v 1.53 2009/05/24 01:40:58 oga Exp $ */
/*-
* Copyright (c) 2000 Doug Rabson
@@ -51,6 +51,16 @@
#define READ4(off) bus_space_read_4(isc->map->bst, isc->map->bsh, off)
#define WRITE4(off,v) bus_space_write_4(isc->map->bst, isc->map->bsh, off, v)
+/*
+ * Intel IGP gtt bits.
+ */
+/* PTE is enabled */
+#define INTEL_ENABLED 0x1
+/* I810/I815 only, memory is in dcache */
+#define INTEL_LOCAL 0x2
+/* Memory is snooped, must not be accessed through gtt from the cpu. */
+#define INTEL_COHERENT 0x6
+
enum {
CHIP_NONE = 0, /* not integrated graphics */
CHIP_I810 = 1, /* i810/i815 */
@@ -653,7 +663,8 @@ agp_i810_bind_memory(void *sc, struct agp_memory *mem, bus_size_t offset)
if (regval != (isc->gatt->ag_physical | 1)) {
printf("agp_i810_bind_memory: PGTBL_CTL is 0x%x - fixing\n",
regval);
- WRITE4(AGP_I810_PGTBL_CTL, isc->gatt->ag_physical | 1);
+ WRITE4(AGP_I810_PGTBL_CTL, isc->gatt->ag_physical |
+ INTEL_ENABLED);
}
if (mem->am_type == 2) {
@@ -671,7 +682,7 @@ agp_i810_bind_memory(void *sc, struct agp_memory *mem, bus_size_t offset)
return (EINVAL);
for (i = 0; i < mem->am_size; i += AGP_PAGE_SIZE)
- intagp_write_gtt(isc, i, i | 3);
+ intagp_write_gtt(isc, i, i | INTEL_ENABLED | INTEL_LOCAL);
mem->am_is_bound = 1;
return (0);
}
@@ -712,7 +723,7 @@ intagp_write_gtt(struct agp_i810_softc *isc, bus_size_t off, paddr_t v)
bus_size_t baseoff, wroff;
if (v != 0) {
- pte = v | 1;
+ pte = v | INTEL_ENABLED;
/* 965+ can do 36-bit addressing, add in the extra bits */
if (isc->chiptype == CHIP_I965 || isc->chiptype == CHIP_G4X)
pte |= (v & 0x0000000f00000000ULL) >> 28;