drm/nouveau/kms/nv50-: convert core head_mode() to new push macros

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>

diff --git a/drivers/gpu/drm/nouveau/dispnv50/head.h b/drivers/gpu/drm/nouveau/dispnv50/head.h
index dfd4d2c..2d376e3 100644 (file)
--- a/drivers/gpu/drm/nouveau/dispnv50/head.h
+++ b/drivers/gpu/drm/nouveau/dispnv50/head.h
@@ -26,7 +26,7 @@ void nv50_head_flush_clr(struct nv50_head *head,
 
 struct nv50_head_func {
        int (*view)(struct nv50_head *, struct nv50_head_atom *);
-       void (*mode)(struct nv50_head *, struct nv50_head_atom *);
+       int (*mode)(struct nv50_head *, struct nv50_head_atom *);
        bool (*olut)(struct nv50_head *, struct nv50_head_atom *, int);
        bool olut_identity;
        int  olut_size;
@@ -51,7 +51,7 @@ struct nv50_head_func {
 
 extern const struct nv50_head_func head507d;
 int head507d_view(struct nv50_head *, struct nv50_head_atom *);
-void head507d_mode(struct nv50_head *, struct nv50_head_atom *);
+int head507d_mode(struct nv50_head *, struct nv50_head_atom *);
 bool head507d_olut(struct nv50_head *, struct nv50_head_atom *, int);
 void head507d_core_calc(struct nv50_head *, struct nv50_head_atom *);
 void head507d_core_clr(struct nv50_head *);
@@ -68,7 +68,7 @@ extern const struct nv50_head_func head827d;
 
 extern const struct nv50_head_func head907d;
 int head907d_view(struct nv50_head *, struct nv50_head_atom *);
-void head907d_mode(struct nv50_head *, struct nv50_head_atom *);
+int head907d_mode(struct nv50_head *, struct nv50_head_atom *);
 bool head907d_olut(struct nv50_head *, struct nv50_head_atom *, int);
 void head907d_olut_set(struct nv50_head *, struct nv50_head_atom *);
 void head907d_olut_clr(struct nv50_head *);

diff --git a/drivers/gpu/drm/nouveau/dispnv50/head507d.c b/drivers/gpu/drm/nouveau/dispnv50/head507d.c
index df0bc70..516c075 100644 (file)
--- a/drivers/gpu/drm/nouveau/dispnv50/head507d.c
+++ b/drivers/gpu/drm/nouveau/dispnv50/head507d.c
@@ -288,28 +288,28 @@ head507d_olut(struct nv50_head *head, struct nv50_head_atom *asyh, int size)
        return true;
 }
 
-void
+int
 head507d_mode(struct nv50_head *head, struct nv50_head_atom *asyh)
 {
-       struct nv50_dmac *core = &nv50_disp(head->base.base.dev)->core->chan;
+       struct nvif_push *push = nv50_disp(head->base.base.dev)->core->chan.push;
        struct nv50_head_mode *m = &asyh->mode;
-       u32 *push;
-       if ((push = evo_wait(core, 13))) {
-               evo_mthd(push, 0x0804 + (head->base.index * 0x400), 2);
-               evo_data(push, 0x00800000 | m->clock);
-               evo_data(push, m->interlace ? 0x00000002 : 0x00000000);
-               evo_mthd(push, 0x0810 + (head->base.index * 0x400), 7);
-               evo_data(push, 0x00000000);
-               evo_data(push, m->v.active  << 16 | m->h.active );
-               evo_data(push, m->v.synce   << 16 | m->h.synce  );
-               evo_data(push, m->v.blanke  << 16 | m->h.blanke );
-               evo_data(push, m->v.blanks  << 16 | m->h.blanks );
-               evo_data(push, m->v.blank2e << 16 | m->v.blank2s);
-               evo_data(push, asyh->mode.v.blankus);
-               evo_mthd(push, 0x082c + (head->base.index * 0x400), 1);
-               evo_data(push, 0x00000000);
-               evo_kick(push, core);
-       }
+       const int i = head->base.index;
+       int ret;
+
+       if ((ret = PUSH_WAIT(push, 13)))
+               return ret;
+
+       PUSH_NVSQ(push, NV507D, 0x0804 + (i * 0x400), 0x00800000 | m->clock,
+                               0x0808 + (i * 0x400), m->interlace ? 0x00000002 : 0x00000000);
+       PUSH_NVSQ(push, NV507D, 0x0810 + (i * 0x400), 0x00000000,
+                               0x0814 + (i * 0x400), m->v.active  << 16 | m->h.active,
+                               0x0818 + (i * 0x400), m->v.synce   << 16 | m->h.synce,
+                               0x081c + (i * 0x400), m->v.blanke  << 16 | m->h.blanke,
+                               0x0820 + (i * 0x400), m->v.blanks  << 16 | m->h.blanks,
+                               0x0824 + (i * 0x400), m->v.blank2e << 16 | m->v.blank2s,
+                               0x0828 + (i * 0x400), asyh->mode.v.blankus);
+       PUSH_NVSQ(push, NV507D, 0x082c + (i * 0x400), 0x00000000);
+       return 0;
 }
 
 int

diff --git a/drivers/gpu/drm/nouveau/dispnv50/head907d.c b/drivers/gpu/drm/nouveau/dispnv50/head907d.c
index 6682a67..2ee1931 100644 (file)
--- a/drivers/gpu/drm/nouveau/dispnv50/head907d.c
+++ b/drivers/gpu/drm/nouveau/dispnv50/head907d.c
@@ -251,29 +251,29 @@ head907d_olut(struct nv50_head *head, struct nv50_head_atom *asyh, int size)
        return true;
 }
 
-void
+int
 head907d_mode(struct nv50_head *head, struct nv50_head_atom *asyh)
 {
-       struct nv50_dmac *core = &nv50_disp(head->base.base.dev)->core->chan;
+       struct nvif_push *push = nv50_disp(head->base.base.dev)->core->chan.push;
        struct nv50_head_mode *m = &asyh->mode;
-       u32 *push;
-       if ((push = evo_wait(core, 14))) {
-               evo_mthd(push, 0x0410 + (head->base.index * 0x300), 6);
-               evo_data(push, 0x00000000);
-               evo_data(push, m->v.active  << 16 | m->h.active );
-               evo_data(push, m->v.synce   << 16 | m->h.synce  );
-               evo_data(push, m->v.blanke  << 16 | m->h.blanke );
-               evo_data(push, m->v.blanks  << 16 | m->h.blanks );
-               evo_data(push, m->v.blank2e << 16 | m->v.blank2s);
-               evo_mthd(push, 0x042c + (head->base.index * 0x300), 2);
-               evo_data(push, 0x00000000); /* ??? */
-               evo_data(push, 0xffffff00);
-               evo_mthd(push, 0x0450 + (head->base.index * 0x300), 3);
-               evo_data(push, m->clock * 1000);
-               evo_data(push, 0x00200000); /* ??? */
-               evo_data(push, m->clock * 1000);
-               evo_kick(push, core);
-       }
+       const int i = head->base.index;
+       int ret;
+
+       if ((ret = PUSH_WAIT(push, 14)))
+               return ret;
+
+       PUSH_NVSQ(push, NV907D, 0x0410 + (i * 0x300), 0x00000000,
+                               0x0414 + (i * 0x300), m->v.active  << 16 | m->h.active,
+                               0x0418 + (i * 0x300), m->v.synce   << 16 | m->h.synce,
+                               0x041c + (i * 0x300), m->v.blanke  << 16 | m->h.blanke,
+                               0x0420 + (i * 0x300), m->v.blanks  << 16 | m->h.blanks,
+                               0x0424 + (i * 0x300), m->v.blank2e << 16 | m->v.blank2s);
+       PUSH_NVSQ(push, NV907D, 0x042c + (i * 0x300), 0x00000000,
+                               0x0430 + (i * 0x300), 0xffffff00);
+       PUSH_NVSQ(push, NV907D, 0x0450 + (i * 0x300), m->clock * 1000,
+                               0x0454 + (i * 0x300), 0x00200000,
+                               0x0458 + (i * 0x300), m->clock * 1000);
+       return 0;
 }
 
 int

diff --git a/drivers/gpu/drm/nouveau/dispnv50/headc37d.c b/drivers/gpu/drm/nouveau/dispnv50/headc37d.c
index 0bdf2d2..e15a4d3 100644 (file)
--- a/drivers/gpu/drm/nouveau/dispnv50/headc37d.c
+++ b/drivers/gpu/drm/nouveau/dispnv50/headc37d.c
@@ -167,29 +167,29 @@ headc37d_olut(struct nv50_head *head, struct nv50_head_atom *asyh, int size)
        return true;
 }
 
