On Mon, May 08, 2017 at 05:18:54PM +0530, Mahesh Kumar wrote: > Don't use fixed_16_16 structure members directly, instead use wrapper to > perform fixed_16_16 division operation. > > Signed-off-by: Mahesh Kumar <[email protected]>
Reviewed-by: Matt Roper <[email protected]> > --- > drivers/gpu/drm/i915/intel_pm.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c > index 91f09dfdb618..66b542ba47ad 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -3867,8 +3867,8 @@ static int skl_compute_plane_wm(const struct > drm_i915_private *dev_priv, > } > > res_blocks = fixed_16_16_to_u32_round_up(selected_result) + 1; > - res_lines = DIV_ROUND_UP(selected_result.val, > - plane_blocks_per_line.val); > + res_lines = div_round_up_fixed_16_16(selected_result, > + plane_blocks_per_line); > > if (level >= 1 && level <= 7) { > if (y_tiled) { > -- > 2.11.0 > -- Matt Roper Graphics Software Engineer IoTG Platform Enabling & Development Intel Corporation (916) 356-2795 _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
