]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
drm/nouveau/pmu: initialise SW state for falcon from constructor
authorBen Skeggs <bskeggs@redhat.com>
Tue, 14 Jan 2020 20:34:21 +0000 (06:34 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Wed, 15 Jan 2020 00:50:26 +0000 (10:50 +1000)
This will allow us to register the falcon with ACR, and further customise
its behaviour by providing the nvkm_falcon_func structure directly.

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
20 files changed:
drivers/gpu/drm/nouveau/include/nvkm/subdev/pmu.h
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/base.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gf100.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gf119.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gk104.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gk110.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gk208.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gk20a.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gm107.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gm20b.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gp100.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gp102.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gp10b.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/gt215.c
drivers/gpu/drm/nouveau/nvkm/subdev/pmu/priv.h
drivers/gpu/drm/nouveau/nvkm/subdev/secboot/acr_r352.c
drivers/gpu/drm/nouveau/nvkm/subdev/secboot/acr_r361.c
drivers/gpu/drm/nouveau/nvkm/subdev/secboot/acr_r375.c
drivers/gpu/drm/nouveau/nvkm/subdev/secboot/base.c
drivers/gpu/drm/nouveau/nvkm/subdev/secboot/ls_ucode_msgqueue.c

index fd250bc48a416c7e4942d16559798e981c2853bb..0b72d63b24272cf79c58ec5a38caece8ebae91e1 100644 (file)
@@ -2,12 +2,12 @@
 #ifndef __NVKM_PMU_H__
 #define __NVKM_PMU_H__
 #include <core/subdev.h>
-#include <engine/falcon.h>
+#include <core/falcon.h>
 
 struct nvkm_pmu {
        const struct nvkm_pmu_func *func;
        struct nvkm_subdev subdev;
-       struct nvkm_falcon *falcon;
+       struct nvkm_falcon falcon;
        struct nvkm_msgqueue *queue;
 
        struct {
index 1c308227ad6b7e1d431803d67083b138fb487330..60e14f9a5415b30af6d3731591648eee6448a9a5 100644 (file)
@@ -134,19 +134,12 @@ nvkm_pmu_init(struct nvkm_subdev *subdev)
        return ret;
 }
 
-static int
-nvkm_pmu_oneinit(struct nvkm_subdev *subdev)
-{
-       struct nvkm_pmu *pmu = nvkm_pmu(subdev);
-       return nvkm_falcon_v1_new(&pmu->subdev, "PMU", 0x10a000, &pmu->falcon);
-}
-
 static void *
 nvkm_pmu_dtor(struct nvkm_subdev *subdev)
 {
        struct nvkm_pmu *pmu = nvkm_pmu(subdev);
        nvkm_msgqueue_del(&pmu->queue);
-       nvkm_falcon_del(&pmu->falcon);
+       nvkm_falcon_dtor(&pmu->falcon);
        return nvkm_pmu(subdev);
 }
 
@@ -154,7 +147,6 @@ static const struct nvkm_subdev_func
 nvkm_pmu = {
        .dtor = nvkm_pmu_dtor,
        .preinit = nvkm_pmu_preinit,
-       .oneinit = nvkm_pmu_oneinit,
        .init = nvkm_pmu_init,
        .fini = nvkm_pmu_fini,
        .intr = nvkm_pmu_intr,
@@ -174,7 +166,10 @@ nvkm_pmu_ctor(const struct nvkm_pmu_fwif *fwif, struct nvkm_device *device,
                return PTR_ERR(fwif);
 
        pmu->func = fwif->func;
-       return 0;
+
+       return nvkm_falcon_ctor(pmu->func->flcn, &pmu->subdev,
+                               nvkm_subdev_name[pmu->subdev.index], 0x10a000,
+                               &pmu->falcon);
 }
 
 int
index bc48dfcfcb0c717d3a4a7d09d87fdceea8c9b1f8..3ecb3d9cbcf234674a42c22d313f10c779cbf1df 100644 (file)
@@ -42,6 +42,7 @@ gf100_pmu_enabled(struct nvkm_pmu *pmu)
 
 static const struct nvkm_pmu_func
 gf100_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .code.data = gf100_pmu_code,
        .code.size = sizeof(gf100_pmu_code),
        .data.data = gf100_pmu_data,
index b4c81e2727060665d56e2e9185eb0f9a8c5431ad..8dd0271aaaee6af471fad5417811d5082189cd39 100644 (file)
@@ -26,6 +26,7 @@
 
 static const struct nvkm_pmu_func
 gf119_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .code.data = gf119_pmu_code,
        .code.size = sizeof(gf119_pmu_code),
        .data.data = gf119_pmu_data,
index c497525d65ca7a0be77d65e3c6c08e6a98d57610..8b70cc17a6341cd3eeb3820738532bb2283097ac 100644 (file)
@@ -105,6 +105,7 @@ gk104_pmu_pgob(struct nvkm_pmu *pmu, bool enable)
 
 static const struct nvkm_pmu_func
 gk104_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .code.data = gk104_pmu_code,
        .code.size = sizeof(gk104_pmu_code),
        .data.data = gk104_pmu_data,
index aa0b9e7c5a99a353229c9e3272d16b2d163bcdfd..0081f2141b1083abf7fa8b67150150781edb05a9 100644 (file)
@@ -84,6 +84,7 @@ gk110_pmu_pgob(struct nvkm_pmu *pmu, bool enable)
 
 static const struct nvkm_pmu_func
 gk110_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .code.data = gk110_pmu_code,
        .code.size = sizeof(gk110_pmu_code),
        .data.data = gk110_pmu_data,
index 47a5ed32dcc561760d156f86d8e88cc54daf087d..b227c701a5e7c6d9c81d1dd356c0f73bf09e430e 100644 (file)
@@ -26,6 +26,7 @@
 
 static const struct nvkm_pmu_func
 gk208_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .code.data = gk208_pmu_code,
        .code.size = sizeof(gk208_pmu_code),
        .data.data = gk208_pmu_data,
index 67f65c54c7fdc706b47e5b36c095dc4425d731cc..26c1adf8f44c6f742c6baeededf2f52636907a37 100644 (file)
@@ -95,7 +95,7 @@ static void
 gk20a_pmu_dvfs_get_dev_status(struct gk20a_pmu *pmu,
                              struct gk20a_pmu_dvfs_dev_status *status)
 {
-       struct nvkm_falcon *falcon = pmu->base.falcon;
+       struct nvkm_falcon *falcon = &pmu->base.falcon;
 
        status->busy = nvkm_falcon_rd32(falcon, 0x508 + (BUSY_SLOT * 0x10));
        status->total= nvkm_falcon_rd32(falcon, 0x508 + (CLK_SLOT * 0x10));
@@ -104,7 +104,7 @@ gk20a_pmu_dvfs_get_dev_status(struct gk20a_pmu *pmu,
 static void
 gk20a_pmu_dvfs_reset_dev_status(struct gk20a_pmu *pmu)
 {
-       struct nvkm_falcon *falcon = pmu->base.falcon;
+       struct nvkm_falcon *falcon = &pmu->base.falcon;
 
        nvkm_falcon_wr32(falcon, 0x508 + (BUSY_SLOT * 0x10), 0x80000000);
        nvkm_falcon_wr32(falcon, 0x508 + (CLK_SLOT * 0x10), 0x80000000);
@@ -160,7 +160,7 @@ gk20a_pmu_fini(struct nvkm_pmu *pmu)
        struct gk20a_pmu *gpmu = gk20a_pmu(pmu);
        nvkm_timer_alarm(pmu->subdev.device->timer, 0, &gpmu->alarm);
 
-       nvkm_falcon_put(pmu->falcon, &pmu->subdev);
+       nvkm_falcon_put(&pmu->falcon, &pmu->subdev);
 }
 
 static int
@@ -169,7 +169,7 @@ gk20a_pmu_init(struct nvkm_pmu *pmu)
        struct gk20a_pmu *gpmu = gk20a_pmu(pmu);
        struct nvkm_subdev *subdev = &pmu->subdev;
        struct nvkm_device *device = pmu->subdev.device;
-       struct nvkm_falcon *falcon = pmu->falcon;
+       struct nvkm_falcon *falcon = &pmu->falcon;
        int ret;
 
        ret = nvkm_falcon_get(falcon, subdev);
@@ -196,6 +196,7 @@ gk20a_dvfs_data= {
 
 static const struct nvkm_pmu_func
 gk20a_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .enabled = gf100_pmu_enabled,
        .init = gk20a_pmu_init,
        .fini = gk20a_pmu_fini,
index 7dd4bec148d2038703f1aed0b489f7daf00645cd..5afb55e58b517e406451a8203066e664629928c3 100644 (file)
@@ -28,6 +28,7 @@
 
 static const struct nvkm_pmu_func
 gm107_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .code.data = gm107_pmu_code,
        .code.size = sizeof(gm107_pmu_code),
        .data.data = gm107_pmu_data,
index 1a7cfdef2d9c44ff88db24ce5d746be55d097f1f..5c262a28aa23119a89d1468c878b141b67a5387c 100644 (file)
@@ -41,6 +41,7 @@ gm20b_pmu_recv(struct nvkm_pmu *pmu)
 
 static const struct nvkm_pmu_func
 gm20b_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .enabled = gf100_pmu_enabled,
        .intr = gt215_pmu_intr,
        .recv = gm20b_pmu_recv,
@@ -55,7 +56,7 @@ MODULE_FIRMWARE("nvidia/gm20b/pmu/sig.bin");
 int
 gm20b_pmu_load(struct nvkm_pmu *pmu, int ver, const struct nvkm_pmu_fwif *fwif)
 {
-       return nvkm_acr_lsfw_load_sig_image_desc(&pmu->subdev, pmu->falcon,
+       return nvkm_acr_lsfw_load_sig_image_desc(&pmu->subdev, &pmu->falcon,
                                                 NVKM_ACR_LSF_PMU, "pmu/",
                                                 ver, fwif->acr);
 }
index 3b7f26c187f4c173b06f0a43dd88723b27c28aa1..09e05db21ff5b8287518780fd72c6b387b67ec95 100644 (file)
@@ -25,6 +25,7 @@
 
 static const struct nvkm_pmu_func
 gp100_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .enabled = gf100_pmu_enabled,
        .reset = gf100_pmu_reset,
 };
index 08fda8088c493961124d218d6037d461591d0b54..262b8a3dd5079db5c487f58871b82b283045f562 100644 (file)
@@ -39,6 +39,7 @@ gp102_pmu_enabled(struct nvkm_pmu *pmu)
 
 static const struct nvkm_pmu_func
 gp102_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .enabled = gp102_pmu_enabled,
        .reset = gp102_pmu_reset,
 };
index 2440e230f3988cab57224068bc6afb18b9197906..7134ef9d91afed9405dbf1b944e597c5b1408baf 100644 (file)
@@ -28,6 +28,7 @@ gp10b_pmu_acr = {
 
 static const struct nvkm_pmu_func
 gp10b_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .enabled = gf100_pmu_enabled,
        .intr = gt215_pmu_intr,
        .recv = gm20b_pmu_recv,
index f52d7519b8e0a7075888176d8f916512ea5fa36d..073558972e42d931b241d47e635d59b29f6a94c1 100644 (file)
@@ -241,8 +241,23 @@ gt215_pmu_init(struct nvkm_pmu *pmu)
        return 0;
 }
 
+const struct nvkm_falcon_func
+gt215_pmu_flcn = {
+       .load_imem = nvkm_falcon_v1_load_imem,
+       .load_dmem = nvkm_falcon_v1_load_dmem,
+       .read_dmem = nvkm_falcon_v1_read_dmem,
+       .bind_context = nvkm_falcon_v1_bind_context,
+       .wait_for_halt = nvkm_falcon_v1_wait_for_halt,
+       .clear_interrupt = nvkm_falcon_v1_clear_interrupt,
+       .set_start_addr = nvkm_falcon_v1_set_start_addr,
+       .start = nvkm_falcon_v1_start,
+       .enable = nvkm_falcon_v1_enable,
+       .disable = nvkm_falcon_v1_disable,
+};
+
 static const struct nvkm_pmu_func
 gt215_pmu = {
+       .flcn = &gt215_pmu_flcn,
        .code.data = gt215_pmu_code,
        .code.size = sizeof(gt215_pmu_code),
        .data.data = gt215_pmu_data,
index 32b2182ce67f29155e9287bb60a1df2db09decdb..91b4a2d3ad4c9d2f171f993124e1bb21bafad120 100644 (file)
@@ -6,6 +6,8 @@
 #include <subdev/pmu/fuc/os.h>
 
 struct nvkm_pmu_func {
+       const struct nvkm_falcon_func *flcn;
+
        struct {
                u32 *data;
                u32  size;
@@ -27,6 +29,7 @@ struct nvkm_pmu_func {
        void (*pgob)(struct nvkm_pmu *, bool);
 };
 
+extern const struct nvkm_falcon_func gt215_pmu_flcn;
 int gt215_pmu_init(struct nvkm_pmu *);
 void gt215_pmu_fini(struct nvkm_pmu *);
 void gt215_pmu_intr(struct nvkm_pmu *);
index 7af971db91bce2121495872cff5188fe9a6ff6ce..53770ecdbb2f75dd87cdb5f393434f18627798dd 100644 (file)
@@ -1150,7 +1150,7 @@ acr_r352_generate_pmu_bl_desc(const struct nvkm_acr *acr,
        base = wpr_addr + img->ucode_off + pdesc->app_start_offset;
        addr_code = (base + pdesc->app_resident_code_offset) >> 8;
        addr_data = (base + pdesc->app_resident_data_offset) >> 8;
-       addr_args = pmu->falcon->data.limit;
+       addr_args = pmu->falcon.data.limit;
        addr_args -= NVKM_MSGQUEUE_CMDLINE_SIZE;
 
        desc->dma_idx = FALCON_DMAIDX_UCODE;
index f6b2d20d7fc3dd3d098fcfbc7577149a0f7580ac..513445856f22eef3673b80dbb1a88da8066c7216 100644 (file)
@@ -126,7 +126,7 @@ acr_r361_generate_pmu_bl_desc(const struct nvkm_acr *acr,
        base = wpr_addr + img->ucode_off + pdesc->app_start_offset;
        addr_code = base + pdesc->app_resident_code_offset;
        addr_data = base + pdesc->app_resident_data_offset;
-       addr_args = pmu->falcon->data.limit;
+       addr_args = pmu->falcon.data.limit;
        addr_args -= NVKM_MSGQUEUE_CMDLINE_SIZE;
 
        desc->dma_idx = FALCON_DMAIDX_UCODE;
index 8f0647766038604635d90def01909cc5be75e378..7112ad74e3c8084d9d17780b26111f4ed6579594 100644 (file)
@@ -40,7 +40,7 @@ acr_r375_generate_pmu_bl_desc(const struct nvkm_acr *acr,
        base = wpr_addr + img->ucode_off + pdesc->app_start_offset;
        addr_code = base + pdesc->app_resident_code_offset;
        addr_data = base + pdesc->app_resident_data_offset;
-       addr_args = pmu->falcon->data.limit;
+       addr_args = pmu->falcon.data.limit;
        addr_args -= NVKM_MSGQUEUE_CMDLINE_SIZE;
 
        desc->ctx_dma = FALCON_DMAIDX_UCODE;
index ee29c6c11afd524a10b8d3dc9eeb4fcca37e7d41..36ae6918beaee7c5bd6cc28879f209a8c5f803f5 100644 (file)
@@ -133,13 +133,13 @@ nvkm_secboot_oneinit(struct nvkm_subdev *subdev)
 
        switch (sb->acr->boot_falcon) {
        case NVKM_SECBOOT_FALCON_PMU:
-               sb->halt_falcon = sb->boot_falcon = subdev->device->pmu->falcon;
+               sb->halt_falcon = sb->boot_falcon = &subdev->device->pmu->falcon;
                break;
        case NVKM_SECBOOT_FALCON_SEC2:
                /* we must keep SEC2 alive forever since ACR will run on it */
                nvkm_engine_ref(&subdev->device->sec2->engine);
                sb->boot_falcon = subdev->device->sec2->falcon;
-               sb->halt_falcon = subdev->device->pmu->falcon;
+               sb->halt_falcon = &subdev->device->pmu->falcon;
                break;
        default:
                nvkm_error(subdev, "Unmanaged boot falcon %s!\n",
index a84a999445bb5fbb6534ce61d70fec646440d2bb..472f38a566393f5e1358597538ad01ae20189026 100644 (file)
@@ -110,7 +110,7 @@ acr_ls_ucode_load_pmu(const struct nvkm_secboot *sb, int maxver,
                return ret;
 
        /* Allocate the PMU queue corresponding to the FW version */
-       ret = nvkm_msgqueue_new(img->ucode_desc.app_version, pmu->falcon,
+       ret = nvkm_msgqueue_new(img->ucode_desc.app_version, &pmu->falcon,
                                sb, &pmu->queue);
        if (ret)
                return ret;
@@ -123,10 +123,10 @@ acr_ls_pmu_post_run(const struct nvkm_acr *acr, const struct nvkm_secboot *sb)
 {
        struct nvkm_device *device = sb->subdev.device;
        struct nvkm_pmu *pmu = device->pmu;
-       u32 addr_args = pmu->falcon->data.limit - NVKM_MSGQUEUE_CMDLINE_SIZE;
+       u32 addr_args = pmu->falcon.data.limit - NVKM_MSGQUEUE_CMDLINE_SIZE;
        int ret;
 
-       ret = acr_ls_msgqueue_post_run(pmu->queue, pmu->falcon, addr_args);
+       ret = acr_ls_msgqueue_post_run(pmu->queue, &pmu->falcon, addr_args);
        if (ret)
                return ret;