drm/nouveau/disp: merge nv50_disp_new_() and nvkm_disp_new()
authorBen Skeggs <bskeggs@redhat.com>
Wed, 1 Jun 2022 10:46:28 +0000 (20:46 +1000)
committerDave Airlie <airlied@redhat.com>
Tue, 26 Jul 2022 23:05:47 +0000 (09:05 +1000)
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
19 files changed:
drivers/gpu/drm/nouveau/nvkm/engine/disp/base.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/g84.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/g94.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/ga102.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gf119.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gk104.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gk110.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gm107.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gm200.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gp100.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gp102.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gt200.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gt215.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gv100.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/mcp77.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/mcp89.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/nv50.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/priv.h
drivers/gpu/drm/nouveau/nvkm/engine/disp/tu102.c

index d68d523..10d4a33 100644 (file)
@@ -30,6 +30,7 @@
 
 #include <core/client.h>
 #include <core/notify.h>
+#include <core/ramht.h>
 #include <subdev/bios.h>
 #include <subdev/bios/dcb.h>
 
@@ -394,8 +395,12 @@ nvkm_disp_dtor(struct nvkm_engine *engine)
        struct nvkm_head *head;
        void *data = disp;
 
-       if (disp->func->dtor)
-               data = disp->func->dtor(disp);
+       nvkm_ramht_del(&disp->ramht);
+       nvkm_gpuobj_del(&disp->inst);
+
+       nvkm_event_fini(&disp->uevent);
+       if (disp->super.wq)
+               destroy_workqueue(disp->super.wq);
 
        nvkm_event_fini(&disp->vblank);
        nvkm_event_fini(&disp->hpd);
@@ -436,23 +441,33 @@ nvkm_disp = {
 };
 
 int
-nvkm_disp_ctor(const struct nvkm_disp_func *func, struct nvkm_device *device,
-              enum nvkm_subdev_type type, int inst, struct nvkm_disp *disp)
+nvkm_disp_new_(const struct nvkm_disp_func *func, struct nvkm_device *device,
+              enum nvkm_subdev_type type, int inst, struct nvkm_disp **pdisp)
 {
+       struct nvkm_disp *disp;
+       int ret;
+
+       if (!(disp = *pdisp = kzalloc(sizeof(**pdisp), GFP_KERNEL)))
+               return -ENOMEM;
+
        disp->func = func;
        INIT_LIST_HEAD(&disp->heads);
        INIT_LIST_HEAD(&disp->iors);
        INIT_LIST_HEAD(&disp->outps);
        INIT_LIST_HEAD(&disp->conns);
        spin_lock_init(&disp->client.lock);
-       return nvkm_engine_ctor(&nvkm_disp, device, type, inst, true, &disp->engine);
-}
 
-int
-nvkm_disp_new_(const struct nvkm_disp_func *func, struct nvkm_device *device,
-              enum nvkm_subdev_type type, int inst, struct nvkm_disp **pdisp)
-{
-       if (!(*pdisp = kzalloc(sizeof(**pdisp), GFP_KERNEL)))
-               return -ENOMEM;
-       return nvkm_disp_ctor(func, device, type, inst, *pdisp);
+       ret = nvkm_engine_ctor(&nvkm_disp, device, type, inst, true, &disp->engine);
+       if (ret)
+               return ret;
+
+       if (func->super) {
+               disp->super.wq = create_singlethread_workqueue("nvkm-disp");
+               if (!disp->super.wq)
+                       return -ENOMEM;
+
+               INIT_WORK(&disp->super.work, func->super);
+       }
+
+       return nvkm_event_init(func->uevent, 1, ARRAY_SIZE(disp->chan), &disp->uevent);
 }
index 69ef40f..af5d351 100644 (file)
@@ -298,7 +298,6 @@ g84_disp_core_new(const struct nvkm_oclass *oclass, void *argv, u32 argc,
 
 static const struct nvkm_disp_func
 g84_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = nv50_disp_init,
        .fini = nv50_disp_fini,
@@ -324,5 +323,5 @@ int
 g84_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
             struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&g84_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&g84_disp, device, type, inst, pdisp);
 }
index 6d5fd5c..5c9a769 100644 (file)
@@ -345,7 +345,6 @@ g94_disp_core_new(const struct nvkm_oclass *oclass, void *argv, u32 argc,
 
 static const struct nvkm_disp_func
 g94_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = nv50_disp_init,
        .fini = nv50_disp_fini,
@@ -371,5 +370,5 @@ int
 g94_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
             struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&g94_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&g94_disp, device, type, inst, pdisp);
 }
