RE: [RESEND RFC 04/18] drm/display/dp_mst: Call them time slots, not VCPI slots
From: Lin, Wayne
Date: Wed Jun 15 2022 - 00:28:44 EST
[Public]
Thank you Lyude for addressing this!
VCPI is also a confusing naming to me at first glance since it stands for
Virtual Channel Payload Identification which is just an ID number ( we can
look up these payload IDs In DPCD 0x2C1 ~0x2FF).
I also look up left VCPI terms in rest of the code. Seems like we still can modify
some parts here? Like:
/**
* struct drm_dp_vcpi - Virtual Channel Payload Identifier
* @vcpi: Virtual channel ID.
* @pbn: Payload Bandwidth Number for this channel
* @aligned_pbn: PBN aligned with slot size
* @num_slots: number of slots for this PBN
*/
struct drm_dp_vcpi {
int vcpi;
int pbn;
int aligned_pbn;
int num_slots;
};
Would like to change the structure name to "struct drm_dp_mst_vcp {}" to represent
the virtual channel payload. Not specific to the ID.
Would like to know your thoughts : )
> -----Original Message-----
> From: Lyude Paul <lyude@xxxxxxxxxx>
> Sent: Wednesday, June 8, 2022 3:29 AM
> To: dri-devel@xxxxxxxxxxxxxxxxxxxxx; nouveau@xxxxxxxxxxxxxxxxxxxxx; amd-
> gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Lin, Wayne <Wayne.Lin@xxxxxxx>; Ville Syrjälä
> <ville.syrjala@xxxxxxxxxxxxxxx>; Zuo, Jerry <Jerry.Zuo@xxxxxxx>; Jani Nikula
> <jani.nikula@xxxxxxxxx>; Imre Deak <imre.deak@xxxxxxxxx>; Daniel Vetter
> <daniel.vetter@xxxxxxxx>; Sean Paul <sean@xxxxxxxxxx>; Wentland, Harry
> <Harry.Wentland@xxxxxxx>; Li, Sun peng (Leo) <Sunpeng.Li@xxxxxxx>;
> Siqueira, Rodrigo <Rodrigo.Siqueira@xxxxxxx>; Deucher, Alexander
> <Alexander.Deucher@xxxxxxx>; Koenig, Christian
> <Christian.Koenig@xxxxxxx>; Pan, Xinhui <Xinhui.Pan@xxxxxxx>; David
> Airlie <airlied@xxxxxxxx>; Daniel Vetter <daniel@xxxxxxxx>; Jani Nikula
> <jani.nikula@xxxxxxxxxxxxxxx>; Joonas Lahtinen
> <joonas.lahtinen@xxxxxxxxxxxxxxx>; Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>;
> Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx>; Ben Skeggs
> <bskeggs@xxxxxxxxxx>; Karol Herbst <kherbst@xxxxxxxxxx>; Kazlauskas,
> Nicholas <Nicholas.Kazlauskas@xxxxxxx>; Li, Roman
> <Roman.Li@xxxxxxx>; Shih, Jude <Jude.Shih@xxxxxxx>; Simon Ser
> <contact@xxxxxxxxxxx>; Wu, Hersen <hersenxs.wu@xxxxxxx>; Thomas
> Zimmermann <tzimmermann@xxxxxxx>; Lakha, Bhawanpreet
> <Bhawanpreet.Lakha@xxxxxxx>; José Roberto de Souza
> <jose.souza@xxxxxxxxx>; He Ying <heying24@xxxxxxxxxx>; Matt Roper
> <matthew.d.roper@xxxxxxxxx>; Sean Paul <seanpaul@xxxxxxxxxxxx>; Hans
> Verkuil <hverkuil-cisco@xxxxxxxxx>; Fernando Ramos <greenfoo@xxxxxx>;
> Javier Martinez Canillas <javierm@xxxxxxxxxx>; open list <linux-
> kernel@xxxxxxxxxxxxxxx>; open list:INTEL DRM DRIVERS <intel-
> gfx@xxxxxxxxxxxxxxxxxxxxx>
> Subject: [RESEND RFC 04/18] drm/display/dp_mst: Call them time slots, not
> VCPI slots
>
> VCPI is only sort of the correct term here, originally the majority of this code
> simply referred to timeslots vaguely as "slots" - and since I started working
> on it and adding atomic functionality, the name "VCPI slots" has been used to
> represent time slots.
>
> Now that we actually have consistent access to the DisplayPort spec thanks to
> VESA, I now know this isn't actually the proper term - as the specification
> refers to these as time slots.
>
> Since we're trying to make this code as easy to figure out as possible, let's
> take this opportunity to correct this nomenclature and call them by their
> proper name - timeslots. Likewise, we rename various functions
> appropriately, along with replacing references in the kernel documentation
> and various debugging messages.
>
> It's important to note that this patch series leaves the legacy MST code
> untouched for the most part, which is fine since we'll be removing it soon
> anyhow. There should be no functional changes in this series.
>
> Signed-off-by: Lyude Paul <lyude@xxxxxxxxxx>
> Cc: Wayne Lin <Wayne.Lin@xxxxxxx>
> Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> Cc: Fangzhi Zuo <Jerry.Zuo@xxxxxxx>
> Cc: Jani Nikula <jani.nikula@xxxxxxxxx>
> Cc: Imre Deak <imre.deak@xxxxxxxxx>
> Cc: Daniel Vetter <daniel.vetter@xxxxxxxx>
> Cc: Sean Paul <sean@xxxxxxxxxx>
> ---
> .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
> .../display/amdgpu_dm/amdgpu_dm_mst_types.c | 28 ++---
> drivers/gpu/drm/display/drm_dp_mst_topology.c | 106 +++++++++---------
> drivers/gpu/drm/i915/display/intel_dp_mst.c | 5 +-
> drivers/gpu/drm/nouveau/dispnv50/disp.c | 4 +-
> include/drm/display/drm_dp_mst_helper.h | 6 +-
> 6 files changed, 75 insertions(+), 76 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index ad4571190a90..f84a4ad736d8 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -7393,7 +7393,7 @@ static int dm_encoder_helper_atomic_check(struct
> drm_encoder *encoder,
> clock = adjusted_mode->clock;
> dm_new_connector_state->pbn =
> drm_dp_calc_pbn_mode(clock, bpp, false);
> }
> - dm_new_connector_state->vcpi_slots =
> drm_dp_atomic_find_vcpi_slots(state,
> + dm_new_connector_state->vcpi_slots =
> +drm_dp_atomic_find_time_slots(state,
>
> mst_mgr,
>
> mst_port,
>
> dm_new_connector_state->pbn, diff --git
> a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> index 9221b6690a4a..e40ff51e7be0 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
> @@ -378,7 +378,7 @@ static int dm_dp_mst_atomic_check(struct
> drm_connector *connector,
> return 0;
> }
>
> - return drm_dp_atomic_release_vcpi_slots(state,
> + return drm_dp_atomic_release_time_slots(state,
> mst_mgr,
> mst_port);
> }
> @@ -689,7 +689,7 @@ static void increase_dsc_bpp(struct
> drm_atomic_state *state,
>
> if (initial_slack[next_index] > fair_pbn_alloc) {
> vars[next_index].pbn += fair_pbn_alloc;
> - if (drm_dp_atomic_find_vcpi_slots(state,
> + if (drm_dp_atomic_find_time_slots(state,
>
> params[next_index].port->mgr,
>
> params[next_index].port,
>
> vars[next_index].pbn,
> @@ -699,7 +699,7 @@ static void increase_dsc_bpp(struct
> drm_atomic_state *state,
> vars[next_index].bpp_x16 =
> bpp_x16_from_pbn(params[next_index], vars[next_index].pbn);
> } else {
> vars[next_index].pbn -= fair_pbn_alloc;
> - if (drm_dp_atomic_find_vcpi_slots(state,
> + if (drm_dp_atomic_find_time_slots(state,
>
> params[next_index].port->mgr,
>
> params[next_index].port,
>
> vars[next_index].pbn,
> @@ -708,7 +708,7 @@ static void increase_dsc_bpp(struct
> drm_atomic_state *state,
> }
> } else {
> vars[next_index].pbn += initial_slack[next_index];
> - if (drm_dp_atomic_find_vcpi_slots(state,
> + if (drm_dp_atomic_find_time_slots(state,
>
> params[next_index].port->mgr,
>
> params[next_index].port,
>
> vars[next_index].pbn,
> @@ -718,7 +718,7 @@ static void increase_dsc_bpp(struct
> drm_atomic_state *state,
> vars[next_index].bpp_x16 =
> params[next_index].bw_range.max_target_bpp_x16;
> } else {
> vars[next_index].pbn -=
> initial_slack[next_index];
> - if (drm_dp_atomic_find_vcpi_slots(state,
> + if (drm_dp_atomic_find_time_slots(state,
>
> params[next_index].port->mgr,
>
> params[next_index].port,
>
> vars[next_index].pbn,
> @@ -775,7 +775,7 @@ static void try_disable_dsc(struct drm_atomic_state
> *state,
> break;
>
> vars[next_index].pbn =
> kbps_to_peak_pbn(params[next_index].bw_range.stream_kbps);
> - if (drm_dp_atomic_find_vcpi_slots(state,
> + if (drm_dp_atomic_find_time_slots(state,
> params[next_index].port-
> >mgr,
> params[next_index].port,
> vars[next_index].pbn,
> @@ -787,7 +787,7 @@ static void try_disable_dsc(struct drm_atomic_state
> *state,
> vars[next_index].bpp_x16 = 0;
> } else {
> vars[next_index].pbn =
> kbps_to_peak_pbn(params[next_index].bw_range.max_kbps);
> - if (drm_dp_atomic_find_vcpi_slots(state,
> + if (drm_dp_atomic_find_time_slots(state,
>
> params[next_index].port->mgr,
>
> params[next_index].port,
>
> vars[next_index].pbn,
> @@ -873,11 +873,11 @@ static bool
> compute_mst_dsc_configs_for_link(struct drm_atomic_state *state,
> vars[i + k].pbn =
> kbps_to_peak_pbn(params[i].bw_range.stream_kbps);
> vars[i + k].dsc_enabled = false;
> vars[i + k].bpp_x16 = 0;
> - if (drm_dp_atomic_find_vcpi_slots(state,
> - params[i].port->mgr,
> - params[i].port,
> - vars[i + k].pbn,
> -
> dm_mst_get_pbn_divider(dc_link)) < 0)
> + if (drm_dp_atomic_find_time_slots(state,
> + params[i].port->mgr,
> + params[i].port,
> + vars[i + k].pbn,
> +
> dm_mst_get_pbn_divider(dc_link)) < 0)
> return false;
> }
> if (!drm_dp_mst_atomic_check(state) && !debugfs_overwrite) { @@
> -891,7 +891,7 @@ static bool compute_mst_dsc_configs_for_link(struct
> drm_atomic_state *state,
> vars[i + k].pbn =
> kbps_to_peak_pbn(params[i].bw_range.min_kbps);
> vars[i + k].dsc_enabled = true;
> vars[i + k].bpp_x16 =
> params[i].bw_range.min_target_bpp_x16;
> - if (drm_dp_atomic_find_vcpi_slots(state,
> + if (drm_dp_atomic_find_time_slots(state,
> params[i].port->mgr,
> params[i].port,
> vars[i + k].pbn,
> @@ -901,7 +901,7 @@ static bool compute_mst_dsc_configs_for_link(struct
> drm_atomic_state *state,
> vars[i + k].pbn =
> kbps_to_peak_pbn(params[i].bw_range.stream_kbps);
> vars[i + k].dsc_enabled = false;
> vars[i + k].bpp_x16 = 0;
> - if (drm_dp_atomic_find_vcpi_slots(state,
> + if (drm_dp_atomic_find_time_slots(state,
> params[i].port->mgr,
> params[i].port,
> vars[i + k].pbn,
> diff --git a/drivers/gpu/drm/display/drm_dp_mst_topology.c
> b/drivers/gpu/drm/display/drm_dp_mst_topology.c
> index 38eecb89e22d..702ff5d9ecc7 100644
> --- a/drivers/gpu/drm/display/drm_dp_mst_topology.c
> +++ b/drivers/gpu/drm/display/drm_dp_mst_topology.c
> @@ -4304,11 +4304,11 @@ struct edid *drm_dp_mst_get_edid(struct
> drm_connector *connector, struct drm_dp_
> EXPORT_SYMBOL(drm_dp_mst_get_edid);
>
> /**
> - * drm_dp_find_vcpi_slots() - Find VCPI slots for this PBN value
> + * drm_dp_find_vcpi_slots() - Find time slots for this PBN value
> * @mgr: manager to use
> * @pbn: payload bandwidth to convert into slots.
> *
> - * Calculate the number of VCPI slots that will be required for the given PBN
> + * Calculate the number of time slots that will be required for the
> + given PBN
> * value. This function is deprecated, and should not be used in atomic
> * drivers.
> *
> @@ -4345,17 +4345,17 @@ static int drm_dp_init_vcpi(struct
> drm_dp_mst_topology_mgr *mgr, }
>
> /**
> - * drm_dp_atomic_find_vcpi_slots() - Find and add VCPI slots to the state
> + * drm_dp_atomic_find_time_slots() - Find and add time slots to the
> + state
> * @state: global atomic state
> * @mgr: MST topology manager for the port
> - * @port: port to find vcpi slots for
> + * @port: port to find time slots for
> * @pbn: bandwidth required for the mode in PBN
> * @pbn_div: divider for DSC mode that takes FEC into account
> *
> - * Allocates VCPI slots to @port, replacing any previous VCPI allocations it
> + * Allocates time slots to @port, replacing any previous timeslot
> + allocations it
> * may have had. Any atomic drivers which support MST must call this
> function
> * in their &drm_encoder_helper_funcs.atomic_check() callback to change
> the
> - * current VCPI allocation for the new state, but only when
> + * current timeslot allocation for the new state, but only when
> * &drm_crtc_state.mode_changed or &drm_crtc_state.connectors_changed
> is set
> * to ensure compatibility with userspace applications that still use the
> * legacy modesetting UAPI.
> @@ -4365,17 +4365,17 @@ static int drm_dp_init_vcpi(struct
> drm_dp_mst_topology_mgr *mgr,
> *
> * Additionally, it is OK to call this function multiple times on the same
> * @port as needed. It is not OK however, to call this function and
> - * drm_dp_atomic_release_vcpi_slots() in the same atomic check phase.
> + * drm_dp_atomic_release_time_slots() in the same atomic check phase.
> *
> * See also:
> - * drm_dp_atomic_release_vcpi_slots()
> + * drm_dp_atomic_release_time_slots()
> * drm_dp_mst_atomic_check()
> *
> * Returns:
> * Total slots in the atomic state assigned for this port, or a negative error
> * code if the port no longer exists
> */
> -int drm_dp_atomic_find_vcpi_slots(struct drm_atomic_state *state,
> +int drm_dp_atomic_find_time_slots(struct drm_atomic_state *state,
> struct drm_dp_mst_topology_mgr *mgr,
> struct drm_dp_mst_port *port, int pbn,
> int pbn_div)
> @@ -4392,17 +4392,17 @@ int drm_dp_atomic_find_vcpi_slots(struct
> drm_atomic_state *state,
> list_for_each_entry(pos, &topology_state->payloads, next) {
> if (pos->port == port) {
> payload = pos;
> - prev_slots = payload->vcpi;
> + prev_slots = payload->time_slots;
> prev_bw = payload->pbn;
>
> /*
> * This should never happen, unless the driver tries
> - * releasing and allocating the same VCPI allocation,
> + * releasing and allocating the same timeslot
> allocation,
> * which is an error
> */
> if (WARN_ON(!prev_slots)) {
> drm_err(mgr->dev,
> - "cannot allocate and release VCPI on
> [MST PORT:%p] in the same state\n",
> + "cannot allocate and release time
> slots on [MST PORT:%p] in the
> +same state\n",
> port);
> return -EINVAL;
> }
> @@ -4420,7 +4420,7 @@ int drm_dp_atomic_find_vcpi_slots(struct
> drm_atomic_state *state,
>
> req_slots = DIV_ROUND_UP(pbn, pbn_div);
>
> - drm_dbg_atomic(mgr->dev, "[CONNECTOR:%d:%s] [MST PORT:%p]
> VCPI %d -> %d\n",
> + drm_dbg_atomic(mgr->dev, "[CONNECTOR:%d:%s] [MST PORT:%p]
> TU %d ->
> +%d\n",
> port->connector->base.id, port->connector->name,
> port, prev_slots, req_slots);
> drm_dbg_atomic(mgr->dev, "[CONNECTOR:%d:%s] [MST PORT:%p]
> PBN %d -> %d\n", @@ -4437,20 +4437,20 @@ int
> drm_dp_atomic_find_vcpi_slots(struct drm_atomic_state *state,
> payload->port = port;
> list_add(&payload->next, &topology_state->payloads);
> }
> - payload->vcpi = req_slots;
> + payload->time_slots = req_slots;
> payload->pbn = pbn;
>
> return req_slots;
> }
> -EXPORT_SYMBOL(drm_dp_atomic_find_vcpi_slots);
> +EXPORT_SYMBOL(drm_dp_atomic_find_time_slots);
>
> /**
> - * drm_dp_atomic_release_vcpi_slots() - Release allocated vcpi slots
> + * drm_dp_atomic_release_time_slots() - Release allocated time slots
> * @state: global atomic state
> * @mgr: MST topology manager for the port
> - * @port: The port to release the VCPI slots from
> + * @port: The port to release the time slots from
> *
> - * Releases any VCPI slots that have been allocated to a port in the atomic
> + * Releases any time slots that have been allocated to a port in the
> + atomic
> * state. Any atomic drivers which support MST must call this function in
> * their &drm_connector_helper_funcs.atomic_check() callback when the
> * connector will no longer have VCPI allocated (e.g. because its CRTC was
> @@ -4459,18 +4459,18 @@
> EXPORT_SYMBOL(drm_dp_atomic_find_vcpi_slots);
> * It is OK to call this even if @port has been removed from the system.
> * Additionally, it is OK to call this function multiple times on the same
> * @port as needed. It is not OK however, to call this function and
> - * drm_dp_atomic_find_vcpi_slots() on the same @port in a single atomic
> check
> + * drm_dp_atomic_find_time_slots() on the same @port in a single atomic
> + check
> * phase.
> *
> * See also:
> - * drm_dp_atomic_find_vcpi_slots()
> + * drm_dp_atomic_find_time_slots()
> * drm_dp_mst_atomic_check()
> *
> * Returns:
> * 0 if all slots for this port were added back to
> * &drm_dp_mst_topology_state.avail_slots or negative error code
> */
> -int drm_dp_atomic_release_vcpi_slots(struct drm_atomic_state *state,
> +int drm_dp_atomic_release_time_slots(struct drm_atomic_state *state,
> struct drm_dp_mst_topology_mgr *mgr,
> struct drm_dp_mst_port *port)
> {
> @@ -4494,16 +4494,16 @@ int drm_dp_atomic_release_vcpi_slots(struct
> drm_atomic_state *state,
> return -EINVAL;
> }
>
> - drm_dbg_atomic(mgr->dev, "[MST PORT:%p] VCPI %d -> 0\n", port,
> pos->vcpi);
> - if (pos->vcpi) {
> + drm_dbg_atomic(mgr->dev, "[MST PORT:%p] TU %d -> 0\n", port,
> pos->time_slots);
> + if (pos->time_slots) {
> drm_dp_mst_put_port_malloc(port);
> - pos->vcpi = 0;
> + pos->time_slots = 0;
> pos->pbn = 0;
> }
>
> return 0;
> }
> -EXPORT_SYMBOL(drm_dp_atomic_release_vcpi_slots);
> +EXPORT_SYMBOL(drm_dp_atomic_release_time_slots);
>
> /**
> * drm_dp_mst_update_slots() - updates the slot info depending on the DP
> ecoding format @@ -4557,7 +4557,7 @@ bool
> drm_dp_mst_allocate_vcpi(struct drm_dp_mst_topology_mgr *mgr,
>
> ret = drm_dp_init_vcpi(mgr, &port->vcpi, pbn, slots);
> if (ret) {
> - drm_dbg_kms(mgr->dev, "failed to init vcpi slots=%d
> ret=%d\n",
> + drm_dbg_kms(mgr->dev, "failed to init time slots=%d
> ret=%d\n",
> DIV_ROUND_UP(pbn, mgr->pbn_div), ret);
> drm_dp_mst_topology_put_port(port);
> goto out;
> @@ -5083,8 +5083,8 @@ drm_dp_mst_duplicate_state(struct
> drm_private_obj *obj)
> INIT_LIST_HEAD(&state->payloads);
>
> list_for_each_entry(pos, &old_state->payloads, next) {
> - /* Prune leftover freed VCPI allocations */
> - if (!pos->vcpi)
> + /* Prune leftover freed timeslot allocations */
> + if (!pos->time_slots)
> continue;
>
> payload = kmemdup(pos, sizeof(*payload), GFP_KERNEL);
> @@ -5116,7 +5116,7 @@ static void drm_dp_mst_destroy_state(struct
> drm_private_obj *obj,
>
> list_for_each_entry_safe(pos, tmp, &mst_state->payloads, next) {
> /* We only keep references to ports with non-zero VCPIs */
> - if (pos->vcpi)
> + if (pos->time_slots)
> drm_dp_mst_put_port_malloc(pos->port);
> kfree(pos);
> }
> @@ -5242,28 +5242,28 @@
> drm_dp_mst_atomic_check_port_bw_limit(struct drm_dp_mst_port *port, }
>
> static inline int
> -drm_dp_mst_atomic_check_vcpi_alloc_limit(struct
> drm_dp_mst_topology_mgr *mgr,
> - struct drm_dp_mst_topology_state
> *mst_state)
> +drm_dp_mst_atomic_check_payload_alloc_limits(struct
> drm_dp_mst_topology_mgr *mgr,
> + struct
> drm_dp_mst_topology_state *mst_state)
> {
> struct drm_dp_mst_atomic_payload *payload;
> int avail_slots = mst_state->total_avail_slots, payload_count = 0;
>
> list_for_each_entry(payload, &mst_state->payloads, next) {
> /* Releasing payloads is always OK-even if the port is gone */
> - if (!payload->vcpi) {
> - drm_dbg_atomic(mgr->dev, "[MST PORT:%p]
> releases all VCPI slots\n",
> + if (!payload->time_slots) {
> + drm_dbg_atomic(mgr->dev, "[MST PORT:%p]
> releases all time slots\n",
> payload->port);
> continue;
> }
>
> - drm_dbg_atomic(mgr->dev, "[MST PORT:%p] requires %d
> vcpi slots\n",
> - payload->port, payload->vcpi);
> + drm_dbg_atomic(mgr->dev, "[MST PORT:%p] requires %d
> time slots\n",
> + payload->port, payload->time_slots);
>
> - avail_slots -= payload->vcpi;
> + avail_slots -= payload->time_slots;
> if (avail_slots < 0) {
> drm_dbg_atomic(mgr->dev,
> - "[MST PORT:%p] not enough VCPI slots in
> mst state %p (avail=%d)\n",
> - payload->port, mst_state, avail_slots +
> payload->vcpi);
> + "[MST PORT:%p] not enough time slots in
> mst state %p (avail=%d)\n",
> + payload->port, mst_state, avail_slots +
> +payload->time_slots);
> return -ENOSPC;
> }
>
> @@ -5274,7 +5274,7 @@ drm_dp_mst_atomic_check_vcpi_alloc_limit(struct
> drm_dp_mst_topology_mgr *mgr,
> return -EINVAL;
> }
> }
> - drm_dbg_atomic(mgr->dev, "[MST MGR:%p] mst state %p VCPI
> avail=%d used=%d\n",
> + drm_dbg_atomic(mgr->dev, "[MST MGR:%p] mst state %p TU
> avail=%d
> +used=%d\n",
> mgr, mst_state, avail_slots, mst_state->total_avail_slots -
> avail_slots);
>
> return 0;
> @@ -5363,7 +5363,7 @@ int drm_dp_mst_atomic_enable_dsc(struct
> drm_atomic_state *state,
> struct drm_dp_mst_topology_state *mst_state;
> struct drm_dp_mst_atomic_payload *pos;
> bool found = false;
> - int vcpi = 0;
> + int time_slots = 0;
>
> mst_state = drm_atomic_get_mst_topology_state(state, port->mgr);
>
> @@ -5379,30 +5379,30 @@ int drm_dp_mst_atomic_enable_dsc(struct
> drm_atomic_state *state,
>
> if (!found) {
> drm_dbg_atomic(state->dev,
> - "[MST PORT:%p] Couldn't find VCPI allocation in
> mst state %p\n",
> + "[MST PORT:%p] Couldn't find payload in mst
> state %p\n",
> port, mst_state);
> return -EINVAL;
> }
>
> if (pos->dsc_enabled == enable) {
> drm_dbg_atomic(state->dev,
> - "[MST PORT:%p] DSC flag is already set to %d,
> returning %d VCPI slots\n",
> - port, enable, pos->vcpi);
> - vcpi = pos->vcpi;
> + "[MST PORT:%p] DSC flag is already set to %d,
> returning %d time slots\n",
> + port, enable, pos->time_slots);
> + time_slots = pos->time_slots;
> }
>
> if (enable) {
> - vcpi = drm_dp_atomic_find_vcpi_slots(state, port->mgr, port,
> pbn, pbn_div);
> + time_slots = drm_dp_atomic_find_time_slots(state, port-
> >mgr, port,
> +pbn, pbn_div);
> drm_dbg_atomic(state->dev,
> - "[MST PORT:%p] Enabling DSC flag, reallocating
> %d VCPI slots on the port\n",
> - port, vcpi);
> - if (vcpi < 0)
> + "[MST PORT:%p] Enabling DSC flag, reallocating
> %d time slots on the port\n",
> + port, time_slots);
> + if (time_slots < 0)
> return -EINVAL;
> }
>
> pos->dsc_enabled = enable;
>
> - return vcpi;
> + return time_slots;
> }
> EXPORT_SYMBOL(drm_dp_mst_atomic_enable_dsc);
> /**
> @@ -5412,15 +5412,15 @@
> EXPORT_SYMBOL(drm_dp_mst_atomic_enable_dsc);
> *
> * Checks the given topology state for an atomic update to ensure that it's
> * valid. This includes checking whether there's enough bandwidth to
> support
> - * the new VCPI allocations in the atomic update.
> + * the new timeslot allocations in the atomic update.
> *
> * Any atomic drivers supporting DP MST must make sure to call this after
> * checking the rest of their state in their
> * &drm_mode_config_funcs.atomic_check() callback.
> *
> * See also:
> - * drm_dp_atomic_find_vcpi_slots()
> - * drm_dp_atomic_release_vcpi_slots()
> + * drm_dp_atomic_find_time_slots()
> + * drm_dp_atomic_release_time_slots()
> *
> * Returns:
> *
> @@ -5436,7 +5436,7 @@ int drm_dp_mst_atomic_check(struct
> drm_atomic_state *state)
> if (!mgr->mst_state)
> continue;
>
> - ret = drm_dp_mst_atomic_check_vcpi_alloc_limit(mgr,
> mst_state);
> + ret = drm_dp_mst_atomic_check_payload_alloc_limits(mgr,
> mst_state);
> if (ret)
> break;
>
> diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c
> b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> index 061b277e5ce7..0c922667398a 100644
> --- a/drivers/gpu/drm/i915/display/intel_dp_mst.c
> +++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c
> @@ -70,7 +70,7 @@ static int intel_dp_mst_compute_link_config(struct
> intel_encoder *encoder,
> crtc_state->pipe_bpp,
> false);
>
> - slots = drm_dp_atomic_find_vcpi_slots(state, &intel_dp-
> >mst_mgr,
> + slots = drm_dp_atomic_find_time_slots(state, &intel_dp-
> >mst_mgr,
> connector->port,
> crtc_state->pbn,
>
> drm_dp_get_vc_payload_bw(&intel_dp->mst_mgr,
> @@ -344,8 +344,7 @@ intel_dp_mst_atomic_check(struct drm_connector
> *connector,
> }
>
> mgr = &enc_to_mst(to_intel_encoder(old_conn_state-
> >best_encoder))->primary->dp.mst_mgr;
> - ret = drm_dp_atomic_release_vcpi_slots(&state->base, mgr,
> - intel_connector->port);
> + ret = drm_dp_atomic_release_time_slots(&state->base, mgr,
> +intel_connector->port);
>
> return ret;
> }
> diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c
> b/drivers/gpu/drm/nouveau/dispnv50/disp.c
> index 4347f0b61797..631dba5a2418 100644
> --- a/drivers/gpu/drm/nouveau/dispnv50/disp.c
> +++ b/drivers/gpu/drm/nouveau/dispnv50/disp.c
> @@ -1070,7 +1070,7 @@ nv50_msto_atomic_check(struct drm_encoder
> *encoder,
> false);
> }
>
> - slots = drm_dp_atomic_find_vcpi_slots(state, &mstm->mgr, mstc-
> >port,
> + slots = drm_dp_atomic_find_time_slots(state, &mstm->mgr, mstc-
> >port,
> asyh->dp.pbn, 0);
> if (slots < 0)
> return slots;
> @@ -1282,7 +1282,7 @@ nv50_mstc_atomic_check(struct drm_connector
> *connector,
> return 0;
> }
>
> - return drm_dp_atomic_release_vcpi_slots(state, mgr, mstc->port);
> + return drm_dp_atomic_release_time_slots(state, mgr, mstc->port);
> }
>
> static int
> diff --git a/include/drm/display/drm_dp_mst_helper.h
> b/include/drm/display/drm_dp_mst_helper.h
> index 5671173f9f37..8ab4f14f2344 100644
> --- a/include/drm/display/drm_dp_mst_helper.h
> +++ b/include/drm/display/drm_dp_mst_helper.h
> @@ -544,7 +544,7 @@ struct drm_dp_payload {
>
> struct drm_dp_mst_atomic_payload {
> struct drm_dp_mst_port *port;
> - int vcpi;
> + int time_slots;
> int pbn;
> bool dsc_enabled;
> struct list_head next;
> @@ -846,7 +846,7 @@ void drm_dp_mst_connector_early_unregister(struct
> drm_connector *connector, struct drm_dp_mst_topology_state
> *drm_atomic_get_mst_topology_state(struct drm_atomic_state *state,
> struct
> drm_dp_mst_topology_mgr *mgr); int __must_check -
> drm_dp_atomic_find_vcpi_slots(struct drm_atomic_state *state,
> +drm_dp_atomic_find_time_slots(struct drm_atomic_state *state,
> struct drm_dp_mst_topology_mgr *mgr,
> struct drm_dp_mst_port *port, int pbn,
> int pbn_div);
> @@ -858,7 +858,7 @@ int __must_check
> drm_dp_mst_add_affected_dsc_crtcs(struct drm_atomic_state *state,
> struct drm_dp_mst_topology_mgr *mgr);
> int __must_check -drm_dp_atomic_release_vcpi_slots(struct
> drm_atomic_state *state,
> +drm_dp_atomic_release_time_slots(struct drm_atomic_state *state,
> struct drm_dp_mst_topology_mgr *mgr,
> struct drm_dp_mst_port *port);
> int drm_dp_send_power_updown_phy(struct drm_dp_mst_topology_mgr
> *mgr,
> --
> 2.35.3
--
Regards,
Wayne Lin