drm/nv04-nv10: Don't re-enable FIFO access multiple times after IRQ dispatch.
authorFrancisco Jerez <currojerez@riseup.net>
Sat, 20 Nov 2010 13:43:51 +0000 (14:43 +0100)
committerFrancisco Jerez <currojerez@riseup.net>
Wed, 8 Dec 2010 02:00:20 +0000 (03:00 +0100)
nvxx_graph_isr is already taking care of it. In some cases this
could've made you miss PGRAPH interrupts (e.g. when you were supposed
to get several IRQs of the same kind in a row).

Signed-off-by: Francisco Jerez <currojerez@riseup.net>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nv04_graph.c
drivers/gpu/drm/nouveau/nv10_graph.c

index 0bc616d35eb6ac11649e1c4b5d22dc2185974b17..af75015068d6bd89477d81f5a5b5aabae41c9a3b 100644 (file)
@@ -373,7 +373,6 @@ nv04_graph_context_switch(struct drm_device *dev)
        struct nouveau_channel *chan = NULL;
        int chid;
 
-       pgraph->fifo_access(dev, false);
        nouveau_wait_for_idle(dev);
 
        /* If previous context is valid, we need to save it */
@@ -384,8 +383,6 @@ nv04_graph_context_switch(struct drm_device *dev)
        chan = dev_priv->channels.ptr[chid];
        if (chan)
                nv04_graph_load_context(chan);
-
-       pgraph->fifo_access(dev, true);
 }
 
 static uint32_t *ctx_reg(struct graph_state *ctx, uint32_t reg)
index 536b39e4a9e2661f31cf386e10a37c1dea310d9f..8c92edb7bbcd1833dcc2c823f230793b243e86e1 100644 (file)
@@ -794,11 +794,9 @@ static void
 nv10_graph_context_switch(struct drm_device *dev)
 {
        struct drm_nouveau_private *dev_priv = dev->dev_private;
-       struct nouveau_pgraph_engine *pgraph = &dev_priv->engine.graph;
        struct nouveau_channel *chan = NULL;
        int chid;
 
-       pgraph->fifo_access(dev, false);
        nouveau_wait_for_idle(dev);
 
        /* If previous context is valid, we need to save it */
@@ -809,8 +807,6 @@ nv10_graph_context_switch(struct drm_device *dev)
        chan = dev_priv->channels.ptr[chid];
        if (chan && chan->pgraph_ctx)
                nv10_graph_load_context(chan);
-
-       pgraph->fifo_access(dev, true);
 }
 
 #define NV_WRITE_CTX(reg, val) do { \
@@ -980,8 +976,6 @@ nv17_graph_mthd_lma_window(struct nouveau_channel *chan,
        struct drm_device *dev = chan->dev;
        struct graph_state *ctx = chan->pgraph_ctx;
        struct pipe_state *pipe = &ctx->pipe_state;
-       struct drm_nouveau_private *dev_priv = dev->dev_private;
-       struct nouveau_pgraph_engine *pgraph = &dev_priv->engine.graph;
        uint32_t pipe_0x0040[1], pipe_0x64c0[8], pipe_0x6a80[3], pipe_0x6ab0[3];
        uint32_t xfmode0, xfmode1;
        int i;
@@ -1048,8 +1042,6 @@ nv17_graph_mthd_lma_window(struct nouveau_channel *chan,
 
        nouveau_wait_for_idle(dev);
 
-       pgraph->fifo_access(dev, true);
-
        return 0;
 }
 
@@ -1058,8 +1050,6 @@ nv17_graph_mthd_lma_enable(struct nouveau_channel *chan,
                           u32 class, u32 mthd, u32 data)
 {
        struct drm_device *dev = chan->dev;
-       struct drm_nouveau_private *dev_priv = dev->dev_private;
-       struct nouveau_pgraph_engine *pgraph = &dev_priv->engine.graph;
 
        nouveau_wait_for_idle(dev);
 
@@ -1068,8 +1058,6 @@ nv17_graph_mthd_lma_enable(struct nouveau_channel *chan,
        nv_wr32(dev, 0x004006b0,
                nv_rd32(dev, 0x004006b0) | 0x8 << 24);
 
-       pgraph->fifo_access(dev, true);
-
        return 0;
 }