The memory target check is a hold-over from a refactor.  It's harmless
but distracting, so just remove it.

Signed-off-by: Timur Tabi <tt...@nvidia.com>
---
 drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c 
b/drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c
index b7da3ab44c27..f0acfaa153d1 100644
--- a/drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c
+++ b/drivers/gpu/drm/nouveau/nvkm/falcon/gm200.c
@@ -249,9 +249,11 @@ int
 gm200_flcn_fw_load(struct nvkm_falcon_fw *fw)
 {
        struct nvkm_falcon *falcon = fw->falcon;
-       int target, ret;
+       int ret;
 
        if (fw->inst) {
+               int target;
+
                nvkm_falcon_mask(falcon, 0x048, 0x00000001, 0x00000001);
 
                switch (nvkm_memory_target(fw->inst)) {
@@ -285,15 +287,6 @@ gm200_flcn_fw_load(struct nvkm_falcon_fw *fw)
        }
 
        if (fw->boot) {
-               switch (nvkm_memory_target(&fw->fw.mem.memory)) {
-               case NVKM_MEM_TARGET_VRAM: target = 4; break;
-               case NVKM_MEM_TARGET_HOST: target = 5; break;
-               case NVKM_MEM_TARGET_NCOH: target = 6; break;
-               default:
-                       WARN_ON(1);
-                       return -EINVAL;
-               }
-
                ret = nvkm_falcon_pio_wr(falcon, fw->boot, 0, 0,
                                         IMEM, falcon->code.limit - 
fw->boot_size, fw->boot_size,
                                         fw->boot_addr >> 8, false);

base-commit: 155a3c003e555a7300d156a5252c004c392ec6b0
-- 
2.43.0

Reply via email to