drm/nouveau/disp: replace hda func pointer check with flag
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)
Simpler, and less error-prone than a separate set of function pointers.

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
17 files changed:
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/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/gt215.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/gv100.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/ior.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/ior.h
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/outp.c
drivers/gpu/drm/nouveau/nvkm/engine/disp/tu102.c

index af5d35149412ad9e0a411cdff2068347287d129e..43dc73abad619f74e0b94d9e821539e89865866e 100644 (file)
@@ -109,7 +109,7 @@ g84_sor = {
 int
 g84_sor_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&g84_sor, disp, SOR, id);
+       return nvkm_ior_new_(&g84_sor, disp, SOR, id, false);
 }
 
 static const struct nvkm_disp_mthd_list
index 5c9a769ff3bdf3e028a76f89c9feca7920e7078d..92feface3624183db939735273f89bb404d0cb66 100644 (file)
@@ -298,7 +298,7 @@ g94_sor = {
 static int
 g94_sor_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&g94_sor, disp, SOR, id);
+       return nvkm_ior_new_(&g94_sor, disp, SOR, id, false);
 }
 
 int
index c693ad09fc8314095e0054c15148650e8b4fc9be..9d008f090efe290baac3786f2cdb77d9c26fc122 100644 (file)
@@ -84,7 +84,7 @@ ga102_sor_clock(struct nvkm_ior *sor)
 }
 
 static const struct nvkm_ior_func
-ga102_sor_hda = {
+ga102_sor = {
        .route = {
                .get = gm200_sor_route_get,
                .set = gm200_sor_route_set,
@@ -114,40 +114,13 @@ ga102_sor_hda = {
        },
 };
 
-static const struct nvkm_ior_func
-ga102_sor = {
-       .route = {
-               .get = gm200_sor_route_get,
-               .set = gm200_sor_route_set,
-       },
-       .state = gv100_sor_state,
-       .power = nv50_sor_power,
-       .clock = ga102_sor_clock,
-       .hdmi = {
-               .ctrl = gv100_sor_hdmi_ctrl,
-               .scdc = gm200_sor_hdmi_scdc,
-       },
-       .dp = {
-               .lanes = { 0, 1, 2, 3 },
-               .links = ga102_sor_dp_links,
-               .power = g94_sor_dp_power,
-               .pattern = gm107_sor_dp_pattern,
-               .drive = gm200_sor_dp_drive,
-               .vcpi = tu102_sor_dp_vcpi,
-               .audio = gv100_sor_dp_audio,
-               .audio_sym = gv100_sor_dp_audio_sym,
-               .watermark = gv100_sor_dp_watermark,
-       },
-};
-
 static int
 ga102_sor_new(struct nvkm_disp *disp, int id)
 {
        struct nvkm_device *device = disp->engine.subdev.device;
        u32 hda = nvkm_rd32(device, 0x08a15c);
-       if (hda & BIT(id))
-               return nvkm_ior_new_(&ga102_sor_hda, disp, SOR, id);
-       return nvkm_ior_new_(&ga102_sor, disp, SOR, id);
+
+       return nvkm_ior_new_(&ga102_sor, disp, SOR, id, hda & BIT(id));
 }
 
 static const struct nvkm_disp_func
index 75b7b41d572758d747cae53ba713480c46fdf3c4..e6ebc72ed6dbab6046fe190153cb9f570814524a 100644 (file)
@@ -309,7 +309,7 @@ gf119_sor = {
 static int
 gf119_sor_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&gf119_sor, disp, SOR, id);
+       return nvkm_ior_new_(&gf119_sor, disp, SOR, id, true);
 }
 
 int
@@ -357,7 +357,7 @@ gf119_dac = {
 int
 gf119_dac_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&gf119_dac, disp, DAC, id);
+       return nvkm_ior_new_(&gf119_dac, disp, DAC, id, false);
 }
 
 int
