From: Lyude Paul <lyude@redhat.com>
To: Harry Wentland <harry.wentland@amd.com>,
amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Cc: stable@vger.kernel.org, stanislav.lisovskiy@intel.com,
jerry.zuo@amd.com, bskeggs@redhat.com, Wayne.Lin@amd.com,
ville.syrjala@linux.intel.com, mario.limonciello@amd.com
Subject: Re: [PATCH 0/7] Fix MST on amdgpu
Date: Fri, 20 Jan 2023 18:15:00 -0500 [thread overview]
Message-ID: <965500917008cd17b47c1be1f5eeda7ccc4eede6.camel@redhat.com> (raw)
In-Reply-To: <20230119235200.441386-1-harry.wentland@amd.com>
For the whole series:
Reviewed-by: Lyude Paul <lyude@redhat.com>
So glad to have this fixed finally ♥
On Thu, 2023-01-19 at 18:51 -0500, Harry Wentland wrote:
> MST has been broken on amdgpu after a refactor in drm_dp_mst
> code that was aligning drm_dp_mst more closely with the atomic
> model.
>
> The gitlab issue: https://gitlab.freedesktop.org/drm/amd/-/issues/2171
>
> This series fixes it.
>
> It can be found at
> https://gitlab.freedesktop.org/hwentland/linux/-/tree/mst_regression
>
> A stable-6.1.y rebase can be found at
> https://gitlab.freedesktop.org/hwentland/linux/-/tree/mst_regression_6.1
>
> Lyude Paul (1):
> drm/amdgpu/display/mst: Fix mst_state->pbn_div and slot count
> assignments
>
> Wayne Lin (6):
> drm/amdgpu/display/mst: limit payload to be updated one by one
> drm/amdgpu/display/mst: update mst_mgr relevant variable when long HPD
> drm/drm_print: correct format problem
> drm/display/dp_mst: Correct the kref of port.
> drm/amdgpu/display/mst: adjust the naming of mst_port and port of
> aconnector
> drm/amdgpu/display/mst: adjust the logic in 2nd phase of updating
> payload
>
> drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 4 +-
> .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 48 +++++++++---
> .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 4 +-
> .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c | 2 +-
> .../amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 16 ++--
> .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 76 +++++++++++++------
> .../display/amdgpu_dm/amdgpu_dm_mst_types.c | 53 ++++++-------
> drivers/gpu/drm/amd/display/dc/core/dc_link.c | 14 +++-
> drivers/gpu/drm/display/drm_dp_mst_topology.c | 4 +-
> include/drm/drm_print.h | 2 +-
> 10 files changed, 143 insertions(+), 80 deletions(-)
>
> --
> 2.39.0
>
--
Cheers,
Lyude Paul (she/her)
Software Engineer at Red Hat
WARNING: multiple messages have this Message-ID (diff)
From: Lyude Paul <lyude@redhat.com>
To: Harry Wentland <harry.wentland@amd.com>,
amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Cc: ville.syrjala@linux.intel.com, stanislav.lisovskiy@intel.com,
bskeggs@redhat.com, jerry.zuo@amd.com, mario.limonciello@amd.com,
stable@vger.kernel.org, Wayne.Lin@amd.com
Subject: Re: [PATCH 0/7] Fix MST on amdgpu
Date: Fri, 20 Jan 2023 18:15:00 -0500 [thread overview]
Message-ID: <965500917008cd17b47c1be1f5eeda7ccc4eede6.camel@redhat.com> (raw)
In-Reply-To: <20230119235200.441386-1-harry.wentland@amd.com>
For the whole series:
Reviewed-by: Lyude Paul <lyude@redhat.com>
So glad to have this fixed finally ♥
On Thu, 2023-01-19 at 18:51 -0500, Harry Wentland wrote:
> MST has been broken on amdgpu after a refactor in drm_dp_mst
> code that was aligning drm_dp_mst more closely with the atomic
> model.
>
> The gitlab issue: https://gitlab.freedesktop.org/drm/amd/-/issues/2171
>
> This series fixes it.
>
> It can be found at
> https://gitlab.freedesktop.org/hwentland/linux/-/tree/mst_regression
>
> A stable-6.1.y rebase can be found at
> https://gitlab.freedesktop.org/hwentland/linux/-/tree/mst_regression_6.1
>
> Lyude Paul (1):
> drm/amdgpu/display/mst: Fix mst_state->pbn_div and slot count
> assignments
>
> Wayne Lin (6):
> drm/amdgpu/display/mst: limit payload to be updated one by one
> drm/amdgpu/display/mst: update mst_mgr relevant variable when long HPD
> drm/drm_print: correct format problem
> drm/display/dp_mst: Correct the kref of port.
> drm/amdgpu/display/mst: adjust the naming of mst_port and port of
> aconnector
> drm/amdgpu/display/mst: adjust the logic in 2nd phase of updating
> payload
>
> drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 4 +-
> .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 48 +++++++++---
> .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 4 +-
> .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c | 2 +-
> .../amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 16 ++--
> .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 76 +++++++++++++------
> .../display/amdgpu_dm/amdgpu_dm_mst_types.c | 53 ++++++-------
> drivers/gpu/drm/amd/display/dc/core/dc_link.c | 14 +++-
> drivers/gpu/drm/display/drm_dp_mst_topology.c | 4 +-
> include/drm/drm_print.h | 2 +-
> 10 files changed, 143 insertions(+), 80 deletions(-)
>
> --
> 2.39.0
>
--
Cheers,
Lyude Paul (she/her)
Software Engineer at Red Hat
WARNING: multiple messages have this Message-ID (diff)
From: Lyude Paul <lyude@redhat.com>
To: Harry Wentland <harry.wentland@amd.com>,
amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Cc: stable@vger.kernel.org, stanislav.lisovskiy@intel.com,
jerry.zuo@amd.com, bskeggs@redhat.com, Wayne.Lin@amd.com,
mario.limonciello@amd.com
Subject: Re: [PATCH 0/7] Fix MST on amdgpu
Date: Fri, 20 Jan 2023 18:15:00 -0500 [thread overview]
Message-ID: <965500917008cd17b47c1be1f5eeda7ccc4eede6.camel@redhat.com> (raw)
In-Reply-To: <20230119235200.441386-1-harry.wentland@amd.com>
For the whole series:
Reviewed-by: Lyude Paul <lyude@redhat.com>
So glad to have this fixed finally ♥
On Thu, 2023-01-19 at 18:51 -0500, Harry Wentland wrote:
> MST has been broken on amdgpu after a refactor in drm_dp_mst
> code that was aligning drm_dp_mst more closely with the atomic
> model.
>
> The gitlab issue: https://gitlab.freedesktop.org/drm/amd/-/issues/2171
>
> This series fixes it.
>
> It can be found at
> https://gitlab.freedesktop.org/hwentland/linux/-/tree/mst_regression
>
> A stable-6.1.y rebase can be found at
> https://gitlab.freedesktop.org/hwentland/linux/-/tree/mst_regression_6.1
>
> Lyude Paul (1):
> drm/amdgpu/display/mst: Fix mst_state->pbn_div and slot count
> assignments
>
> Wayne Lin (6):
> drm/amdgpu/display/mst: limit payload to be updated one by one
> drm/amdgpu/display/mst: update mst_mgr relevant variable when long HPD
> drm/drm_print: correct format problem
> drm/display/dp_mst: Correct the kref of port.
> drm/amdgpu/display/mst: adjust the naming of mst_port and port of
> aconnector
> drm/amdgpu/display/mst: adjust the logic in 2nd phase of updating
> payload
>
> drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 4 +-
> .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 48 +++++++++---
> .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 4 +-
> .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c | 2 +-
> .../amd/display/amdgpu_dm/amdgpu_dm_debugfs.c | 16 ++--
> .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 76 +++++++++++++------
> .../display/amdgpu_dm/amdgpu_dm_mst_types.c | 53 ++++++-------
> drivers/gpu/drm/amd/display/dc/core/dc_link.c | 14 +++-
> drivers/gpu/drm/display/drm_dp_mst_topology.c | 4 +-
> include/drm/drm_print.h | 2 +-
> 10 files changed, 143 insertions(+), 80 deletions(-)
>
> --
> 2.39.0
>
--
Cheers,
Lyude Paul (she/her)
Software Engineer at Red Hat
next prev parent reply other threads:[~2023-01-20 23:15 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-19 23:51 [PATCH 0/7] Fix MST on amdgpu Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` [PATCH 1/7] drm/amdgpu/display/mst: Fix mst_state->pbn_div and slot count assignments Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` [PATCH 2/7] drm/amdgpu/display/mst: limit payload to be updated one by one Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` [PATCH 3/7] drm/amdgpu/display/mst: update mst_mgr relevant variable when long HPD Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` [PATCH 4/7] drm/drm_print: correct format problem Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:54 ` kernel test robot
2023-01-19 23:51 ` [PATCH 5/7] drm/display/dp_mst: Correct the kref of port Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` [PATCH 6/7] drm/amdgpu/display/mst: adjust the naming of mst_port and port of aconnector Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:51 ` Harry Wentland
2023-01-19 23:52 ` [PATCH 7/7] drm/amdgpu/display/mst: adjust the logic in 2nd phase of updating payload Harry Wentland
2023-01-19 23:52 ` Harry Wentland
2023-01-19 23:52 ` Harry Wentland
2023-01-20 23:15 ` Lyude Paul [this message]
2023-01-20 23:15 ` [PATCH 0/7] Fix MST on amdgpu Lyude Paul
2023-01-20 23:15 ` Lyude Paul
2023-01-23 17:03 ` Harry Wentland
2023-01-23 17:03 ` Harry Wentland
2023-01-23 17:03 ` Harry Wentland
2023-01-22 19:12 ` Didier 'OdyX' Raboud
2023-01-22 19:12 ` Didier 'OdyX' Raboud
2023-01-22 19:12 ` Didier 'OdyX' Raboud
2023-01-23 17:03 ` Harry Wentland
2023-01-23 17:03 ` Harry Wentland
2023-01-23 17:03 ` Harry Wentland
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=965500917008cd17b47c1be1f5eeda7ccc4eede6.camel@redhat.com \
--to=lyude@redhat.com \
--cc=Wayne.Lin@amd.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=bskeggs@redhat.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=harry.wentland@amd.com \
--cc=jerry.zuo@amd.com \
--cc=mario.limonciello@amd.com \
--cc=stable@vger.kernel.org \
--cc=stanislav.lisovskiy@intel.com \
--cc=ville.syrjala@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.