-static void
+static int
 headc37d_mode(struct nv50_head *head, struct nv50_head_atom *asyh)
 {
-       struct nv50_dmac *core = &nv50_disp(head->base.base.dev)->core->chan;
+       struct nvif_push *push = nv50_disp(head->base.base.dev)->core->chan.push;
        struct nv50_head_mode *m = &asyh->mode;
-       u32 *push;
-       if ((push = evo_wait(core, 13))) {
-               evo_mthd(push, 0x2064 + (head->base.index * 0x400), 5);
-               evo_data(push, (m->v.active  << 16) | m->h.active );
-               evo_data(push, (m->v.synce   << 16) | m->h.synce  );
-               evo_data(push, (m->v.blanke  << 16) | m->h.blanke );
-               evo_data(push, (m->v.blanks  << 16) | m->h.blanks );
-               evo_data(push, (m->v.blank2e << 16) | m->v.blank2s);
-               evo_mthd(push, 0x2008 + (head->base.index * 0x400), 2);
-               evo_data(push, m->interlace);
-               evo_data(push, m->clock * 1000);
-               evo_mthd(push, 0x2028 + (head->base.index * 0x400), 1);
-               evo_data(push, m->clock * 1000);
-               /*XXX: HEAD_USAGE_BOUNDS, doesn't belong here. */
-               evo_mthd(push, 0x2030 + (head->base.index * 0x400), 1);
-               evo_data(push, 0x00000124);
-               evo_kick(push, core);
-       }
+       const int i = head->base.index;
+       int ret;
+
+       if ((ret = PUSH_WAIT(push, 13)))
+               return ret;
+
+       PUSH_NVSQ(push, NVC37D, 0x2064 + (i * 0x400), m->v.active  << 16 | m->h.active,
+                               0x2068 + (i * 0x400), m->v.synce   << 16 | m->h.synce,
+                               0x206c + (i * 0x400), m->v.blanke  << 16 | m->h.blanke,
+                               0x2070 + (i * 0x400), m->v.blanks  << 16 | m->h.blanks,
+                               0x2074 + (i * 0x400), m->v.blank2e << 16 | m->v.blank2s);
+       PUSH_NVSQ(push, NVC37D, 0x2008 + (i * 0x400), m->interlace,
+                               0x200c + (i * 0x400), m->clock * 1000);
+       PUSH_NVSQ(push, NVC37D, 0x2028 + (i * 0x400), m->clock * 1000);
+
+       /*XXX: HEAD_USAGE_BOUNDS, doesn't belong here. */
+       PUSH_NVSQ(push, NVC37D, 0x2030 + (i * 0x400), 0x00000124);
+       return 0;
 }
 
 int