index 1c9a5e0d1dddd51233f5d085c160bb96bfbda1e2..b562a4d8ad2ffcbd42a6550409ef204684a0ef1b 100644 (file)
@@ -116,7 +116,7 @@ gk104_sor = {
 int
 gk104_sor_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&gk104_sor, disp, SOR, id);
+       return nvkm_ior_new_(&gk104_sor, disp, SOR, id, true);
 }
 
 static const struct nvkm_disp_mthd_list
index 301d7a9692c8806a2fd6f24b923d3dbdce92bb6a..0b25999e0e25f002562ce51fb4764ce6370cb4d1 100644 (file)
@@ -81,7 +81,7 @@ gm107_sor = {
 static int
 gm107_sor_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&gm107_sor, disp, SOR, id);
+       return nvkm_ior_new_(&gm107_sor, disp, SOR, id, true);
 }
 
 static const struct nvkm_disp_func
index c65d85b4299eb303a1aafd8210a1710461a4682d..990f3782d0c34207bd6c9b1f620ae57660448d3a 100644 (file)
@@ -110,7 +110,7 @@ gm200_sor_route_get(struct nvkm_outp *outp, int *link)
 }
 
 static const struct nvkm_ior_func
-gm200_sor_hda = {
+gm200_sor = {
        .route = {
                .get = gm200_sor_route_get,
                .set = gm200_sor_route_set,
@@ -140,32 +140,6 @@ gm200_sor_hda = {
        },
 };
 
-static const struct nvkm_ior_func
-gm200_sor = {
-       .route = {
-               .get = gm200_sor_route_get,
-               .set = gm200_sor_route_set,
-       },
-       .state = gf119_sor_state,
-       .power = nv50_sor_power,
-       .clock = gf119_sor_clock,
-       .hdmi = {
-               .ctrl = gk104_sor_hdmi_ctrl,
-               .scdc = gm200_sor_hdmi_scdc,
-       },
-       .dp = {
-               .lanes = { 0, 1, 2, 3 },
-               .links = gf119_sor_dp_links,
-               .power = g94_sor_dp_power,
-               .pattern = gm107_sor_dp_pattern,
-               .drive = gm200_sor_dp_drive,
-               .vcpi = gf119_sor_dp_vcpi,
-               .audio = gf119_sor_dp_audio,
-               .audio_sym = gf119_sor_dp_audio_sym,
-               .watermark = gf119_sor_dp_watermark,
-       },
-};
-
 static int
 gm200_sor_new(struct nvkm_disp *disp, int id)
 {
@@ -175,10 +149,7 @@ gm200_sor_new(struct nvkm_disp *disp, int id)
        if (!((hda = nvkm_rd32(device, 0x08a15c)) & 0x40000000))
                hda = nvkm_rd32(device, 0x101034);
 
-       if (hda & BIT(id))
-               return nvkm_ior_new_(&gm200_sor_hda, disp, SOR, id);
-
-       return nvkm_ior_new_(&gm200_sor, disp, SOR, id);
+       return nvkm_ior_new_(&gm200_sor, disp, SOR, id, hda & BIT(id));
 }
 
 static const struct nvkm_disp_func
index 603b7188f1cf144ad846c1a77e3799ac1d2d429c..beec3c8187a2cf46059269b66b7832fd5895e5cc 100644 (file)
@@ -29,7 +29,7 @@
 #include <nvif/class.h>
 
 static const struct nvkm_ior_func
-gp100_sor_hda = {
+gp100_sor = {
        .route = {
                .get = gm200_sor_route_get,
                .set = gm200_sor_route_set,
@@ -59,32 +59,6 @@ gp100_sor_hda = {
        },
 };
 
-static const struct nvkm_ior_func
-gp100_sor = {
-       .route = {
-               .get = gm200_sor_route_get,
-               .set = gm200_sor_route_set,
-       },
-       .state = gf119_sor_state,
-       .power = nv50_sor_power,
-       .clock = gf119_sor_clock,
-       .hdmi = {
-               .ctrl = gk104_sor_hdmi_ctrl,
-               .scdc = gm200_sor_hdmi_scdc,
-       },
-       .dp = {
-               .lanes = { 0, 1, 2, 3 },
-               .links = gf119_sor_dp_links,
-               .power = g94_sor_dp_power,
-               .pattern = gm107_sor_dp_pattern,
-               .drive = gm200_sor_dp_drive,
-               .vcpi = gf119_sor_dp_vcpi,
-               .audio = gf119_sor_dp_audio,
-               .audio_sym = gf119_sor_dp_audio_sym,
-               .watermark = gf119_sor_dp_watermark,
-       },
-};
-
 int
 gp100_sor_new(struct nvkm_disp *disp, int id)
 {
@@ -94,9 +68,7 @@ gp100_sor_new(struct nvkm_disp *disp, int id)
        if (!((hda = nvkm_rd32(device, 0x08a15c)) & 0x40000000))
                hda = nvkm_rd32(device, 0x10ebb0) >> 8;
 
-       if (hda & BIT(id))
-               return nvkm_ior_new_(&gp100_sor_hda, disp, SOR, id);
-       return nvkm_ior_new_(&gp100_sor, disp, SOR, id);
+       return nvkm_ior_new_(&gp100_sor, disp, SOR, id, hda & BIT(id));
 }
 
 static const struct nvkm_disp_func
index 98d99e1c01b23d6ef0dacab1a244b536e2be9815..72a4c28ccd185d87b4334bd547dcf5f250df451d 100644 (file)
@@ -168,7 +168,7 @@ gt215_sor = {
 static int
 gt215_sor_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&gt215_sor, disp, SOR, id);
+       return nvkm_ior_new_(&gt215_sor, disp, SOR, id, true);
 }
 
 static const struct nvkm_disp_func
index 18c40e398ea57eb4560255ae3739d108f241d55e..aac1cdbea1602496dd6e80f1d3b2e91eabca77ff 100644 (file)
@@ -165,7 +165,7 @@ gv100_sor_state(struct nvkm_ior *sor, struct nvkm_ior_state *state)
 }
 
 static const struct nvkm_ior_func
-gv100_sor_hda = {
+gv100_sor = {
        .route = {
                .get = gm200_sor_route_get,
                .set = gm200_sor_route_set,
@@ -194,31 +194,6 @@ gv100_sor_hda = {
        },
 };
 
-static const struct nvkm_ior_func
-gv100_sor = {
-       .route = {
-               .get = gm200_sor_route_get,
-               .set = gm200_sor_route_set,
-       },
-       .state = gv100_sor_state,
-       .power = nv50_sor_power,
-       .clock = gf119_sor_clock,
-       .hdmi = {
-               .ctrl = gv100_sor_hdmi_ctrl,
-               .scdc = gm200_sor_hdmi_scdc,
-       },
-       .dp = {
-               .lanes = { 0, 1, 2, 3 },
-               .links = gf119_sor_dp_links,
-               .power = g94_sor_dp_power,
-               .pattern = gm107_sor_dp_pattern,
-               .drive = gm200_sor_dp_drive,
-               .audio = gv100_sor_dp_audio,
-               .audio_sym = gv100_sor_dp_audio_sym,
-               .watermark = gv100_sor_dp_watermark,
-       },
-};
-
 static int
 gv100_sor_new(struct nvkm_disp *disp, int id)
 {
@@ -228,10 +203,7 @@ gv100_sor_new(struct nvkm_disp *disp, int id)
        if (!((hda = nvkm_rd32(device, 0x08a15c)) & 0x40000000))
                hda = nvkm_rd32(device, 0x118fb0) >> 8;
 
-       if (hda & BIT(id))
-               return nvkm_ior_new_(&gv100_sor_hda, disp, SOR, id);
-
-       return nvkm_ior_new_(&gv100_sor, disp, SOR, id);
+       return nvkm_ior_new_(&gv100_sor, disp, SOR, id, hda & BIT(id));
 }
 
 int
index 1963cc183b87276c72cf41340496f4f4bfc9625d..e420bf2e433096e53ea3eaddadb46b055984edeb 100644 (file)
@@ -55,7 +55,7 @@ nvkm_ior_del(struct nvkm_ior **pior)
 
 int
 nvkm_ior_new_(const struct nvkm_ior_func *func, struct nvkm_disp *disp,
-             enum nvkm_ior_type type, int id)
+             enum nvkm_ior_type type, int id, bool hda)
 {
        struct nvkm_ior *ior;
        if (!(ior = kzalloc(sizeof(*ior), GFP_KERNEL)))
@@ -64,6 +64,7 @@ nvkm_ior_new_(const struct nvkm_ior_func *func, struct nvkm_disp *disp,
        ior->disp = disp;
        ior->type = type;
        ior->id = id;
+       ior->hda = hda;
        snprintf(ior->name, sizeof(ior->name), "%s-%d", nvkm_ior_name[ior->type], ior->id);
        list_add_tail(&ior->head, &disp->iors);
        IOR_DBG(ior, "ctor");
index a8bd12a6bbf9da594515d702e759270a73c76847..d1e494e06c382522740c754aba6040614f5bc599 100644 (file)
@@ -13,6 +13,7 @@ struct nvkm_ior {
                PIOR,
        } type;
        int id;
+       bool hda;
        char name[8];
 
        struct list_head head;
@@ -93,7 +94,7 @@ struct nvkm_ior_func {
 };
 
 int nvkm_ior_new_(const struct nvkm_ior_func *func, struct nvkm_disp *,
-                 enum nvkm_ior_type type, int id);
+                 enum nvkm_ior_type type, int id, bool hda);
 void nvkm_ior_del(struct nvkm_ior **);
 struct nvkm_ior *nvkm_ior_find(struct nvkm_disp *, enum nvkm_ior_type, int id);
 
index fb8bfb98c105ba9a4d3b345d2e3cfc6878c815d9..f0d23a66b7824ffb36a342bacfb8ff84bb545f0f 100644 (file)
@@ -49,7 +49,7 @@ mcp77_sor = {
 static int
 mcp77_sor_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&mcp77_sor, disp, SOR, id);
+       return nvkm_ior_new_(&mcp77_sor, disp, SOR, id, false);
 }
 
 static const struct nvkm_disp_func
index 8b9e97665805cbde72fb5adb30c371b8942006e5..25a1934f688221458f808c8543fbd60dfd7cc609 100644 (file)
@@ -54,7 +54,7 @@ mcp89_sor = {
 static int
 mcp89_sor_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&mcp89_sor, disp, SOR, id);
+       return nvkm_ior_new_(&mcp89_sor, disp, SOR, id, true);
 }
 
 static const struct nvkm_disp_func
index 1a3151528e7ee798737f4b16fbb3b415c90e5786..a50239d2077fbfc39db202a98e16a70982d1f26a 100644 (file)
@@ -147,7 +147,7 @@ nv50_pior = {
 int
 nv50_pior_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&nv50_pior, disp, PIOR, id);
+       return nvkm_ior_new_(&nv50_pior, disp, PIOR, id, false);
 }
 
 int
@@ -228,7 +228,7 @@ nv50_sor = {
 static int
 nv50_sor_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&nv50_sor, disp, SOR, id);
+       return nvkm_ior_new_(&nv50_sor, disp, SOR, id, false);
 }
 
 int
