> Subject: [PATCH 2/4] drm/i915/display: use struct intel_atomic_state *state
> variable naming
> 
> The struct intel_atomic_state * pointer variables and parameters are named
> "state" everywhere. Follow suit in intel_atomic_commit_fence_wait().
> 
> Cc: Ville Syrjälä <[email protected]>
> Cc: Maarten Lankhorst <[email protected]>
> Signed-off-by: Jani Nikula <[email protected]>

LGTM,
Reviewed-by: Suraj Kandpal <[email protected]>

> ---
>  drivers/gpu/drm/i915/display/intel_display.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c
> b/drivers/gpu/drm/i915/display/intel_display.c
> index 579b03034c46..1c2e50c6d98a 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -7288,14 +7288,14 @@ static void skl_commit_modeset_enables(struct
> intel_atomic_state *state)
>       drm_WARN_ON(display->drm, update_pipes);  }
> 
> -static void intel_atomic_commit_fence_wait(struct intel_atomic_state
> *intel_state)
> +static void intel_atomic_commit_fence_wait(struct intel_atomic_state
> +*state)
>  {
>       struct drm_plane *plane;
>       struct drm_plane_state *new_plane_state;
>       long ret;
>       int i;
> 
> -     for_each_new_plane_in_state(&intel_state->base, plane,
> new_plane_state, i) {
> +     for_each_new_plane_in_state(&state->base, plane, new_plane_state,
> i) {
>               if (new_plane_state->fence) {
>                       ret = dma_fence_wait(new_plane_state->fence, false);
>                       if (ret < 0)
> --
> 2.47.3

Reply via email to