drm/nouveau/kms/nv50-: convert wndw ntfy_clr() to new push macros
authorBen Skeggs <bskeggs@redhat.com>
Fri, 19 Jun 2020 22:32:28 +0000 (08:32 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Fri, 24 Jul 2020 08:50:52 +0000 (18:50 +1000)
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>
drivers/gpu/drm/nouveau/dispnv50/base.h
drivers/gpu/drm/nouveau/dispnv50/base507c.c
drivers/gpu/drm/nouveau/dispnv50/ovly.h
drivers/gpu/drm/nouveau/dispnv50/ovly507e.c
drivers/gpu/drm/nouveau/dispnv50/ovly827e.c
drivers/gpu/drm/nouveau/dispnv50/ovly907e.c
drivers/gpu/drm/nouveau/dispnv50/wndw.h
drivers/gpu/drm/nouveau/dispnv50/wndwc37e.c

index a93e2e31464079a13b40a9f3832a692e2ecad850..03b655f02b680ddbb5a6f42ab26a45bf52db3c42 100644 (file)
@@ -13,7 +13,6 @@ void base507c_release(struct nv50_wndw *, struct nv50_wndw_atom *,
                      struct nv50_head_atom *);
 int base507c_sema_set(struct nv50_wndw *, struct nv50_wndw_atom *);
 int base507c_sema_clr(struct nv50_wndw *);
-void base507c_ntfy_clr(struct nv50_wndw *);
 void base507c_xlut_set(struct nv50_wndw *, struct nv50_wndw_atom *);
 void base507c_xlut_clr(struct nv50_wndw *);
 void base507c_image_clr(struct nv50_wndw *);
index d0d6a5056171e605192ce52106dc8f8768c9d6a9..d707a75d9fdbba0b803b3e9eca6d6ed46e4559b8 100644 (file)
@@ -124,15 +124,17 @@ base507c_ntfy_wait_begun(struct nouveau_bo *bo, u32 offset,
        return time < 0 ? time : 0;
 }
 
-void
+int
 base507c_ntfy_clr(struct nv50_wndw *wndw)
 {
-       u32 *push;
-       if ((push = evo_wait(&wndw->wndw, 2))) {
-               evo_mthd(push, 0x00a4, 1);
-               evo_data(push, 0x00000000);
-               evo_kick(push, &wndw->wndw);
-       }
+       struct nvif_push *push = wndw->wndw.push;
+       int ret;
+
+       if ((ret = PUSH_WAIT(push, 2)))
+               return ret;
+
+       PUSH_NVSQ(push, NV507C, 0x00a4, 0x00000000);
+       return 0;
 }
 
 int
index dee606213d275019825afc45188cd9179dabf1a4..1048fc6bc7785c9b18d54e12d35dbe649791a0b2 100644 (file)
@@ -10,7 +10,6 @@ int ovly507e_acquire(struct nv50_wndw *, struct nv50_wndw_atom *,
                     struct nv50_head_atom *);
 void ovly507e_release(struct nv50_wndw *, struct nv50_wndw_atom *,
                      struct nv50_head_atom *);
-void ovly507e_ntfy_clr(struct nv50_wndw *);
 void ovly507e_image_clr(struct nv50_wndw *);
 void ovly507e_scale_set(struct nv50_wndw *, struct nv50_wndw_atom *);
 void ovly507e_update(struct nv50_wndw *, u32 *);
index 91ea3ea3e7fa948ef3229e9d57085e300b0f8b53..151064aa19e183ae8e8c97d8fa253e2cb9acd53a 100644 (file)
@@ -92,17 +92,6 @@ ovly507e_image_set(struct nv50_wndw *wndw, struct nv50_wndw_atom *asyw)
        }
 }
 
-void
-ovly507e_ntfy_clr(struct nv50_wndw *wndw)
-{
-       u32 *push;
-       if ((push = evo_wait(&wndw->wndw, 2))) {
-               evo_mthd(push, 0x00a4, 1);
-               evo_data(push, 0x00000000);
-               evo_kick(push, &wndw->wndw);
-       }
-}
-
 void
 ovly507e_release(struct nv50_wndw *wndw, struct nv50_wndw_atom *asyw,
                 struct nv50_head_atom *asyh)
