mirror of
https://github.com/torvalds/linux.git
synced 2025-11-02 09:40:27 +02:00
drm/amd/display: Remove unnecessary amdgpu_irq_get/put
[WHY & HOW] commit7fb363c575("drm/amd/display: Let drm_crtc_vblank_on/off manage interrupts") lets drm_crtc_vblank_* to manage interrupts in amdgpu_dm_crtc_set_vblank, and amdgpu_irq_get/put do not need to be called here. Part of that patch got lost somehow, so fix it up. Fixes:7fb363c575("drm/amd/display: Let drm_crtc_vblank_on/off manage interrupts") Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Reviewed-by: Leo Li <sunpeng.li@amd.com> Signed-off-by: Alex Hung <alex.hung@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> (cherry picked from commit3782305ce5) Cc: stable@vger.kernel.org
This commit is contained in:
parent
273b3eb600
commit
5009628d85
1 changed files with 0 additions and 31 deletions
|
|
@ -8400,16 +8400,6 @@ static void manage_dm_interrupts(struct amdgpu_device *adev,
|
||||||
struct amdgpu_crtc *acrtc,
|
struct amdgpu_crtc *acrtc,
|
||||||
struct dm_crtc_state *acrtc_state)
|
struct dm_crtc_state *acrtc_state)
|
||||||
{
|
{
|
||||||
/*
|
|
||||||
* We have no guarantee that the frontend index maps to the same
|
|
||||||
* backend index - some even map to more than one.
|
|
||||||
*
|
|
||||||
* TODO: Use a different interrupt or check DC itself for the mapping.
|
|
||||||
*/
|
|
||||||
int irq_type =
|
|
||||||
amdgpu_display_crtc_idx_to_irq_type(
|
|
||||||
adev,
|
|
||||||
acrtc->crtc_id);
|
|
||||||
struct drm_vblank_crtc_config config = {0};
|
struct drm_vblank_crtc_config config = {0};
|
||||||
struct dc_crtc_timing *timing;
|
struct dc_crtc_timing *timing;
|
||||||
int offdelay;
|
int offdelay;
|
||||||
|
|
@ -8435,28 +8425,7 @@ static void manage_dm_interrupts(struct amdgpu_device *adev,
|
||||||
|
|
||||||
drm_crtc_vblank_on_config(&acrtc->base,
|
drm_crtc_vblank_on_config(&acrtc->base,
|
||||||
&config);
|
&config);
|
||||||
|
|
||||||
amdgpu_irq_get(
|
|
||||||
adev,
|
|
||||||
&adev->pageflip_irq,
|
|
||||||
irq_type);
|
|
||||||
#if defined(CONFIG_DRM_AMD_SECURE_DISPLAY)
|
|
||||||
amdgpu_irq_get(
|
|
||||||
adev,
|
|
||||||
&adev->vline0_irq,
|
|
||||||
irq_type);
|
|
||||||
#endif
|
|
||||||
} else {
|
} else {
|
||||||
#if defined(CONFIG_DRM_AMD_SECURE_DISPLAY)
|
|
||||||
amdgpu_irq_put(
|
|
||||||
adev,
|
|
||||||
&adev->vline0_irq,
|
|
||||||
irq_type);
|
|
||||||
#endif
|
|
||||||
amdgpu_irq_put(
|
|
||||||
adev,
|
|
||||||
&adev->pageflip_irq,
|
|
||||||
irq_type);
|
|
||||||
drm_crtc_vblank_off(&acrtc->base);
|
drm_crtc_vblank_off(&acrtc->base);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue