Op 17-11-16 om 14:06 schreef Paulo Zanoni:
> Em Qui, 2016-11-17 às 10:00 +0000, Tvrtko Ursulin escreveu:
>> From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
>>
>> It looks to me skl_update_scaler will already log interesting
>> debug messages when the state transitions or there is an error.
>>
>> In this case it feels we can remove the two unconditional
>> debug messages which happen immediately before calling
>> skl_update_scaler. This way we get rid of the sole debug
>> message when switching virtual terminals for example.
> I also got annoyed by this at some point.
>
> Reviewed-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx>
>
>> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx>
>> ---
>> drivers/gpu/drm/i915/intel_display.c | 10 ----------
>> 1 file changed, 10 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/intel_display.c
>> b/drivers/gpu/drm/i915/intel_display.c
>> index 2ebb8b833395..cd5256c0b2d6 100644
>> --- a/drivers/gpu/drm/i915/intel_display.c
>> +++ b/drivers/gpu/drm/i915/intel_display.c
>> @@ -4727,13 +4727,8 @@ skl_update_scaler(struct intel_crtc_state
>> *crtc_state, bool force_detach,
>> */
>> int skl_update_scaler_crtc(struct intel_crtc_state *state)
>> {
>> - struct intel_crtc *intel_crtc = to_intel_crtc(state-
>>> base.crtc);
>> const struct drm_display_mode *adjusted_mode = &state-
>>> base.adjusted_mode;
>>
>> - DRM_DEBUG_KMS("Updating scaler for [CRTC:%d:%s] scaler_user
>> index %u.%u\n",
>> - intel_crtc->base.base.id, intel_crtc-
>>> base.name,
>> - intel_crtc->pipe, SKL_CRTC_INDEX);
>> -
>> return skl_update_scaler(state, !state->base.active,
>> SKL_CRTC_INDEX,
>> &state->scaler_state.scaler_id, DRM_ROTATE_0,
>> state->pipe_src_w, state->pipe_src_h,
>> @@ -4754,7 +4749,6 @@ static int skl_update_scaler_plane(struct
>> intel_crtc_state *crtc_state,
>> struct intel_plane_state
>> *plane_state)
>> {
>>
>> - struct intel_crtc *intel_crtc = to_intel_crtc(crtc_state-
>>> base.crtc);
>> struct intel_plane *intel_plane =
>> to_intel_plane(plane_state->base.plane);
>> struct drm_framebuffer *fb = plane_state->base.fb;
>> @@ -4762,10 +4756,6 @@ static int skl_update_scaler_plane(struct
>> intel_crtc_state *crtc_state,
>>
>> bool force_detach = !fb || !plane_state->base.visible;
>>
>> - DRM_DEBUG_KMS("Updating scaler for [PLANE:%d:%s] scaler_user
>> index %u.%u\n",
>> - intel_plane->base.base.id, intel_plane-
>>> base.name,
>> - intel_crtc->pipe,
>> drm_plane_index(&intel_plane->base));
>> -
>> ret = skl_update_scaler(crtc_state, force_detach,
>> drm_plane_index(&intel_plane->base),
>> &plane_state->scaler_id,
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