diff --git a/drivers/gpu/drm/nouveau/dispnv50/headc57d.c b/drivers/gpu/drm/nouveau/dispnv50/headc57d.c
index 28d4431..1024436 100644 (file)
--- a/drivers/gpu/drm/nouveau/dispnv50/headc57d.c
+++ b/drivers/gpu/drm/nouveau/dispnv50/headc57d.c
@@ -23,6 +23,8 @@
 #include "atom.h"
 #include "core.h"
 
+#include <nvif/pushc37b.h>
+
 static void
 headc57d_or(struct nv50_head *head, struct nv50_head_atom *asyh)
 {
@@ -171,29 +173,29 @@ headc57d_olut(struct nv50_head *head, struct nv50_head_atom *asyh, int size)
        return true;
 }
 
-static void
+static int
 headc57d_mode(struct nv50_head *head, struct nv50_head_atom *asyh)
 {
-       struct nv50_dmac *core = &nv50_disp(head->base.base.dev)->core->chan;
+       struct nvif_push *push = nv50_disp(head->base.base.dev)->core->chan.push;
        struct nv50_head_mode *m = &asyh->mode;
-       u32 *push;
-       if ((push = evo_wait(core, 13))) {
-               evo_mthd(push, 0x2064 + (head->base.index * 0x400), 5);
-               evo_data(push, (m->v.active  << 16) | m->h.active );
-               evo_data(push, (m->v.synce   << 16) | m->h.synce  );
-               evo_data(push, (m->v.blanke  << 16) | m->h.blanke );
-               evo_data(push, (m->v.blanks  << 16) | m->h.blanks );
-               evo_data(push, (m->v.blank2e << 16) | m->v.blank2s);
-               evo_mthd(push, 0x2008 + (head->base.index * 0x400), 2);
-               evo_data(push, m->interlace);
-               evo_data(push, m->clock * 1000);
-               evo_mthd(push, 0x2028 + (head->base.index * 0x400), 1);
-               evo_data(push, m->clock * 1000);
-               /*XXX: HEAD_USAGE_BOUNDS, doesn't belong here. */
-               evo_mthd(push, 0x2030 + (head->base.index * 0x400), 1);
-               evo_data(push, 0x00001114);
-               evo_kick(push, core);
-       }
+       const int i = head->base.index;
+       int ret;
+
+       if ((ret = PUSH_WAIT(push, 13)))
+               return ret;
+
+       PUSH_NVSQ(push, NVC57D, 0x2064 + (i * 0x400), m->v.active  << 16 | m->h.active,
+                               0x2068 + (i * 0x400), m->v.synce   << 16 | m->h.synce,
+                               0x206c + (i * 0x400), m->v.blanke  << 16 | m->h.blanke,
+                               0x2070 + (i * 0x400), m->v.blanks  << 16 | m->h.blanks,
+                               0x2074 + (i * 0x400), m->v.blank2e << 16 | m->v.blank2s);
+       PUSH_NVSQ(push, NVC57D, 0x2008 + (i * 0x400), m->interlace,
+                               0x200c + (i * 0x400), m->clock * 1000);
+       PUSH_NVSQ(push, NVC57D, 0x2028 + (i * 0x400), m->clock * 1000);
+
+       /*XXX: HEAD_USAGE_BOUNDS, doesn't belong here. */
+       PUSH_NVSQ(push, NVC57D, 0x2030 + (i * 0x400), 0x00001114);
+       return 0;
 }
 
 const struct nv50_head_func