nouveau: fw: sync dma after setup is called.

When this code moved to non-coherent allocator the sync was put too
early for some firmwares which called the setup function, move the
sync down after the setup function.

Reported-by: Diogo Ivo <diogo.ivo@tecnico.ulisboa.pt>
Tested-by: Diogo Ivo <diogo.ivo@tecnico.ulisboa.pt>
Reviewed-by: Lyude Paul <lyude@redhat.com>
Fixes: 9b340aeb26d5 ("nouveau/firmware: use dma non-coherent allocator")
Cc: stable@vger.kernel.org
Signed-off-by: Dave Airlie <airlied@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241114004603.3095485-1-airlied@gmail.com

diff --git a/drivers/gpu/drm/nouveau/nvkm/falcon/fw.c b/drivers/gpu/drm/nouveau/nvkm/falcon/fw.c
index a1c8545..cac6d64 100644 (file)
--- a/drivers/gpu/drm/nouveau/nvkm/falcon/fw.c
+++ b/drivers/gpu/drm/nouveau/nvkm/falcon/fw.c
@@ -89,11 +89,6 @@ nvkm_falcon_fw_boot(struct nvkm_falcon_fw *fw, struct nvkm_subdev *user,
                nvkm_falcon_fw_dtor_sigs(fw);
        }
 
-       /* after last write to the img, sync dma mappings */
-       dma_sync_single_for_device(fw->fw.device->dev,
-                                  fw->fw.phys,
-                                  sg_dma_len(&fw->fw.mem.sgl),
-                                  DMA_TO_DEVICE);
 
        FLCNFW_DBG(fw, "resetting");
        fw->func->reset(fw);
@@ -105,6 +100,12 @@ nvkm_falcon_fw_boot(struct nvkm_falcon_fw *fw, struct nvkm_subdev *user,
                        goto done;
        }
 
+       /* after last write to the img, sync dma mappings */
+       dma_sync_single_for_device(fw->fw.device->dev,
+                                  fw->fw.phys,
+                                  sg_dma_len(&fw->fw.mem.sgl),
+                                  DMA_TO_DEVICE);
+
        ret = fw->func->load(fw);
        if (ret)
                goto done;