On Thu, Mar 26, 2020 at 08:10:03PM +0200, Stanislav Lisovskiy wrote: > That is a preparation patch before next one where we > introduce old_bw_state and a bunch of other changes > as well. > In a review comment it was suggested to split out > at least that renaming into a separate patch, what > is done here. > > Signed-off-by: Stanislav Lisovskiy <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_bw.c | 24 ++++++++++++------------ > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_bw.c > b/drivers/gpu/drm/i915/display/intel_bw.c > index a8b2038db4d2..d16771dd2b10 100644 > --- a/drivers/gpu/drm/i915/display/intel_bw.c > +++ b/drivers/gpu/drm/i915/display/intel_bw.c > @@ -418,7 +418,7 @@ int intel_bw_atomic_check(struct intel_atomic_state > *state) > { > struct drm_i915_private *dev_priv = to_i915(state->base.dev); > struct intel_crtc_state *new_crtc_state, *old_crtc_state; > - struct intel_bw_state *bw_state = NULL; > + struct intel_bw_state *new_bw_state = NULL; > unsigned int data_rate, max_data_rate; > unsigned int num_active_planes; > struct intel_crtc *crtc; > @@ -447,29 +447,29 @@ int intel_bw_atomic_check(struct intel_atomic_state > *state) > old_active_planes == new_active_planes) > continue; > > - bw_state = intel_atomic_get_bw_state(state); > - if (IS_ERR(bw_state)) > - return PTR_ERR(bw_state); > + new_bw_state = intel_atomic_get_bw_state(state); ^
Pls remove the spurious space while at it. Reviewed-by: Ville Syrjälä <[email protected]> > + if (IS_ERR(new_bw_state)) > + return PTR_ERR(new_bw_state); > > - bw_state->data_rate[crtc->pipe] = new_data_rate; > - bw_state->num_active_planes[crtc->pipe] = new_active_planes; > + new_bw_state->data_rate[crtc->pipe] = new_data_rate; > + new_bw_state->num_active_planes[crtc->pipe] = new_active_planes; > > drm_dbg_kms(&dev_priv->drm, > "pipe %c data rate %u num active planes %u\n", > pipe_name(crtc->pipe), > - bw_state->data_rate[crtc->pipe], > - bw_state->num_active_planes[crtc->pipe]); > + new_bw_state->data_rate[crtc->pipe], > + new_bw_state->num_active_planes[crtc->pipe]); > } > > - if (!bw_state) > + if (!new_bw_state) > return 0; > > - ret = intel_atomic_lock_global_state(&bw_state->base); > + ret = intel_atomic_lock_global_state(&new_bw_state->base); > if (ret) > return ret; > > - data_rate = intel_bw_data_rate(dev_priv, bw_state); > - num_active_planes = intel_bw_num_active_planes(dev_priv, bw_state); > + data_rate = intel_bw_data_rate(dev_priv, new_bw_state); > + num_active_planes = intel_bw_num_active_planes(dev_priv, new_bw_state); > > max_data_rate = intel_max_data_rate(dev_priv, num_active_planes); > > -- > 2.24.1.485.gad05a3d8e5 -- Ville Syrjälä Intel _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
