drm/nv50: make nv50_mem_vm_{bind,unbind} operate only on vram
authorBen Skeggs <bskeggs@redhat.com>
Thu, 11 Feb 2010 00:23:30 +0000 (10:23 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Tue, 23 Feb 2010 03:41:16 +0000 (13:41 +1000)
GART is handled elsewhere, no reason to have the code for it here too.

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nouveau_mem.c

index 8f3a12f614ed27e4bcade0cd1f76863af59c8d48..04885d2fb15fa0cb7378783ccbd1e7936c5588c9 100644 (file)
@@ -285,53 +285,36 @@ nv50_mem_vm_bind_linear(struct drm_device *dev, uint64_t virt, uint32_t size,
                        uint32_t flags, uint64_t phys)
 {
        struct drm_nouveau_private *dev_priv = dev->dev_private;
-       struct nouveau_gpuobj **pgt;
-       unsigned psz, pfl, pages;
-
-       if (virt >= dev_priv->vm_gart_base &&
-           (virt + size) < (dev_priv->vm_gart_base + dev_priv->vm_gart_size)) {
-               psz = 12;
-               pgt = &dev_priv->gart_info.sg_ctxdma;
-               pfl = 0x21;
-               virt -= dev_priv->vm_gart_base;
-       } else
-       if (virt >= dev_priv->vm_vram_base &&
-           (virt + size) < (dev_priv->vm_vram_base + dev_priv->vm_vram_size)) {
-               psz = 16;
-               pgt = dev_priv->vm_vram_pt;
-               pfl = 0x01;
-               virt -= dev_priv->vm_vram_base;
-       } else {
-               NV_ERROR(dev, "Invalid address: 0x%16llx-0x%16llx\n",
-                        virt, virt + size - 1);
-               return -EINVAL;
-       }
+       unsigned pages;
 
-       pages = size >> psz;
+       virt -= dev_priv->vm_vram_base;
+       pages = size >> 16;
 
        dev_priv->engine.instmem.prepare_access(dev, true);
        if (flags & 0x80000000) {
                while (pages--) {
-                       struct nouveau_gpuobj *pt = pgt[virt >> 29];
-                       unsigned pte = ((virt & 0x1fffffffULL) >> psz) << 1;
+                       struct nouveau_gpuobj *pt =
+                               dev_priv->vm_vram_pt[virt >> 29];
+                       unsigned pte = ((virt & 0x1fffffffULL) >> 16) << 1;
 
                        nv_wo32(dev, pt, pte++, 0x00000000);
                        nv_wo32(dev, pt, pte++, 0x00000000);
 
-                       virt += (1 << psz);
+                       virt += (1 << 16);
                }
        } else {
                while (pages--) {
-                       struct nouveau_gpuobj *pt = pgt[virt >> 29];
-                       unsigned pte = ((virt & 0x1fffffffULL) >> psz) << 1;
+                       struct nouveau_gpuobj *pt =
+                               dev_priv->vm_vram_pt[virt >> 29];
+                       unsigned pte = ((virt & 0x1fffffffULL) >> 16) << 1;
                        unsigned offset_h = upper_32_bits(phys) & 0xff;
                        unsigned offset_l = lower_32_bits(phys);
 
-                       nv_wo32(dev, pt, pte++, offset_l | pfl);
+                       nv_wo32(dev, pt, pte++, offset_l | 1);
                        nv_wo32(dev, pt, pte++, offset_h | flags);
 
-                       phys += (1 << psz);
-                       virt += (1 << psz);
+                       phys += (1 << 16);
+                       virt += (1 << 16);
                }
        }
        dev_priv->engine.instmem.finish_access(dev);