[PATCH] drm/nouveau: remove unused memory target test

Zhi Wang zhiw at nvidia.com
Fri Jul 18 09:12:53 UTC 2025


On Thu, 17 Jul 2025 15:06:24 +0000
Timur Tabi <ttabi at nvidia.com> wrote:

Oh, you are right. LGTM then.

Reviewed-by: Zhi Wang <zhiw at nvidia.com>

> On Thu, 2025-07-17 at 11:09 +0300, Zhi Wang wrote:
> > On Tue, 15 Jul 2025 16:05:59 -0500
> > Timur Tabi <ttabi at nvidia.com> wrote:
> > 
> > There is also a similar code block in this function:
> > 
> >         if (fw->inst) {
> >                 nvkm_falcon_mask(falcon, 0x048, 0x00000001,
> > 0x00000001);
> > 
> >                 switch (nvkm_memory_target(fw->inst)) {
> >                 case NVKM_MEM_TARGET_VRAM: target = 0; break;
> >                 case NVKM_MEM_TARGET_HOST: target = 2; break;
> >                 case NVKM_MEM_TARGET_NCOH: target = 3; break;
> >                 default:
> >                         WARN_ON(1);
> >                         return -EINVAL;
> >                 }
> > 
> > Should this be removed with the same purpose here?
> 
> No, 'target' is actually used in this block.  That's why I moved that
> variable to inside the block.
> 
> 	falcon->func->bind_inst(falcon, target,
> nvkm_memory_addr(fw->inst));
> 



More information about the Nouveau mailing list