drm/nouveau/pmu: move preinit() falcon reset to devinit

Cleanup before falcon changes.

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/devinit/gm200.c b/drivers/gpu/drm/nouveau/nvkm/subdev/devinit/gm200.c
index a308b9bde4497e403377d8f048f8407ba63ef2c9..f30718d7e61af0457b719732ebf6b4117e1cdc0a 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/devinit/gm200.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/devinit/gm200.c
@@ -26,6 +26,7 @@
 #include <subdev/bios.h>
 #include <subdev/bios/bit.h>
 #include <subdev/bios/pmu.h>
+#include <subdev/pmu.h>
 #include <subdev/timer.h>
 
 static void
@@ -85,13 +86,18 @@ pmu_load(struct nv50_devinit *init, u8 type, bool post,
        struct nvkm_subdev *subdev = &init->base.subdev;
        struct nvkm_bios *bios = subdev->device->bios;
        struct nvbios_pmuR pmu;
+       int ret;
 
        if (!nvbios_pmuRm(bios, type, &pmu))
                return -EINVAL;
 
-       if (!post)
+       if (!post || !subdev->device->pmu)
                return 0;
 
+       ret = nvkm_falcon_reset(&subdev->device->pmu->falcon);
+       if (ret)
+               return ret;
+
        pmu_code(init, pmu.boot_addr_pmu, pmu.boot_addr, pmu.boot_size, false);
        pmu_code(init, pmu.code_addr_pmu, pmu.code_addr, pmu.code_size, true);
        pmu_data(init, pmu.data_addr_pmu, pmu.data_addr, pmu.data_size);

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/pmu/base.c b/drivers/gpu/drm/nouveau/nvkm/subdev/pmu/base.c
index 455e95a89259fdc3749b26edb05b2a1ef10f9502..e10f46c7e817fe7715b8b6c2bdfe934b6634c672 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/pmu/base.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/pmu/base.c
@@ -99,9 +99,6 @@ nvkm_pmu_reset(struct nvkm_pmu *pmu)
 {
        struct nvkm_device *device = pmu->subdev.device;
 
-       if (!pmu->func->enabled(pmu))
-               return;
-
        /* Reset. */
        if (pmu->func->reset)
                pmu->func->reset(pmu);
@@ -113,14 +110,6 @@ nvkm_pmu_reset(struct nvkm_pmu *pmu)
        );
 }
 
-static int
-nvkm_pmu_preinit(struct nvkm_subdev *subdev)
-{
-       struct nvkm_pmu *pmu = nvkm_pmu(subdev);
-       nvkm_pmu_reset(pmu);
-       return 0;
-}
-
 static int
 nvkm_pmu_init(struct nvkm_subdev *subdev)
 {
@@ -160,7 +149,6 @@ nvkm_pmu_dtor(struct nvkm_subdev *subdev)
 static const struct nvkm_subdev_func
 nvkm_pmu = {
        .dtor = nvkm_pmu_dtor,
-       .preinit = nvkm_pmu_preinit,
        .init = nvkm_pmu_init,
        .fini = nvkm_pmu_fini,
        .intr = nvkm_pmu_intr,