From: Mario Limonciello <mario.limonciello@xxxxxxx> I was looking at amdgpu_dm for some issues and just noticed some cases that could use cleanups for scoped cleanups and error messaging. Mario Limonciello (13): drm/amd/display: Change amdgpu_dm_irq_suspend() to void drm/amd/display: Drop `ret` variable from dm_suspend() drm/amd/display: Catch failures for amdgpu_dm_commit_zero_streams() drm/amd/display: Use a _free() macro for amdgpu_dm_commit_zero_streams() drm/amd/display: Use drm_err() instead of DRM_ERROR in dm_resume() drm/amd/display: Use scoped guard for dm_resume() drm/amd/display: Change amdgpu_dm_irq_resume_*() to use drm_dbg() drm/amd/display: Change amdgpu_dm_irq_resume_*() to void drm/amd/display: Use _free(kfree) for dm_gpureset_commit_state() drm/amd/display: Use scoped guard for amdgpu_dm_update_connector_after_detect() drm/amd/display: Use _free() macro fro releasing sink in amdgpu_dm_update_connector_after_detect() drm/amd/display: Use scoped guards for handle_hpd_irq_helper() drm/amd/display: Use drm_err() for handle_hpd_irq_helper() .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 89 ++++++++----------- .../drm/amd/display/amdgpu_dm/amdgpu_dm_irq.c | 14 ++- .../drm/amd/display/amdgpu_dm/amdgpu_dm_irq.h | 6 +- 3 files changed, 44 insertions(+), 65 deletions(-) -- 2.43.0