[AMD Official Use Only] Thanks for the catch. Reviewed-by: Jonathan Kim <jonathan.kim@xxxxxxx> > -----Original Message----- > From: Das, Nirmoy <Nirmoy.Das@xxxxxxx> > Sent: Tuesday, June 15, 2021 6:35 AM > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Kim, Jonathan <Jonathan.Kim@xxxxxxx>; Kuehling, Felix > <Felix.Kuehling@xxxxxxx> > Subject: Re: [PATCH 1/1] drm/amdkfd: remove unused variable > > > On 6/15/2021 12:33 PM, Nirmoy Das wrote: > > Remove it. > > > > CC: jonathan.kim@xxxxxxx > > CC: felix.kuehling@xxxxxxx > > Fixes: d7b132507384c("drm/amdkfd: fix circular locking on get_wave_state") > > Signed-off-by: Nirmoy Das <nirmoy.das@xxxxxxx> > > --- > > drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c > b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c > > index e6366b408420..539212039876 100644 > > --- a/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c > > +++ b/drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c > > @@ -997,7 +997,7 @@ static int start_nocpsch(struct > device_queue_manager *dqm) > > { > > pr_info("SW scheduler is used"); > > init_interrupts(dqm); > > - > > + > Please ignore this extra space. I will push without it once I get a r-b. > > if (dqm->dev->device_info->asic_family == CHIP_HAWAII) > > return pm_init(&dqm->packets, dqm); > > dqm->sched_running = true; > > @@ -1674,7 +1674,6 @@ static int get_wave_state(struct > device_queue_manager *dqm, > > u32 *save_area_used_size) > > { > > struct mqd_manager *mqd_mgr; > > - int r; > > > > dqm_lock(dqm); > > _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx