drm/nouveau/gr/gf100: load firmware in outer function
authorAlexandre Courbot <acourbot@nvidia.com>
Wed, 24 Feb 2016 05:42:16 +0000 (14:42 +0900)
committerBen Skeggs <bskeggs@redhat.com>
Mon, 14 Mar 2016 00:13:20 +0000 (10:13 +1000)
The firmwares required by GR may vary from chip to chip, especially with
the introduction of secure boot and NVIDIA-provided firmwares. Move the
firmware loading outside of gf100_gr_ctor so other chips may still call
it while managing their firmwares themselves.

Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nvkm/engine/gr/gf100.c
drivers/gpu/drm/nouveau/nvkm/engine/gr/gk20a.c

index 7a8a0c7..3fc5f7d 100644 (file)
@@ -1762,15 +1762,6 @@ gf100_gr_ctor(const struct gf100_gr_func *func, struct nvkm_device *device,
        if (ret)
                return ret;
 
-       if (gr->firmware) {
-               nvkm_info(&gr->base.engine.subdev, "using external firmware\n");
-               if (gf100_gr_ctor_fw(gr, "fecs_inst", &gr->fuc409c) ||
-                   gf100_gr_ctor_fw(gr, "fecs_data", &gr->fuc409d) ||
-                   gf100_gr_ctor_fw(gr, "gpccs_inst", &gr->fuc41ac) ||
-                   gf100_gr_ctor_fw(gr, "gpccs_data", &gr->fuc41ad))
-                       return -ENODEV;
-       }
-
        return 0;
 }
 
@@ -1779,10 +1770,25 @@ gf100_gr_new_(const struct gf100_gr_func *func, struct nvkm_device *device,
              int index, struct nvkm_gr **pgr)
 {
        struct gf100_gr *gr;
+       int ret;
+
        if (!(gr = kzalloc(sizeof(*gr), GFP_KERNEL)))
                return -ENOMEM;
        *pgr = &gr->base;
-       return gf100_gr_ctor(func, device, index, gr);
+
+       ret = gf100_gr_ctor(func, device, index, gr);
+       if (ret)
+               return ret;
+
+       if (gr->firmware) {
+               if (gf100_gr_ctor_fw(gr, "fecs_inst", &gr->fuc409c) ||
+                   gf100_gr_ctor_fw(gr, "fecs_data", &gr->fuc409d) ||
+                   gf100_gr_ctor_fw(gr, "gpccs_inst", &gr->fuc41ac) ||
+                   gf100_gr_ctor_fw(gr, "gpccs_data", &gr->fuc41ad))
+                       return -ENODEV;
+       }
+
+       return 0;
 }
 
 int
index a321dd0..a55331d 100644 (file)
@@ -283,6 +283,12 @@ gk20a_gr_new_(const struct gf100_gr_func *func, struct nvkm_device *device,
        if (ret)
                return ret;
 
+       if (gf100_gr_ctor_fw(gr, "fecs_inst", &gr->fuc409c) ||
+           gf100_gr_ctor_fw(gr, "fecs_data", &gr->fuc409d) ||
+           gf100_gr_ctor_fw(gr, "gpccs_inst", &gr->fuc41ac) ||
+           gf100_gr_ctor_fw(gr, "gpccs_data", &gr->fuc41ad))
+               return -ENODEV;
+
        ret = gf100_gr_ctor_fw(gr, "sw_nonctx", &fuc);
        if (ret)
                return ret;