On Thu, Jun 04, 2026 at 02:59:09PM +0000, Govindapillai, Vinod wrote:
> On Thu, 2026-05-28 at 13:34 +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä <[email protected]>
> > 
> > Replace a hand rolled copy of icl_qgv_bw() with the real thing.
> > 
> > Signed-off-by: Ville Syrjälä <[email protected]>
> > ---
> >  drivers/gpu/drm/i915/display/intel_bw.c | 10 ++--------
> >  1 file changed, 2 insertions(+), 8 deletions(-)
> > 
> > diff --git a/drivers/gpu/drm/i915/display/intel_bw.c
> > b/drivers/gpu/drm/i915/display/intel_bw.c
> > index f4121223a58b..423cae2ff208 100644
> > --- a/drivers/gpu/drm/i915/display/intel_bw.c
> > +++ b/drivers/gpu/drm/i915/display/intel_bw.c
> > @@ -1106,14 +1106,8 @@ static int mtl_find_qgv_points(struct
> > intel_display *display,
> >      * offered per plane group
> >      */
> >     for (i = 0; i < num_qgv_points; i++) {
> > -           unsigned int bw_index =
> > -                   tgl_max_bw_index(display, num_active_planes,
> > i);
> > -           unsigned int max_data_rate;
> > -
> > -           if (bw_index >= ARRAY_SIZE(display->bw.max))
> > -                   continue;
> > -
> > -           max_data_rate = display-
> > >bw.max[bw_index].deratedbw[i];
> > +           unsigned int max_data_rate =
> > +                   icl_qgv_bw(display, num_active_planes, i);
> >  
> >             if (max_data_rate < data_rate)
> >                     continue;
> 
> Wonder if this could be squashed to patch 3 in the series for
> completeness.

This has nothing to do with that other patch.

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

-- 
Ville Syrjälä
Intel

Reply via email to