* [PATCH] drm/amd/display: Remove unused variable 'abo'
@ 2022-06-18 1:56 Souptick Joarder
2022-06-18 5:38 ` Simon Ser
0 siblings, 1 reply; 3+ messages in thread
From: Souptick Joarder @ 2022-06-18 1:56 UTC (permalink / raw)
To: harry.wentland, sunpeng.li, Rodrigo.Siqueira, alexander.deucher,
christian.koenig, Xinhui.Pan, airlied, daniel,
nicholas.kazlauskas, Roman.Li, shenshih, Wayne.Lin, contact
Cc: amd-gfx, dri-devel, linux-kernel, Souptick Joarder (HPE),
Kernel test robot
From: "Souptick Joarder (HPE)" <jrdr.linux@gmail.com>
Kernel test robot throws below warning ->
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm.c:9143:27:
warning: variable 'abo' set but not used [-Wunused-but-set-variable]
Remove the unused variable.
Reported-by: Kernel test robot <lkp@intel.com>
Signed-off-by: Souptick Joarder (HPE) <jrdr.linux@gmail.com>
---
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index c9004f7e700d..0103b34aabdd 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -9136,7 +9136,6 @@ static void amdgpu_dm_commit_planes(struct drm_atomic_state *state,
to_dm_crtc_state(drm_atomic_get_old_crtc_state(state, pcrtc));
int planes_count = 0, vpos, hpos;
unsigned long flags;
- struct amdgpu_bo *abo;
uint32_t target_vblank, last_flip_vblank;
bool vrr_active = amdgpu_dm_vrr_active(acrtc_state);
bool pflip_present = false;
@@ -9208,7 +9207,6 @@ static void amdgpu_dm_commit_planes(struct drm_atomic_state *state,
continue;
}
- abo = gem_to_amdgpu_bo(fb->obj[0]);
fill_dc_plane_info_and_addr(
dm->adev, new_plane_state,
afb->tiling_flags,
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/amd/display: Remove unused variable 'abo'
2022-06-18 1:56 [PATCH] drm/amd/display: Remove unused variable 'abo' Souptick Joarder
@ 2022-06-18 5:38 ` Simon Ser
2022-06-20 22:24 ` Alex Deucher
0 siblings, 1 reply; 3+ messages in thread
From: Simon Ser @ 2022-06-18 5:38 UTC (permalink / raw)
To: Souptick Joarder
Cc: harry.wentland, sunpeng.li, Rodrigo.Siqueira, alexander.deucher,
christian.koenig, Xinhui.Pan, airlied, daniel,
nicholas.kazlauskas, Roman.Li, shenshih, Wayne.Lin, amd-gfx,
dri-devel, linux-kernel, Kernel test robot
Reviewed-by: Simon Ser <contact@emersion.fr>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/amd/display: Remove unused variable 'abo'
2022-06-18 5:38 ` Simon Ser
@ 2022-06-20 22:24 ` Alex Deucher
0 siblings, 0 replies; 3+ messages in thread
From: Alex Deucher @ 2022-06-20 22:24 UTC (permalink / raw)
To: Simon Ser
Cc: Souptick Joarder, Kernel test robot, Leo (Sunpeng) Li, Jude Shih,
xinhui pan, Siqueira, Rodrigo, Roman Li, amd-gfx list,
Kazlauskas, Nicholas, Dave Airlie, Maling list - DRI developers,
Wayne Lin, Deucher, Alexander, Christian Koenig, LKML
I sent out the same patch last week. I just pushed it to drm-misc-next.
Thanks!
Alex
On Sat, Jun 18, 2022 at 1:38 AM Simon Ser <contact@emersion.fr> wrote:
>
> Reviewed-by: Simon Ser <contact@emersion.fr>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-06-20 22:25 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-06-18 1:56 [PATCH] drm/amd/display: Remove unused variable 'abo' Souptick Joarder
2022-06-18 5:38 ` Simon Ser
2022-06-20 22:24 ` Alex Deucher
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox