drm/nouveau/disp/nv50-: simplify definition of overlay immediate channels
authorBen Skeggs <bskeggs@redhat.com>
Tue, 8 May 2018 10:39:46 +0000 (20:39 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Fri, 18 May 2018 05:01:20 +0000 (15:01 +1000)
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
21 files changed:
drivers/gpu/drm/nouveau/nvkm/engine/disp/Kbuild
drivers/gpu/drm/nouveau/nvkm/engine/disp/channv50.h
drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmg84.c [deleted file]
drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmgf119.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmgk104.c [deleted file]
drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmgp102.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmgt215.c [deleted file]
drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmnv50.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootg84.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootg94.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootgf119.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootgk104.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootgk110.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootgm107.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootgm200.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootgp100.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootgp102.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootgt200.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootgt215.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootnv50.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/rootnv50.h

index 67861f2..b53a0e2 100644 (file)
@@ -111,8 +111,5 @@ nvkm-y += nvkm/engine/disp/cursgk104.o
 nvkm-y += nvkm/engine/disp/cursgp102.o
 
 nvkm-y += nvkm/engine/disp/oimmnv50.o
-nvkm-y += nvkm/engine/disp/oimmg84.o
-nvkm-y += nvkm/engine/disp/oimmgt215.o
 nvkm-y += nvkm/engine/disp/oimmgf119.o
-nvkm-y += nvkm/engine/disp/oimmgk104.o
 nvkm-y += nvkm/engine/disp/oimmgp102.o
index b222a1d..ba30766 100644 (file)
@@ -50,12 +50,18 @@ void nv50_disp_chan_uevent_send(struct nv50_disp *, int);
 
 extern const struct nvkm_event_func gf119_disp_chan_uevent;
 
+int nv50_disp_oimm_new_(const struct nv50_disp_chan_func *,
+                       struct nv50_disp *, int ctrl, int user,
+                       const struct nvkm_oclass *, void *argv, u32 argc,
+                       struct nvkm_object **);
 int nv50_disp_ovly_new_(const struct nv50_disp_dmac_func *,
                        const struct nv50_disp_chan_mthd *,
                        struct nv50_disp *, int chid,
                        const struct nvkm_oclass *, void *argv, u32 argc,
                        struct nvkm_object **);
 
+int nv50_disp_oimm_new(const struct nvkm_oclass *, void *, u32,
+                      struct nv50_disp *, struct nvkm_object **);
 int nv50_disp_ovly_new(const struct nvkm_oclass *, void *, u32,
                       struct nv50_disp *, struct nvkm_object **);
 
@@ -65,12 +71,16 @@ int g84_disp_ovly_new(const struct nvkm_oclass *, void *, u32,
 int gt200_disp_ovly_new(const struct nvkm_oclass *, void *, u32,
                        struct nv50_disp *, struct nvkm_object **);
 
+int gf119_disp_oimm_new(const struct nvkm_oclass *, void *, u32,
+                       struct nv50_disp *, struct nvkm_object **);
 int gf119_disp_ovly_new(const struct nvkm_oclass *, void *, u32,
                        struct nv50_disp *, struct nvkm_object **);
 
 int gk104_disp_ovly_new(const struct nvkm_oclass *, void *, u32,
                        struct nv50_disp *, struct nvkm_object **);
 
+int gp102_disp_oimm_new(const struct nvkm_oclass *, void *, u32,
+                       struct nv50_disp *, struct nvkm_object **);
 int gp102_disp_ovly_new(const struct nvkm_oclass *, void *, u32,
                        struct nv50_disp *, struct nvkm_object **);
 
@@ -133,22 +143,16 @@ struct nv50_disp_pioc_oclass {
        } chid;
 };
 
-extern const struct nv50_disp_pioc_oclass nv50_disp_oimm_oclass;
 extern const struct nv50_disp_pioc_oclass nv50_disp_curs_oclass;
 
-extern const struct nv50_disp_pioc_oclass g84_disp_oimm_oclass;
 extern const struct nv50_disp_pioc_oclass g84_disp_curs_oclass;
 
-extern const struct nv50_disp_pioc_oclass gt215_disp_oimm_oclass;
 extern const struct nv50_disp_pioc_oclass gt215_disp_curs_oclass;
 
-extern const struct nv50_disp_pioc_oclass gf119_disp_oimm_oclass;
 extern const struct nv50_disp_pioc_oclass gf119_disp_curs_oclass;
 
-extern const struct nv50_disp_pioc_oclass gk104_disp_oimm_oclass;
 extern const struct nv50_disp_pioc_oclass gk104_disp_curs_oclass;
 
-extern const struct nv50_disp_pioc_oclass gp102_disp_oimm_oclass;
 extern const struct nv50_disp_pioc_oclass gp102_disp_curs_oclass;
 
 int nv50_disp_curs_new(const struct nv50_disp_chan_func *,
@@ -156,9 +160,4 @@ int nv50_disp_curs_new(const struct nv50_disp_chan_func *,
                       struct nv50_disp_root *, int ctrl, int user,
                       const struct nvkm_oclass *, void *data, u32 size,
                       struct nvkm_object **);
-int nv50_disp_oimm_new(const struct nv50_disp_chan_func *,
-                      const struct nv50_disp_chan_mthd *,
-                      struct nv50_disp_root *, int ctrl, int user,
-                      const struct nvkm_oclass *, void *data, u32 size,
-                      struct nvkm_object **);
 #endif
diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmg84.c b/drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmg84.c
deleted file mode 100644 (file)
index 5ad5d0f..0000000
+++ /dev/null
@@ -1,37 +0,0 @@
-/*
- * Copyright 2012 Red Hat Inc.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
- * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
- * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
- * OTHER DEALINGS IN THE SOFTWARE.
- *
- * Authors: Ben Skeggs
- */
-#include "channv50.h"
-#include "rootnv50.h"
-
-#include <nvif/class.h>
-
-const struct nv50_disp_pioc_oclass
-g84_disp_oimm_oclass = {
-       .base.oclass = G82_DISP_OVERLAY,
-       .base.minver = 0,
-       .base.maxver = 0,
-       .ctor = nv50_disp_oimm_new,
-       .func = &nv50_disp_pioc_func,
-       .chid = { 5, 5 },
-};
index 1f9fd34..1ae0bcf 100644 (file)
  * Authors: Ben Skeggs
  */
 #include "channv50.h"
-#include "rootnv50.h"
 
-#include <nvif/class.h>
-
-const struct nv50_disp_pioc_oclass
-gf119_disp_oimm_oclass = {
-       .base.oclass = GF110_DISP_OVERLAY,
-       .base.minver = 0,
-       .base.maxver = 0,
-       .ctor = nv50_disp_oimm_new,
-       .func = &gf119_disp_pioc_func,
-       .chid = { 9, 9 },
-};
+int
+gf119_disp_oimm_new(const struct nvkm_oclass *oclass, void *argv, u32 argc,
+                   struct nv50_disp *disp, struct nvkm_object **pobject)
+{
+       return nv50_disp_oimm_new_(&gf119_disp_pioc_func, disp, 9, 9,
+                                  oclass, argv, argc, pobject);
+}
diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmgk104.c b/drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmgk104.c
deleted file mode 100644 (file)
index 0c09fe8..0000000
+++ /dev/null
@@ -1,37 +0,0 @@
-/*
- * Copyright 2012 Red Hat Inc.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
- * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
- * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
- * OTHER DEALINGS IN THE SOFTWARE.
- *
- * Authors: Ben Skeggs
- */
-#include "channv50.h"
-#include "rootnv50.h"
-
-#include <nvif/class.h>
-
-const struct nv50_disp_pioc_oclass
-gk104_disp_oimm_oclass = {
-       .base.oclass = GK104_DISP_OVERLAY,
-       .base.minver = 0,
-       .base.maxver = 0,
-       .ctor = nv50_disp_oimm_new,
-       .func = &gf119_disp_pioc_func,
-       .chid = { 9, 9 },
-};
index abf8236..30ffb10 100644 (file)
  * Authors: Ben Skeggs <bskeggs@redhat.com>
  */
 #include "channv50.h"
-#include "rootnv50.h"
 
-#include <nvif/class.h>
-
-const struct nv50_disp_pioc_oclass
-gp102_disp_oimm_oclass = {
-       .base.oclass = GK104_DISP_OVERLAY,
-       .base.minver = 0,
-       .base.maxver = 0,
-       .ctor = nv50_disp_oimm_new,
-       .func = &gf119_disp_pioc_func,
-       .chid = { 9, 13 },
-};
+int
+gp102_disp_oimm_new(const struct nvkm_oclass *oclass, void *argv, u32 argc,
+                   struct nv50_disp *disp, struct nvkm_object **pobject)
+{
+       return nv50_disp_oimm_new_(&gf119_disp_pioc_func, disp, 9, 13,
+                                  oclass, argv, argc, pobject);
+}
diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmgt215.c b/drivers/gpu/drm/nouveau/nvkm/engine/disp/oimmgt215.c
deleted file mode 100644 (file)
index 1281db2..0000000
+++ /dev/null
@@ -1,37 +0,0 @@
-/*
- * Copyright 2012 Red Hat Inc.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE COPYRIGHT HOLDER(S) OR AUTHOR(S) BE LIABLE FOR ANY CLAIM, DAMAGES OR
- * OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
- * ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
- * OTHER DEALINGS IN THE SOFTWARE.
- *
- * Authors: Ben Skeggs
- */
-#include "channv50.h"
-#include "rootnv50.h"
-
-#include <nvif/class.h>
-
-const struct nv50_disp_pioc_oclass
-gt215_disp_oimm_oclass = {
-       .base.oclass = GT214_DISP_OVERLAY,
-       .base.minver = 0,
-       .base.maxver = 0,
-       .ctor = nv50_disp_oimm_new,
-       .func = &nv50_disp_pioc_func,
-       .chid = { 5, 5 },
-};
index 7e013cd..0db99bf 100644 (file)
  */
 #include "channv50.h"
 #include "head.h"
-#include "rootnv50.h"
 
 #include <core/client.h>
 
-#include <nvif/class.h>
 #include <nvif/cl507b.h>
 #include <nvif/unpack.h>
 
 int
-nv50_disp_oimm_new(const struct nv50_disp_chan_func *func,
-                  const struct nv50_disp_chan_mthd *mthd,
-                  struct nv50_disp_root *root, int ctrl, int user,
-                  const struct nvkm_oclass *oclass, void *data, u32 size,
-                  struct nvkm_object **pobject)
+nv50_disp_oimm_new_(const struct nv50_disp_chan_func *func,
+                   struct nv50_disp *disp, int ctrl, int user,
+                   const struct nvkm_oclass *oclass, void *argv, u32 argc,
+                   struct nvkm_object **pobject)
 {
        union {
                struct nv50_disp_overlay_v0 v0;
-       } *args = data;
+       } *args = argv;
        struct nvkm_object *parent = oclass->parent;
-       struct nv50_disp *disp = root->disp;
        int head, ret = -ENOSYS;
 
-       nvif_ioctl(parent, "create disp overlay size %d\n", size);
-       if (!(ret = nvif_unpack(ret, &data, &size, args->v0, 0, 0, false))) {
+       nvif_ioctl(parent, "create disp overlay size %d\n", argc);
+       if (!(ret = nvif_unpack(ret, &argv, &argc, args->v0, 0, 0, false))) {
                nvif_ioctl(parent, "create disp overlay vers %d head %d\n",
                           args->v0.version, args->v0.head);
                if (!nvkm_head_find(&disp->base, args->v0.head))
@@ -55,16 +51,14 @@ nv50_disp_oimm_new(const struct nv50_disp_chan_func *func,
        } else
                return ret;
 
-       return nv50_disp_chan_new_(func, mthd, disp, ctrl + head, user + head,
+       return nv50_disp_chan_new_(func, NULL, disp, ctrl + head, user + head,
                                   head, oclass, pobject);
 }
 
-const struct nv50_disp_pioc_oclass
-nv50_disp_oimm_oclass = {
-       .base.oclass = NV50_DISP_OVERLAY,
-       .base.minver = 0,
-       .base.maxver = 0,
-       .ctor = nv50_disp_oimm_new,
-       .func = &nv50_disp_pioc_func,
-       .chid = { 5, 5 },
-};
+int
+nv50_disp_oimm_new(const struct nvkm_oclass *oclass, void *argv, u32 argc,
+                  struct nv50_disp *disp, struct nvkm_object **pobject)
+{
+       return nv50_disp_oimm_new_(&nv50_disp_pioc_func, disp, 5, 5,
+                                  oclass, argv, argc, pobject);
+}
index ebb0803..650ed0d 100644 (file)
@@ -33,10 +33,10 @@ g84_disp_root = {
                &g84_disp_base_oclass,
        },
        .pioc = {
-               &g84_disp_oimm_oclass,
                &g84_disp_curs_oclass,
        },
        .user = {
+               {{0,0,G82_DISP_OVERLAY            }, nv50_disp_oimm_new },
                {{0,0,G82_DISP_OVERLAY_CHANNEL_DMA},  g84_disp_ovly_new },
                {}
        },
index 6949cf0..19d23e0 100644 (file)
@@ -33,10 +33,10 @@ g94_disp_root = {
                &gt200_disp_base_oclass,
        },
        .pioc = {
-               &g84_disp_oimm_oclass,
                &g84_disp_curs_oclass,
        },
        .user = {
+               {{0,0,  G82_DISP_OVERLAY            },  nv50_disp_oimm_new },
                {{0,0,GT200_DISP_OVERLAY_CHANNEL_DMA}, gt200_disp_ovly_new },
                {}
        },
index acc897f..ef8be6a 100644 (file)
@@ -33,10 +33,10 @@ gf119_disp_root = {
                &gf119_disp_base_oclass,
        },
        .pioc = {
-               &gf119_disp_oimm_oclass,
                &gf119_disp_curs_oclass,
        },
        .user = {
+               {{0,0,GF110_DISP_OVERLAY            }, gf119_disp_oimm_new },
                {{0,0,GF110_DISP_OVERLAY_CONTROL_DMA}, gf119_disp_ovly_new },
                {}
        },
index 2d19a06..67002c0 100644 (file)
@@ -33,10 +33,10 @@ gk104_disp_root = {
                &gk104_disp_base_oclass,
        },
        .pioc = {
-               &gk104_disp_oimm_oclass,
                &gk104_disp_curs_oclass,
        },
        .user = {
+               {{0,0,GK104_DISP_OVERLAY            }, gf119_disp_oimm_new },
                {{0,0,GK104_DISP_OVERLAY_CONTROL_DMA}, gk104_disp_ovly_new },
                {}
        },
index d7e2249..f82cf9c 100644 (file)
@@ -33,10 +33,10 @@ gk110_disp_root = {
                &gk110_disp_base_oclass,
        },
        .pioc = {
-               &gk104_disp_oimm_oclass,
                &gk104_disp_curs_oclass,
        },
        .user = {
+               {{0,0,GK104_DISP_OVERLAY            }, gf119_disp_oimm_new },
                {{0,0,GK104_DISP_OVERLAY_CONTROL_DMA}, gk104_disp_ovly_new },
                {}
        },
index de87b27..170961e 100644 (file)
@@ -33,10 +33,10 @@ gm107_disp_root = {
                &gk110_disp_base_oclass,
        },
        .pioc = {
-               &gk104_disp_oimm_oclass,
                &gk104_disp_curs_oclass,
        },
        .user = {
+               {{0,0,GK104_DISP_OVERLAY            }, gf119_disp_oimm_new },
                {{0,0,GK104_DISP_OVERLAY_CONTROL_DMA}, gk104_disp_ovly_new },
                {}
        },
index 9e97c1e..3f77682 100644 (file)
@@ -33,10 +33,10 @@ gm200_disp_root = {
                &gk110_disp_base_oclass,
        },
        .pioc = {
-               &gk104_disp_oimm_oclass,
                &gk104_disp_curs_oclass,
        },
        .user = {
+               {{0,0,GK104_DISP_OVERLAY            }, gf119_disp_oimm_new },
                {{0,0,GK104_DISP_OVERLAY_CONTROL_DMA}, gk104_disp_ovly_new },
                {}
        },
index af7031f..c87b1d2 100644 (file)
@@ -33,10 +33,10 @@ gp100_disp_root = {
                &gk110_disp_base_oclass,
        },
        .pioc = {
-               &gk104_disp_oimm_oclass,
                &gk104_disp_curs_oclass,
        },
        .user = {
+               {{0,0,GK104_DISP_OVERLAY            }, gf119_disp_oimm_new },
                {{0,0,GK104_DISP_OVERLAY_CONTROL_DMA}, gk104_disp_ovly_new },
                {}
        },
index 7603a41..16516a4 100644 (file)
@@ -33,10 +33,10 @@ gp102_disp_root = {
                &gp102_disp_base_oclass,
        },
        .pioc = {
-               &gp102_disp_oimm_oclass,
                &gp102_disp_curs_oclass,
        },
        .user = {
+               {{0,0,GK104_DISP_OVERLAY            }, gp102_disp_oimm_new },
                {{0,0,GK104_DISP_OVERLAY_CONTROL_DMA}, gp102_disp_ovly_new },
                {}
        },
index 8ef149d..6d46bf6 100644 (file)
@@ -33,10 +33,10 @@ gt200_disp_root = {
                &gt200_disp_base_oclass,
        },
        .pioc = {
-               &g84_disp_oimm_oclass,
                &g84_disp_curs_oclass,
        },
        .user = {
+               {{0,0,  G82_DISP_OVERLAY            },  nv50_disp_oimm_new },
                {{0,0,GT200_DISP_OVERLAY_CHANNEL_DMA}, gt200_disp_ovly_new },
                {}
        },
index 07c8013..6863c94 100644 (file)
@@ -33,10 +33,10 @@ gt215_disp_root = {
                &gt215_disp_base_oclass,
        },
        .pioc = {
-               &gt215_disp_oimm_oclass,
                &gt215_disp_curs_oclass,
        },
        .user = {
+               {{0,0,GT214_DISP_OVERLAY            },  nv50_disp_oimm_new },
                {{0,0,GT214_DISP_OVERLAY_CHANNEL_DMA},   g84_disp_ovly_new },
                {}
        },
index f02368f..41219c2 100644 (file)
@@ -371,10 +371,10 @@ nv50_disp_root = {
                &nv50_disp_base_oclass,
        },
        .pioc = {
-               &nv50_disp_oimm_oclass,
                &nv50_disp_curs_oclass,
        },
        .user = {
+               {{0,0,NV50_DISP_OVERLAY            }, nv50_disp_oimm_new },
                {{0,0,NV50_DISP_OVERLAY_CHANNEL_DMA}, nv50_disp_ovly_new },
                {}
        },
index 1c4985a..67f9518 100644 (file)
@@ -14,7 +14,7 @@ struct nv50_disp_root {
 
 struct nv50_disp_root_func {
        const struct nv50_disp_dmac_oclass *dmac[2];
-       const struct nv50_disp_pioc_oclass *pioc[2];
+       const struct nv50_disp_pioc_oclass *pioc[1];
        struct nv50_disp_user {
                struct nvkm_sclass base;
                int (*ctor)(const struct nvkm_oclass *, void *argv, u32 argc,