drm/nouveau/secboot/gp102-: remove WAR for SEC2 RTOS start bug
authorBen Skeggs <bskeggs@redhat.com>
Tue, 2 Jul 2019 06:29:40 +0000 (16:29 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Fri, 19 Jul 2019 06:26:51 +0000 (16:26 +1000)
Appears to be fixed by "flcn/gp102-: improve implementation of
bind_context() on SEC2/GSP".

Tested on GP10[24678] and GV100.

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nvkm/subdev/secboot/ls_ucode_msgqueue.c

index 77c13b096a67833b6b8c945554df370994d5a1d8..a84a999445bb5fbb6534ce61d70fec646440d2bb 100644 (file)
@@ -164,41 +164,12 @@ acr_ls_sec2_post_run(const struct nvkm_acr *acr, const struct nvkm_secboot *sb)
        struct nvkm_sec2 *sec = device->sec2;
        /* on SEC arguments are always at the beginning of EMEM */
        const u32 addr_args = 0x01000000;
-       u32 reg;
        int ret;
 
        ret = acr_ls_msgqueue_post_run(sec->queue, sec->falcon, addr_args);
        if (ret)
                return ret;
 
-       /*
-        * There is a bug where the LS firmware sometimes require to be started
-        * twice (this happens only on SEC). Detect and workaround that
-        * condition.
-        *
-        * Once started, the falcon will end up in STOPPED condition (bit 5)
-        * if successful, or in HALT condition (bit 4) if not.
-        */
-       nvkm_msec(device, 1,
-                 if ((reg = nvkm_falcon_rd32(sb->boot_falcon, 0x100) & 0x30) != 0)
-                         break;
-       );
-       if (reg & BIT(4)) {
-               nvkm_debug(subdev, "applying workaround for start bug...\n");
-               nvkm_falcon_start(sb->boot_falcon);
-               nvkm_msec(subdev->device, 1,
-                       if ((reg = nvkm_rd32(subdev->device,
-                                            sb->boot_falcon->addr + 0x100)
-                            & 0x30) != 0)
-                               break;
-               );
-               if (reg & BIT(4)) {
-                       nvkm_error(subdev, "%s failed to start\n",
-                              nvkm_secboot_falcon_name[acr->boot_falcon]);
-                       return -EINVAL;
-               }
-       }
-
        nvkm_debug(&sb->subdev, "%s started\n",
                   nvkm_secboot_falcon_name[acr->boot_falcon]);