@@ -324,7 +324,7 @@ nv50_dac = {
 int
 nv50_dac_new(struct nvkm_disp *disp, int id)
 {
-       return nvkm_ior_new_(&nv50_dac, disp, DAC, id);
+       return nvkm_ior_new_(&nv50_dac, disp, DAC, id, false);
 }
 
 int
index f527adc2e883d16358292a2c9cd046c6b6720aa4..6094805fbd63b4237e1019054164eafa6071f91f 100644 (file)
@@ -120,7 +120,7 @@ nvkm_outp_acquire_hda(struct nvkm_outp *outp, enum nvkm_ior_type type,
 
        /* Failing that, a completely unused OR is the next best thing. */
        list_for_each_entry(ior, &outp->disp->iors, head) {
-               if (!ior->identity && !!ior->func->hda.hpd == hda &&
+               if (!ior->identity && ior->hda == hda &&
                    !ior->asy.outp && ior->type == type && !ior->arm.outp &&
                    (ior->func->route.set || ior->id == __ffs(outp->info.or)))
                        return nvkm_outp_acquire_ior(outp, user, ior);
@@ -130,7 +130,7 @@ nvkm_outp_acquire_hda(struct nvkm_outp *outp, enum nvkm_ior_type type,
         * but will be released during the next modeset.
         */
        list_for_each_entry(ior, &outp->disp->iors, head) {
-               if (!ior->identity && !!ior->func->hda.hpd == hda &&
+               if (!ior->identity && ior->hda == hda &&
                    !ior->asy.outp && ior->type == type &&
                    (ior->func->route.set || ior->id == __ffs(outp->info.or)))
                        return nvkm_outp_acquire_ior(outp, user, ior);
@@ -181,7 +181,7 @@ nvkm_outp_acquire(struct nvkm_outp *outp, u8 user, bool hda)
                         *
                         *     This warning is to make it obvious if that proves wrong.
                         */
-                       WARN_ON(hda && !ior->func->hda.hpd);
+                       WARN_ON(hda && !ior->hda);
                        return nvkm_outp_acquire_ior(outp, user, ior);
                }
        }
index b9cd706f545184919109e7a294abddbfd0786e13..febc9090632f103df7e839f072bbb19346984f1b 100644 (file)
@@ -67,7 +67,7 @@ tu102_sor_dp_links(struct nvkm_ior *sor, struct nvkm_i2c_aux *aux)
 }
 
 static const struct nvkm_ior_func
-tu102_sor_hda = {
+tu102_sor = {
        .route = {
                .get = gm200_sor_route_get,
                .set = gm200_sor_route_set,
@@ -97,40 +97,13 @@ tu102_sor_hda = {
        },
 };
 
-static const struct nvkm_ior_func
-tu102_sor = {
-       .route = {
-               .get = gm200_sor_route_get,
-               .set = gm200_sor_route_set,
-       },
-       .state = gv100_sor_state,
-       .power = nv50_sor_power,
-       .clock = gf119_sor_clock,
-       .hdmi = {
-               .ctrl = gv100_sor_hdmi_ctrl,
-               .scdc = gm200_sor_hdmi_scdc,
-       },
-       .dp = {
-               .lanes = { 0, 1, 2, 3 },
-               .links = tu102_sor_dp_links,
-               .power = g94_sor_dp_power,
-               .pattern = gm107_sor_dp_pattern,
-               .drive = gm200_sor_dp_drive,
-               .vcpi = tu102_sor_dp_vcpi,
-               .audio = gv100_sor_dp_audio,
-               .audio_sym = gv100_sor_dp_audio_sym,
-               .watermark = gv100_sor_dp_watermark,
-       },
-};
-
 static int
 tu102_sor_new(struct nvkm_disp *disp, int id)
 {
        struct nvkm_device *device = disp->engine.subdev.device;
        u32 hda = nvkm_rd32(device, 0x08a15c);
-       if (hda & BIT(id))
-               return nvkm_ior_new_(&tu102_sor_hda, disp, SOR, id);
-       return nvkm_ior_new_(&tu102_sor, disp, SOR, id);
+
+       return nvkm_ior_new_(&tu102_sor, disp, SOR, id, hda & BIT(id));
 }
 
 int