@@ -135,7 +124,7 @@ ovly507e = {
        .acquire = ovly507e_acquire,
        .release = ovly507e_release,
        .ntfy_set = base507c_ntfy_set,
-       .ntfy_clr = ovly507e_ntfy_clr,
+       .ntfy_clr = base507c_ntfy_clr,
        .ntfy_reset = base507c_ntfy_reset,
        .ntfy_wait_begun = base507c_ntfy_wait_begun,
        .image_set = ovly507e_image_set,
index 72a4117529e59a55862b64a1db6eceb506c065a0..37e37a7d4ba9c800b330b252e6602ed8ca7b0335 100644 (file)
@@ -78,7 +78,7 @@ ovly827e = {
        .acquire = ovly507e_acquire,
        .release = ovly507e_release,
        .ntfy_set = base507c_ntfy_set,
-       .ntfy_clr = ovly507e_ntfy_clr,
+       .ntfy_clr = base507c_ntfy_clr,
        .ntfy_reset = ovly827e_ntfy_reset,
        .ntfy_wait_begun = ovly827e_ntfy_wait_begun,
        .image_set = ovly827e_image_set,
index 255a8415eb451182e98a10d0ad701368cc7d37b9..73fb5d4eb34338b00cb5b01a0a4793294744a765 100644 (file)
@@ -52,7 +52,7 @@ ovly907e = {
        .acquire = ovly507e_acquire,
        .release = ovly507e_release,
        .ntfy_set = base507c_ntfy_set,
-       .ntfy_clr = ovly507e_ntfy_clr,
+       .ntfy_clr = base507c_ntfy_clr,
        .ntfy_reset = ovly827e_ntfy_reset,
        .ntfy_wait_begun = ovly827e_ntfy_wait_begun,
        .image_set = ovly907e_image_set,
index 6c994461ea63f83e4d8de26f361f5fdb2689d647..d632a30022e45a5c1085711ce9a814a5c7af8cf2 100644 (file)
@@ -61,7 +61,7 @@ struct nv50_wndw_func {
        int (*sema_clr)(struct nv50_wndw *);
        void (*ntfy_reset)(struct nouveau_bo *, u32 offset);
        int (*ntfy_set)(struct nv50_wndw *, struct nv50_wndw_atom *);
-       void (*ntfy_clr)(struct nv50_wndw *);
+       int (*ntfy_clr)(struct nv50_wndw *);
        int (*ntfy_wait_begun)(struct nouveau_bo *, u32 offset,
                               struct nvif_device *);
        bool (*ilut)(struct nv50_wndw *, struct nv50_wndw_atom *, int);
@@ -86,6 +86,7 @@ extern const struct drm_plane_funcs nv50_wndw;
 
 void base507c_ntfy_reset(struct nouveau_bo *, u32);
 int base507c_ntfy_set(struct nv50_wndw *, struct nv50_wndw_atom *);
+int base507c_ntfy_clr(struct nv50_wndw *);
 int base507c_ntfy_wait_begun(struct nouveau_bo *, u32, struct nvif_device *);
 
 void base907c_csc(struct nv50_wndw *, struct nv50_wndw_atom *,
@@ -119,7 +120,7 @@ void wndwc37e_release(struct nv50_wndw *, struct nv50_wndw_atom *,
 int wndwc37e_sema_set(struct nv50_wndw *, struct nv50_wndw_atom *);
 int wndwc37e_sema_clr(struct nv50_wndw *);
 int wndwc37e_ntfy_set(struct nv50_wndw *, struct nv50_wndw_atom *);
-void wndwc37e_ntfy_clr(struct nv50_wndw *);
+int wndwc37e_ntfy_clr(struct nv50_wndw *);
 void wndwc37e_image_clr(struct nv50_wndw *);
 void wndwc37e_blend_set(struct nv50_wndw *, struct nv50_wndw_atom *);
 void wndwc37e_update(struct nv50_wndw *, u32 *);
index 9738205949bf843275c28d0c69703e35c7060b0b..2dded1a8407a51ca140a220cc7cdbaa9f2520ee9 100644 (file)
@@ -152,15 +152,17 @@ wndwc37e_image_set(struct nv50_wndw *wndw, struct nv50_wndw_atom *asyw)
        evo_kick(push, &wndw->wndw);
 }
 
-void
+int
 wndwc37e_ntfy_clr(struct nv50_wndw *wndw)
 {
-       u32 *push;
-       if ((push = evo_wait(&wndw->wndw, 2))) {
-               evo_mthd(push, 0x021c, 1);
-               evo_data(push, 0x00000000);
-               evo_kick(push, &wndw->wndw);
-       }
+       struct nvif_push *push = wndw->wndw.push;
+       int ret;
+
+       if ((ret = PUSH_WAIT(push, 2)))
+               return ret;
+
+       PUSH_NVSQ(push, NVC37E, 0x021c, 0x00000000);
+       return 0;
 }
 
 int