On Fri, 2026-05-22 at 23:03 +0300, Ville Syrjala wrote:
> From: Ville Syrjälä <[email protected]>
> 
> The DRAM bandwidth value should be rounded down, not up.
> 
> Signed-off-by: Ville Syrjälä <[email protected]>
> ---
>  drivers/gpu/drm/i915/display/intel_bw.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Bspec: 64631

Reviewed-by: Vinod Govindapillai <[email protected]>

> 
> diff --git a/drivers/gpu/drm/i915/display/intel_bw.c
> b/drivers/gpu/drm/i915/display/intel_bw.c
> index 442e6b657ca5..bbaafcc1b6be 100644
> --- a/drivers/gpu/drm/i915/display/intel_bw.c
> +++ b/drivers/gpu/drm/i915/display/intel_bw.c
> @@ -555,7 +555,7 @@ static int icl_get_bw_info(struct intel_display
> *display,
>                        */
>                       ct = max_t(int, sp->t_rc, sp->t_rp + sp-
> >t_rcd +
>                                  (clpchgroup - 1) * qi.t_bl + sp-
> >t_rdpre);
> -                     bw = DIV_ROUND_UP(sp->dclk * clpchgroup * 32
> * num_channels, ct);
> +                     bw = sp->dclk * clpchgroup * 32 *
> num_channels / ct;
>  
>                       bi->deratedbw[j] = min(maxdebw,
>                                              bw * (100 -
> soc_bw_params->derating) / 100);
> @@ -658,7 +658,7 @@ static int tgl_get_bw_info(struct intel_display
> *display,
>                        */
>                       ct = max_t(int, sp->t_rc, sp->t_rp + sp-
> >t_rcd +
>                                  (clpchgroup - 1) * qi.t_bl + sp-
> >t_rdpre);
> -                     bw = DIV_ROUND_UP(sp->dclk * clpchgroup * 32
> * num_channels, ct);
> +                     bw = sp->dclk * clpchgroup * 32 *
> num_channels / ct;
>  
>                       bi->deratedbw[j] = min(maxdebw,
>                                              bw * (100 -
> soc_bw_params->derating) / 100);

Reply via email to