index b3cfed2..c693ad0 100644 (file)
@@ -152,7 +152,6 @@ ga102_sor_new(struct nvkm_disp *disp, int id)
 
 static const struct nvkm_disp_func
 ga102_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = tu102_disp_init,
        .fini = gv100_disp_fini,
@@ -178,5 +177,5 @@ int
 ga102_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&ga102_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&ga102_disp, device, type, inst, pdisp);
 }
index 492e2d2..75b7b41 100644 (file)
@@ -1202,7 +1202,6 @@ gf119_disp_init(struct nvkm_disp *disp)
 
 static const struct nvkm_disp_func
 gf119_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = gf119_disp_init,
        .fini = gf119_disp_fini,
@@ -1228,5 +1227,5 @@ int
 gf119_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gf119_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gf119_disp, device, type, inst, pdisp);
 }
index b7b7a0c..1c9a5e0 100644 (file)
@@ -296,7 +296,6 @@ gk104_disp_core_new(const struct nvkm_oclass *oclass, void *argv, u32 argc,
 
 static const struct nvkm_disp_func
 gk104_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = gf119_disp_init,
        .fini = gf119_disp_fini,
@@ -322,5 +321,5 @@ int
 gk104_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gk104_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gk104_disp, device, type, inst, pdisp);
 }
index d4097d3..018a92d 100644 (file)
@@ -30,7 +30,6 @@
 
 static const struct nvkm_disp_func
 gk110_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = gf119_disp_init,
        .fini = gf119_disp_fini,
@@ -56,5 +55,5 @@ int
 gk110_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gk110_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gk110_disp, device, type, inst, pdisp);
 }
index 9a0a519..301d7a9 100644 (file)
@@ -86,7 +86,6 @@ gm107_sor_new(struct nvkm_disp *disp, int id)
 
 static const struct nvkm_disp_func
 gm107_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = gf119_disp_init,
        .fini = gf119_disp_fini,
@@ -112,5 +111,5 @@ int
 gm107_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gm107_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gm107_disp, device, type, inst, pdisp);
 }
index 242c939..c65d85b 100644 (file)
@@ -183,7 +183,6 @@ gm200_sor_new(struct nvkm_disp *disp, int id)
 
 static const struct nvkm_disp_func
 gm200_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = gf119_disp_init,
        .fini = gf119_disp_fini,
@@ -209,5 +208,5 @@ int
 gm200_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gm200_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gm200_disp, device, type, inst, pdisp);
 }
index 8b9effd..603b718 100644 (file)
@@ -101,7 +101,6 @@ gp100_sor_new(struct nvkm_disp *disp, int id)
 
 static const struct nvkm_disp_func
 gp100_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = gf119_disp_init,
        .fini = gf119_disp_fini,
@@ -126,5 +125,5 @@ int
 gp100_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gp100_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gp100_disp, device, type, inst, pdisp);
 }
index 5b8ca2d..5cafd7c 100644 (file)
@@ -172,7 +172,6 @@ gp102_disp_intr_error(struct nvkm_disp *disp, int chid)
 
 static const struct nvkm_disp_func
 gp102_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = gf119_disp_init,
        .fini = gf119_disp_fini,
@@ -197,5 +196,5 @@ int
 gp102_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gp102_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gp102_disp, device, type, inst, pdisp);
 }
index d7f9d14..31b5fd3 100644 (file)
@@ -80,7 +80,6 @@ gt200_disp_ovly_new(const struct nvkm_oclass *oclass, void *argv, u32 argc,
 
 static const struct nvkm_disp_func
 gt200_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = nv50_disp_init,
        .fini = nv50_disp_fini,
@@ -106,5 +105,5 @@ int
 gt200_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gt200_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gt200_disp, device, type, inst, pdisp);
 }
index 42cc17d..98d99e1 100644 (file)
@@ -173,7 +173,6 @@ gt215_sor_new(struct nvkm_disp *disp, int id)
 
 static const struct nvkm_disp_func
 gt215_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = nv50_disp_init,
        .fini = nv50_disp_fini,
@@ -199,5 +198,5 @@ int
 gt215_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gt215_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gt215_disp, device, type, inst, pdisp);
 }
index bb61ba6..18c40e3 100644 (file)
@@ -1285,7 +1285,6 @@ gv100_disp_init(struct nvkm_disp *disp)
 
 static const struct nvkm_disp_func
 gv100_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = gv100_disp_init,
        .fini = gv100_disp_fini,
@@ -1311,5 +1310,5 @@ int
 gv100_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&gv100_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&gv100_disp, device, type, inst, pdisp);
 }
