Op 18-11-2019 om 18:39 schreef Ville Syrjälä:
> On Thu, Nov 14, 2019 at 05:05:13PM +0100, Maarten Lankhorst wrote:
>> The members in hw.mode can be used from adjusted_mode as well,
>> use that when available.
>>
>> Some places that use hw.mode can be converted to use adjusted_mode
>> as well.
>>
>> Signed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
>> ---
>>  drivers/gpu/drm/i915/display/intel_display.c  | 49 ++++++++++---------
>>  .../drm/i915/display/intel_display_types.h    |  2 +-
>>  drivers/gpu/drm/i915/display/intel_dvo.c      |  2 +-
>>  drivers/gpu/drm/i915/display/intel_sdvo.c     | 16 +++---
>>  4 files changed, 34 insertions(+), 35 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/i915/display/intel_display.c 
>> b/drivers/gpu/drm/i915/display/intel_display.c
>> index adf50c4b38ad..18acecc3642d 100644
>> --- a/drivers/gpu/drm/i915/display/intel_display.c
>> +++ b/drivers/gpu/drm/i915/display/intel_display.c
>> @@ -8440,9 +8440,6 @@ static void intel_get_pipe_src_size(struct intel_crtc 
>> *crtc,
>>      tmp = I915_READ(PIPESRC(crtc->pipe));
>>      pipe_config->pipe_src_h = (tmp & 0xffff) + 1;
>>      pipe_config->pipe_src_w = ((tmp >> 16) & 0xffff) + 1;
>> -
>> -    pipe_config->hw.mode.vdisplay = pipe_config->pipe_src_h;
>> -    pipe_config->hw.mode.hdisplay = pipe_config->pipe_src_w;
>>  }
>>  
>>  void intel_mode_from_pipe_config(struct drm_display_mode *mode,
>> @@ -12098,8 +12095,8 @@ static int icl_add_sync_mode_crtcs(struct 
>> intel_crtc_state *crtc_state)
>>                      continue;
>>              if (!connector->has_tile)
>>                      continue;
>> -            if (crtc_state->hw.mode.hdisplay != connector->tile_h_size ||
>> -                crtc_state->hw.mode.vdisplay != connector->tile_v_size)
>> +            if (crtc_state->hw.adjusted_mode.crtc_hdisplay != 
>> connector->tile_h_size ||
>> +                crtc_state->hw.adjusted_mode.crtc_vdisplay != 
>> connector->tile_v_size)
>>                      return 0;
>>              if (connector->tile_h_loc == connector->num_h_tile - 1 &&
>>                  connector->tile_v_loc == connector->num_v_tile - 1)
>> @@ -12506,7 +12503,7 @@ static void intel_dump_pipe_config(const struct 
>> intel_crtc_state *pipe_config,
>>              intel_dump_infoframe(dev_priv, &pipe_config->infoframes.hdmi);
>>  
>>      DRM_DEBUG_KMS("requested mode:\n");
>> -    drm_mode_debug_printmodeline(&pipe_config->hw.mode);
>> +    drm_mode_debug_printmodeline(&pipe_config->uapi.mode);
>>      DRM_DEBUG_KMS("adjusted mode:\n");
>>      drm_mode_debug_printmodeline(&pipe_config->hw.adjusted_mode);
>>      intel_dump_crtc_timings(&pipe_config->hw.adjusted_mode);
>> @@ -12640,16 +12637,16 @@ intel_crtc_copy_uapi_to_hw_state(struct 
>> intel_crtc_state *crtc_state)
>>  {
>>      crtc_state->hw.enable = crtc_state->uapi.enable;
>>      crtc_state->hw.active = crtc_state->uapi.active;
>> -    crtc_state->hw.mode = crtc_state->uapi.mode;
>>      crtc_state->hw.adjusted_mode = crtc_state->uapi.adjusted_mode;
>>      intel_crtc_copy_uapi_to_hw_state_nomodeset(crtc_state);
>>  }
>>  
>> -static void intel_crtc_copy_hw_to_uapi_state(struct intel_crtc_state 
>> *crtc_state)
>> +static void intel_crtc_copy_hw_to_uapi_state(struct intel_crtc_state 
>> *crtc_state,
>> +                                         struct drm_display_mode *user_mode)
>>  {
>>      crtc_state->uapi.enable = crtc_state->hw.enable;
>>      crtc_state->uapi.active = crtc_state->hw.active;
>> -    WARN_ON(drm_atomic_set_mode_for_crtc(&crtc_state->uapi, 
>> &crtc_state->hw.mode) < 0);
>> +    WARN_ON(drm_atomic_set_mode_for_crtc(&crtc_state->uapi, user_mode) < 0);
>>  
>>      crtc_state->uapi.adjusted_mode = crtc_state->hw.adjusted_mode;
>>  
>> @@ -12702,6 +12699,10 @@ intel_crtc_prepare_cleared_state(struct 
>> intel_crtc_state *crtc_state)
>>      memcpy(crtc_state, saved_state, sizeof(*crtc_state));
>>      kfree(saved_state);
>>  
>> +    /* Clear I915_MODE_FLAG_INHERITED */
>> +    crtc_state->uapi.mode.private_flags = 0;
>> +    crtc_state->uapi.adjusted_mode.private_flags = 0;
>> +
>>      intel_crtc_copy_uapi_to_hw_state(crtc_state);
>>  
>>      return 0;
>> @@ -12750,7 +12751,7 @@ intel_modeset_pipe_config(struct intel_crtc_state 
>> *pipe_config)
>>       * computation to clearly distinguish it from the adjusted mode, which
>>       * can be changed by the connectors in the below retry loop.
>>       */
>> -    drm_mode_get_hv_timing(&pipe_config->hw.mode,
>> +    drm_mode_get_hv_timing(&pipe_config->hw.pipe_mode,
>>                             &pipe_config->pipe_src_w,
>>                             &pipe_config->pipe_src_h);
>>  
>> @@ -12852,6 +12853,8 @@ intel_modeset_pipe_config(struct intel_crtc_state 
>> *pipe_config)
>>       */
>>      pipe_config->uapi.adjusted_mode = pipe_config->hw.adjusted_mode;
>>  
>> +    /* without bigjoiner, pipe_mode == adjusted_mode */
>> +    pipe_config->hw.pipe_mode = pipe_config->hw.adjusted_mode;
>>      return 0;
>>  }
>>  
>> @@ -12995,8 +12998,8 @@ intel_pipe_config_compare(const struct 
>> intel_crtc_state *current_config,
>>      bool ret = true;
>>      u32 bp_gamma = 0;
>>      bool fixup_inherited = fastset &&
>> -            (current_config->hw.mode.private_flags & 
>> I915_MODE_FLAG_INHERITED) &&
>> -            !(pipe_config->hw.mode.private_flags & 
>> I915_MODE_FLAG_INHERITED);
>> +            (current_config->hw.adjusted_mode.private_flags & 
>> I915_MODE_FLAG_INHERITED) &&
>> +            !(pipe_config->hw.adjusted_mode.private_flags & 
>> I915_MODE_FLAG_INHERITED);
>>  
>>      if (fixup_inherited && !fastboot_enabled(dev_priv)) {
>>              DRM_DEBUG_KMS("initial modeset and fastboot not set\n");
>> @@ -14106,8 +14109,8 @@ static int intel_atomic_check(struct drm_device *dev,
>>      /* Catch I915_MODE_FLAG_INHERITED */
>>      for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state,
>>                                          new_crtc_state, i) {
>> -            if (new_crtc_state->hw.mode.private_flags !=
>> -                old_crtc_state->hw.mode.private_flags)
>> +            if (new_crtc_state->hw.adjusted_mode.private_flags !=
>> +                old_crtc_state->hw.adjusted_mode.private_flags)
>>                      new_crtc_state->uapi.mode_changed = true;
>>      }
>>  
>> @@ -14361,7 +14364,7 @@ static void intel_update_crtc(struct intel_crtc 
>> *crtc,
>>       * of enabling them on the CRTC's first fastset.
>>       */
>>      if (new_crtc_state->update_pipe && !modeset &&
>> -        old_crtc_state->hw.mode.private_flags & I915_MODE_FLAG_INHERITED)
>> +        old_crtc_state->hw.adjusted_mode.private_flags & 
>> I915_MODE_FLAG_INHERITED)
>>              intel_crtc_arm_fifo_underrun(crtc, new_crtc_state);
>>  }
>>  
>> @@ -14558,7 +14561,7 @@ static void intel_post_crtc_enable_updates(struct 
>> intel_crtc *crtc,
>>       * of enabling them on the CRTC's first fastset.
>>       */
>>      if (new_crtc_state->update_pipe && !modeset &&
>> -        old_crtc_state->hw.mode.private_flags & I915_MODE_FLAG_INHERITED)
>> +        old_crtc_state->hw.adjusted_mode.private_flags & 
>> I915_MODE_FLAG_INHERITED)
>>              intel_crtc_arm_fifo_underrun(crtc, new_crtc_state);
>>  }
>>  
>> @@ -17524,15 +17527,11 @@ static void intel_modeset_readout_hw_state(struct 
>> drm_device *dev)
>>              int min_cdclk = 0;
>>  
>>              if (crtc_state->hw.active) {
>> -                    struct drm_display_mode *mode = &crtc_state->hw.mode;
>> +                    struct drm_display_mode mode;
>>  
>>                      
>> intel_mode_from_pipe_config(&crtc_state->hw.adjusted_mode,
>>                                                  crtc_state);
>>  
>> -                    *mode = crtc_state->hw.adjusted_mode;
>> -                    mode->hdisplay = crtc_state->pipe_src_w;
>> -                    mode->vdisplay = crtc_state->pipe_src_h;
>> -
>>                      /*
>>                       * The initial mode needs to be set in order to keep
>>                       * the atomic core happy. It wants a valid mode if the
>> @@ -17542,13 +17541,17 @@ static void intel_modeset_readout_hw_state(struct 
>> drm_device *dev)
>>                       * set a flag to indicate that a full recalculation is
>>                       * needed on the next commit.
>>                       */
>> -                    mode->private_flags = I915_MODE_FLAG_INHERITED;
>> +                    crtc_state->hw.adjusted_mode.private_flags = 
>> I915_MODE_FLAG_INHERITED;
>> +
>> +                    mode = crtc_state->hw.adjusted_mode;
>> +                    mode.hdisplay = crtc_state->pipe_src_w;
>> +                    mode.vdisplay = crtc_state->pipe_src_h;
>>  
>>                      intel_crtc_compute_pixel_rate(crtc_state);
>>  
>>                      intel_crtc_update_active_timings(crtc_state);
>>  
>> -                    intel_crtc_copy_hw_to_uapi_state(crtc_state);
>> +                    intel_crtc_copy_hw_to_uapi_state(crtc_state, &mode);
>>              }
>>  
>>              for_each_intel_plane_on_crtc(&dev_priv->drm, crtc, plane) {
>> diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h 
>> b/drivers/gpu/drm/i915/display/intel_display_types.h
>> index 83ea04149b77..0f20c7190eed 100644
>> --- a/drivers/gpu/drm/i915/display/intel_display_types.h
>> +++ b/drivers/gpu/drm/i915/display/intel_display_types.h
>> @@ -802,7 +802,7 @@ struct intel_crtc_state {
>>      struct {
>>              bool active, enable;
>>              struct drm_property_blob *degamma_lut, *gamma_lut, *ctm;
>> -            struct drm_display_mode mode, adjusted_mode;
>> +            struct drm_display_mode adjusted_mode;
>>      } hw;
>>  
>>      /**
>> diff --git a/drivers/gpu/drm/i915/display/intel_dvo.c 
>> b/drivers/gpu/drm/i915/display/intel_dvo.c
>> index a74dc5b915d1..ad8e9959d6e1 100644
>> --- a/drivers/gpu/drm/i915/display/intel_dvo.c
>> +++ b/drivers/gpu/drm/i915/display/intel_dvo.c
>> @@ -207,7 +207,7 @@ static void intel_enable_dvo(struct intel_encoder 
>> *encoder,
>>      u32 temp = I915_READ(dvo_reg);
>>  
>>      intel_dvo->dev.dev_ops->mode_set(&intel_dvo->dev,
>> -                                     &pipe_config->hw.mode,
>> +                                     &pipe_config->hw.adjusted_mode,
>>                                       &pipe_config->hw.adjusted_mode);
> This looks very wrong.
Hmm yeah, should probably be pipe_config->uapi.mode for the first parameter, or 
panel fitter for IVCH stops working.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to