diff mbox

[1/2] drm/nouveau: Fix overlap while zeroing zcull regions

Message ID 1402619670-21368-1-git-send-email-pierre.morrow@free.fr (mailing list archive)
State New, archived
Headers show

Commit Message

Pierre Moreau June 13, 2014, 12:34 a.m. UTC
The specified stride was not correct, resulting in erases overlapping and part
of the zcull regions being not erased at all.

Signed-off-by: Pierre Moreau <pierre.morrow@free.fr>
---
 drivers/gpu/drm/nouveau/core/engine/graph/nv50.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Ben Skeggs June 13, 2014, 2:08 a.m. UTC | #1
On Fri, Jun 13, 2014 at 10:34 AM, Pierre Moreau <pierre.morrow@free.fr> wrote:
> The specified stride was not correct, resulting in erases overlapping and part
> of the zcull regions being not erased at all.
Hey Pierre,

Thanks, I've merged both patches.

>
> Signed-off-by: Pierre Moreau <pierre.morrow@free.fr>
> ---
>  drivers/gpu/drm/nouveau/core/engine/graph/nv50.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/gpu/drm/nouveau/core/engine/graph/nv50.c b/drivers/gpu/drm/nouveau/core/engine/graph/nv50.c
> index 2c7809e..a36adcd 100644
> --- a/drivers/gpu/drm/nouveau/core/engine/graph/nv50.c
> +++ b/drivers/gpu/drm/nouveau/core/engine/graph/nv50.c
> @@ -991,10 +991,10 @@ nv50_graph_init(struct nouveau_object *object)
>
>         /* zero out zcull regions */
>         for (i = 0; i < 8; i++) {
> -               nv_wr32(priv, 0x402c20 + (i * 8), 0x00000000);
> -               nv_wr32(priv, 0x402c24 + (i * 8), 0x00000000);
> -               nv_wr32(priv, 0x402c28 + (i * 8), 0x00000000);
> -               nv_wr32(priv, 0x402c2c + (i * 8), 0x00000000);
> +               nv_wr32(priv, 0x402c20 + (i * 0x10), 0x00000000);
> +               nv_wr32(priv, 0x402c24 + (i * 0x10), 0x00000000);
> +               nv_wr32(priv, 0x402c28 + (i * 0x10), 0x00000000);
> +               nv_wr32(priv, 0x402c2c + (i * 0x10), 0x00000000);
>         }
>         return 0;
>  }
> --
> 2.0.0
>
> _______________________________________________
> Nouveau mailing list
> Nouveau@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/nouveau
diff mbox

Patch

diff --git a/drivers/gpu/drm/nouveau/core/engine/graph/nv50.c b/drivers/gpu/drm/nouveau/core/engine/graph/nv50.c
index 2c7809e..a36adcd 100644
--- a/drivers/gpu/drm/nouveau/core/engine/graph/nv50.c
+++ b/drivers/gpu/drm/nouveau/core/engine/graph/nv50.c
@@ -991,10 +991,10 @@  nv50_graph_init(struct nouveau_object *object)
 
 	/* zero out zcull regions */
 	for (i = 0; i < 8; i++) {
-		nv_wr32(priv, 0x402c20 + (i * 8), 0x00000000);
-		nv_wr32(priv, 0x402c24 + (i * 8), 0x00000000);
-		nv_wr32(priv, 0x402c28 + (i * 8), 0x00000000);
-		nv_wr32(priv, 0x402c2c + (i * 8), 0x00000000);
+		nv_wr32(priv, 0x402c20 + (i * 0x10), 0x00000000);
+		nv_wr32(priv, 0x402c24 + (i * 0x10), 0x00000000);
+		nv_wr32(priv, 0x402c28 + (i * 0x10), 0x00000000);
+		nv_wr32(priv, 0x402c2c + (i * 0x10), 0x00000000);
 	}
 	return 0;
 }