index e23e3b8..fb8bfb9 100644 (file)
@@ -54,7 +54,6 @@ mcp77_sor_new(struct nvkm_disp *disp, int id)
 
 static const struct nvkm_disp_func
 mcp77_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = nv50_disp_init,
        .fini = nv50_disp_fini,
@@ -80,5 +79,5 @@ int
 mcp77_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&mcp77_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&mcp77_disp, device, type, inst, pdisp);
 }
index 2dc9989..8b9e976 100644 (file)
@@ -59,7 +59,6 @@ mcp89_sor_new(struct nvkm_disp *disp, int id)
 
 static const struct nvkm_disp_func
 mcp89_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = nv50_disp_init,
        .fini = nv50_disp_fini,
@@ -85,5 +84,5 @@ int
 mcp89_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&mcp89_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&mcp89_disp, device, type, inst, pdisp);
 }
index 056daf3..1a31515 100644 (file)
@@ -1788,22 +1788,8 @@ nv50_disp_oneinit(struct nvkm_disp *disp)
                              0x1000, 0, disp->inst, &disp->ramht);
 }
 
-void *
-nv50_disp_dtor(struct nvkm_disp *disp)
-{
-       nvkm_ramht_del(&disp->ramht);
-       nvkm_gpuobj_del(&disp->inst);
-
-       nvkm_event_fini(&disp->uevent);
-       if (disp->super.wq)
-               destroy_workqueue(disp->super.wq);
-
-       return disp;
-}
-
 static const struct nvkm_disp_func
 nv50_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = nv50_disp_init,
        .fini = nv50_disp_fini,
@@ -1826,34 +1812,8 @@ nv50_disp = {
 };
 
 int
-nv50_disp_new_(const struct nvkm_disp_func *func, struct nvkm_device *device,
-              enum nvkm_subdev_type type, int inst, struct nvkm_disp **pdisp)
-{
-       struct nvkm_disp *disp;
-       int ret;
-
-       if (!(disp = kzalloc(sizeof(*disp), GFP_KERNEL)))
-               return -ENOMEM;
-       disp->func = func;
-       *pdisp = disp;
-
-       ret = nvkm_disp_ctor(func, device, type, inst, disp);
-       if (ret)
-               return ret;
-
-       disp->super.wq = create_singlethread_workqueue("nvkm-disp");
-       if (!disp->super.wq)
-               return -ENOMEM;
-
-       INIT_WORK(&disp->super.work, func->super);
-
-       return nvkm_event_init(func->uevent, 1, ARRAY_SIZE(disp->chan),
-                              &disp->uevent);
-}
-
-int
 nv50_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
              struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&nv50_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&nv50_disp, device, type, inst, pdisp);
 }
index 472ff32..a73b453 100644 (file)
@@ -15,7 +15,6 @@ int nvkm_disp_new_(const struct nvkm_disp_func *, struct nvkm_device *, enum nvk
 void nvkm_disp_vblank(struct nvkm_disp *, int head);
 
 struct nvkm_disp_func {
-       void *(*dtor)(struct nvkm_disp *);
        int (*oneinit)(struct nvkm_disp *);
        int (*init)(struct nvkm_disp *);
        void (*fini)(struct nvkm_disp *);
@@ -46,9 +45,6 @@ int  nvkm_disp_ntfy(struct nvkm_object *, u32, struct nvkm_event **);
 int nv04_disp_mthd(struct nvkm_object *, u32, void *, u32);
 int nv50_disp_root_mthd_(struct nvkm_object *, u32, void *, u32);
 
-int nv50_disp_new_(const struct nvkm_disp_func *, struct nvkm_device *, enum nvkm_subdev_type, int,
-                  struct nvkm_disp **);
-void *nv50_disp_dtor(struct nvkm_disp *);
 int nv50_disp_oneinit(struct nvkm_disp *);
 int nv50_disp_init(struct nvkm_disp *);
 void nv50_disp_fini(struct nvkm_disp *);
index 2c0a10d..b9cd706 100644 (file)
@@ -238,7 +238,6 @@ tu102_disp_init(struct nvkm_disp *disp)
 
 static const struct nvkm_disp_func
 tu102_disp = {
-       .dtor = nv50_disp_dtor,
        .oneinit = nv50_disp_oneinit,
        .init = tu102_disp_init,
        .fini = gv100_disp_fini,
@@ -264,5 +263,5 @@ int
 tu102_disp_new(struct nvkm_device *device, enum nvkm_subdev_type type, int inst,
               struct nvkm_disp **pdisp)
 {
-       return nv50_disp_new_(&tu102_disp, device, type, inst, pdisp);
+       return nvkm_disp_new_(&tu102_disp, device, type, inst, pdisp);
 }