diff options
| author | Nicholas Kazlauskas | 2019-03-29 16:40:43 +0100 |
|---|---|---|
| committer | Alex Deucher | 2019-04-15 07:21:45 +0200 |
| commit | 6e8dc7b0a7a4d5532c2e0706795a95b7beb9592e (patch) | |
| tree | ad07f3e298891a30603ba9b37fb437e3c5e3d303 /drivers/gpu/drm/amd/display/amdgpu_dm | |
| parent | drm/amd/display: Use surface directly when checking update type (diff) | |
| download | kernel-qcow2-linux-6e8dc7b0a7a4d5532c2e0706795a95b7beb9592e.tar.gz kernel-qcow2-linux-6e8dc7b0a7a4d5532c2e0706795a95b7beb9592e.tar.xz kernel-qcow2-linux-6e8dc7b0a7a4d5532c2e0706795a95b7beb9592e.zip | |
drm/amd/display: Don't warn when DC update type > DM guess
[Why]
DM thinks that the update type should be full whenever a stream or
plane is added or removed (including recreations).
This won't match in the case where DC thinks what looks like a fast
update to DM is actually a medium or full - like scaling changes that
affect bandwidth and clocks.
[How]
Drop this warning. DC knows better than the DM does for determining
cases like this.
The other warning can be kept for now since it would warn on a pretty
serious DC or DM bug.
Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
Reviewed-by: David Francis <David.Francis@amd.com>
Acked-by: Bhawanpreet Lakha <Bhawanpreet Lakha@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/display/amdgpu_dm')
| -rw-r--r-- | drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 |
1 files changed, 0 insertions, 3 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 5c0b3bbfd18f..409979f8bf0b 100644 --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c @@ -6600,9 +6600,6 @@ static int amdgpu_dm_atomic_check(struct drm_device *dev, */ if (lock_and_validation_needed && overall_update_type <= UPDATE_TYPE_FAST) WARN(1, "Global lock should be Set, overall_update_type should be UPDATE_TYPE_MED or UPDATE_TYPE_FULL"); - else if (!lock_and_validation_needed && overall_update_type > UPDATE_TYPE_FAST) - WARN(1, "Global lock should NOT be set, overall_update_type should be UPDATE_TYPE_FAST"); - if (overall_update_type > UPDATE_TYPE_FAST) { ret = dm_atomic_get_state(state, &dm_state); |