On Thu, Sep 14, 2023 at 10:26:59PM +0300, Imre Deak wrote:
> After the previous patch the BW limits on the whole MST topology will be
> checked after computing the state for all the streams in the topology.
> Accordingly remove the check during the stream's encoder compute config
> step, to prevent failing an atomic commit due to a BW limit, if this can
> be resolved only by reducing the BW of other streams on the same MST
> link.
Reviewed-by: Stanislav Lisovskiy <stanislav.lisovs...@intel.com>
>
> Signed-off-by: Imre Deak <imre.d...@intel.com>
> ---
> drivers/gpu/drm/i915/display/intel_dp_mst.c | 11 ++---------
> 1 file changed, 2 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c
> b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> index bcfd5f19d994f..64867289174d9 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_mst.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> @@ -121,15 +121,8 @@ static int intel_dp_mst_find_vcpi_slots_for_bpp(struct
> intel_encoder *encoder,
> if (slots == -EDEADLK)
> return slots;
>
> - if (slots >= 0) {
> - ret = drm_dp_mst_atomic_check(state);
> - /*
> - * If we got slots >= 0 and we can fit those based on
> check
> - * then we can exit the loop. Otherwise keep trying.
> - */
> - if (!ret)
> - break;
> - }
> + if (slots >= 0)
> + break;
> }
>
> /* We failed to find a proper bpp/timeslots, return error */
> --
> 2.37.2
>
