mbox series

[PULL] topic/amdgpu-dp2.0-mst

Message ID bf8e724cc0c8803d58a8d730fd6883c991376a76.camel@redhat.com (mailing list archive)
State New, archived
Headers show
Series [PULL] topic/amdgpu-dp2.0-mst | expand

Pull-request

git://anongit.freedesktop.org/drm/drm-misc tags/topic/amdgpu-dp2.0-mst-2021-

Message

Lyude Paul Oct. 28, 2021, 12:26 a.m. UTC
topic/amdgpu-dp2.0-mst-2021-10-27:
UAPI Changes:
Nope!

Cross-subsystem Changes:
drm_dp_update_payload_part1() takes a new argument for specifying what the
VCPI slot start is

Core Changes:
Make the DP MST helpers aware of the current starting VCPI slot/VCPI total
slot count...

Driver Changes:
...and then add support for taking advantage of this for 128b/132b links on DP
2.0 for amdgpu
The following changes since commit 6f2f7c83303d2227f47551423e507d77d9ea01c7:

  Merge tag 'drm-intel-gt-next-2021-10-21' of
git://anongit.freedesktop.org/drm/drm-intel into drm-next (2021-10-22 06:30:34
+1000)

are available in the Git repository at:

  git://anongit.freedesktop.org/drm/drm-misc tags/topic/amdgpu-dp2.0-mst-2021-
10-27

for you to fetch changes up to 00f965e700ef5aa2d889e7e65c7458531d2a4bcf:

  drm/amdgpu/display: fix build when CONFIG_DRM_AMD_DC_DCN is not set (2021-
10-27 19:50:26 -0400)

----------------------------------------------------------------
UAPI Changes:
Nope!

Cross-subsystem Changes:
drm_dp_update_payload_part1() takes a new argument for specifying what the
VCPI slot start is

Core Changes:
Make the DP MST helpers aware of the current starting VCPI slot/VCPI total
slot count...

Driver Changes:
...and then add support for taking advantage of this for 128b/132b links on DP
2.0 for amdgpu

----------------------------------------------------------------
Alex Deucher (1):
      drm/amdgpu/display: fix build when CONFIG_DRM_AMD_DC_DCN is not set

Bhawanpreet Lakha (3):
      drm: Remove slot checks in dp mst topology during commit
      drm: Update MST First Link Slot Information Based on Encoding Format
      drm/amd/display: Add DP 2.0 MST DM Support

Fangzhi Zuo (1):
      drm/amd/display: Add DP 2.0 MST DC Support

 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c  |  29 ++
 .../drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c  |   3 +
 .../drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c  |   7 +-
 drivers/gpu/drm/amd/display/dc/core/dc.c           |  14 +
 drivers/gpu/drm/amd/display/dc/core/dc_link.c      | 292
+++++++++++++++++++++
 drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c   |  19 ++
 drivers/gpu/drm/amd/display/dc/dc_link.h           |   7 +
 drivers/gpu/drm/amd/display/dc/dc_stream.h         |  13 +
 drivers/gpu/drm/drm_dp_mst_topology.c              |  42 ++-
 drivers/gpu/drm/i915/display/intel_dp_mst.c        |   4 +-
 drivers/gpu/drm/nouveau/dispnv50/disp.c            |   2 +-
 drivers/gpu/drm/radeon/radeon_dp_mst.c             |   4 +-
 include/drm/drm_dp_mst_helper.h                    |   5 +-
 13 files changed, 425 insertions(+), 16 deletions(-)

Comments

Jani Nikula Oct. 29, 2021, 10:23 a.m. UTC | #1
On Wed, 27 Oct 2021, Lyude Paul <lyude@redhat.com> wrote:
> topic/amdgpu-dp2.0-mst-2021-10-27:
> UAPI Changes:
> Nope!
>
> Cross-subsystem Changes:
> drm_dp_update_payload_part1() takes a new argument for specifying what the
> VCPI slot start is
>
> Core Changes:
> Make the DP MST helpers aware of the current starting VCPI slot/VCPI total
> slot count...
>
> Driver Changes:
> ...and then add support for taking advantage of this for 128b/132b links on DP
> 2.0 for amdgpu
> The following changes since commit 6f2f7c83303d2227f47551423e507d77d9ea01c7:
>
>   Merge tag 'drm-intel-gt-next-2021-10-21' of
> git://anongit.freedesktop.org/drm/drm-intel into drm-next (2021-10-22 06:30:34
> +1000)
>
> are available in the Git repository at:
>
>   git://anongit.freedesktop.org/drm/drm-misc tags/topic/amdgpu-dp2.0-mst-2021-
> 10-27

I'm curious, how did you generate and send this pull request? The lines
are wrapped with newlines, and you have non-breaking spaces instead of
regular spaces there.

So for me this fails with:

Pulling   git://anongit.freedesktop.org/drm/drm-misc tags/topic/amdgpu-dp2.0-mst-2021- 10-27 ...
fatal: invalid refspec 'git://anongit.freedesktop.org/drm/drm-misc'

BR,
Jani.


>
> for you to fetch changes up to 00f965e700ef5aa2d889e7e65c7458531d2a4bcf:
>
>   drm/amdgpu/display: fix build when CONFIG_DRM_AMD_DC_DCN is not set (2021-
> 10-27 19:50:26 -0400)
>
> ----------------------------------------------------------------
> UAPI Changes:
> Nope!
>
> Cross-subsystem Changes:
> drm_dp_update_payload_part1() takes a new argument for specifying what the
> VCPI slot start is
>
> Core Changes:
> Make the DP MST helpers aware of the current starting VCPI slot/VCPI total
> slot count...
>
> Driver Changes:
> ...and then add support for taking advantage of this for 128b/132b links on DP
> 2.0 for amdgpu
>
> ----------------------------------------------------------------
> Alex Deucher (1):
>       drm/amdgpu/display: fix build when CONFIG_DRM_AMD_DC_DCN is not set
>
> Bhawanpreet Lakha (3):
>       drm: Remove slot checks in dp mst topology during commit
>       drm: Update MST First Link Slot Information Based on Encoding Format
>       drm/amd/display: Add DP 2.0 MST DM Support
>
> Fangzhi Zuo (1):
>       drm/amd/display: Add DP 2.0 MST DC Support
>
>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c  |  29 ++
>  .../drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c  |   3 +
>  .../drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c  |   7 +-
>  drivers/gpu/drm/amd/display/dc/core/dc.c           |  14 +
>  drivers/gpu/drm/amd/display/dc/core/dc_link.c      | 292
> +++++++++++++++++++++
>  drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c   |  19 ++
>  drivers/gpu/drm/amd/display/dc/dc_link.h           |   7 +
>  drivers/gpu/drm/amd/display/dc/dc_stream.h         |  13 +
>  drivers/gpu/drm/drm_dp_mst_topology.c              |  42 ++-
>  drivers/gpu/drm/i915/display/intel_dp_mst.c        |   4 +-
>  drivers/gpu/drm/nouveau/dispnv50/disp.c            |   2 +-
>  drivers/gpu/drm/radeon/radeon_dp_mst.c             |   4 +-
>  include/drm/drm_dp_mst_helper.h                    |   5 +-
>  13 files changed, 425 insertions(+), 16 deletions(-)
Jani Nikula Oct. 29, 2021, 10:35 a.m. UTC | #2
On Fri, 29 Oct 2021, Jani Nikula <jani.nikula@linux.intel.com> wrote:
> On Wed, 27 Oct 2021, Lyude Paul <lyude@redhat.com> wrote:
>> topic/amdgpu-dp2.0-mst-2021-10-27:
>> UAPI Changes:
>> Nope!
>>
>> Cross-subsystem Changes:
>> drm_dp_update_payload_part1() takes a new argument for specifying what the
>> VCPI slot start is
>>
>> Core Changes:
>> Make the DP MST helpers aware of the current starting VCPI slot/VCPI total
>> slot count...
>>
>> Driver Changes:
>> ...and then add support for taking advantage of this for 128b/132b links on DP
>> 2.0 for amdgpu
>> The following changes since commit 6f2f7c83303d2227f47551423e507d77d9ea01c7:
>>
>>   Merge tag 'drm-intel-gt-next-2021-10-21' of
>> git://anongit.freedesktop.org/drm/drm-intel into drm-next (2021-10-22 06:30:34
>> +1000)
>>
>> are available in the Git repository at:
>>
>>   git://anongit.freedesktop.org/drm/drm-misc tags/topic/amdgpu-dp2.0-mst-2021-
>> 10-27
>
> I'm curious, how did you generate and send this pull request? The lines
> are wrapped with newlines, and you have non-breaking spaces instead of
> regular spaces there.
>
> So for me this fails with:
>
> Pulling   git://anongit.freedesktop.org/drm/drm-misc tags/topic/amdgpu-dp2.0-mst-2021- 10-27 ...
> fatal: invalid refspec 'git://anongit.freedesktop.org/drm/drm-misc'

Fixed manually, but I can't pull this into drm-intel-next directly after
all, because the baseline is not in drm-intel-next history. The diffstat
for drm-intel-next is:

65 files changed, 3656 insertions(+), 780 deletions(-)

I asked for this to be a topic branch so I could pull it into
drm-intel-next. I guess I'll just have to do a drm-next backmerge
instead.

BR,
Jani.


>
> BR,
> Jani.
>
>
>>
>> for you to fetch changes up to 00f965e700ef5aa2d889e7e65c7458531d2a4bcf:
>>
>>   drm/amdgpu/display: fix build when CONFIG_DRM_AMD_DC_DCN is not set (2021-
>> 10-27 19:50:26 -0400)
>>
>> ----------------------------------------------------------------
>> UAPI Changes:
>> Nope!
>>
>> Cross-subsystem Changes:
>> drm_dp_update_payload_part1() takes a new argument for specifying what the
>> VCPI slot start is
>>
>> Core Changes:
>> Make the DP MST helpers aware of the current starting VCPI slot/VCPI total
>> slot count...
>>
>> Driver Changes:
>> ...and then add support for taking advantage of this for 128b/132b links on DP
>> 2.0 for amdgpu
>>
>> ----------------------------------------------------------------
>> Alex Deucher (1):
>>       drm/amdgpu/display: fix build when CONFIG_DRM_AMD_DC_DCN is not set
>>
>> Bhawanpreet Lakha (3):
>>       drm: Remove slot checks in dp mst topology during commit
>>       drm: Update MST First Link Slot Information Based on Encoding Format
>>       drm/amd/display: Add DP 2.0 MST DM Support
>>
>> Fangzhi Zuo (1):
>>       drm/amd/display: Add DP 2.0 MST DC Support
>>
>>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c  |  29 ++
>>  .../drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c  |   3 +
>>  .../drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c  |   7 +-
>>  drivers/gpu/drm/amd/display/dc/core/dc.c           |  14 +
>>  drivers/gpu/drm/amd/display/dc/core/dc_link.c      | 292
>> +++++++++++++++++++++
>>  drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c   |  19 ++
>>  drivers/gpu/drm/amd/display/dc/dc_link.h           |   7 +
>>  drivers/gpu/drm/amd/display/dc/dc_stream.h         |  13 +
>>  drivers/gpu/drm/drm_dp_mst_topology.c              |  42 ++-
>>  drivers/gpu/drm/i915/display/intel_dp_mst.c        |   4 +-
>>  drivers/gpu/drm/nouveau/dispnv50/disp.c            |   2 +-
>>  drivers/gpu/drm/radeon/radeon_dp_mst.c             |   4 +-
>>  include/drm/drm_dp_mst_helper.h                    |   5 +-
>>  13 files changed, 425 insertions(+), 16 deletions(-)
Lyude Paul Oct. 29, 2021, 5:20 p.m. UTC | #3
JFYI - the wrapping was because of evolution, sorry about that. Going to make
sure that gets fixed the next time I have to send out a topic branch

On Fri, 2021-10-29 at 13:35 +0300, Jani Nikula wrote:
> On Fri, 29 Oct 2021, Jani Nikula <jani.nikula@linux.intel.com> wrote:
> > On Wed, 27 Oct 2021, Lyude Paul <lyude@redhat.com> wrote:
> > > topic/amdgpu-dp2.0-mst-2021-10-27:
> > > UAPI Changes:
> > > Nope!
> > > 
> > > Cross-subsystem Changes:
> > > drm_dp_update_payload_part1() takes a new argument for specifying what
> > > the
> > > VCPI slot start is
> > > 
> > > Core Changes:
> > > Make the DP MST helpers aware of the current starting VCPI slot/VCPI
> > > total
> > > slot count...
> > > 
> > > Driver Changes:
> > > ...and then add support for taking advantage of this for 128b/132b links
> > > on DP
> > > 2.0 for amdgpu
> > > The following changes since commit
> > > 6f2f7c83303d2227f47551423e507d77d9ea01c7:
> > > 
> > >   Merge tag 'drm-intel-gt-next-2021-10-21' of
> > > git://anongit.freedesktop.org/drm/drm-intel into drm-next (2021-10-22
> > > 06:30:34
> > > +1000)
> > > 
> > > are available in the Git repository at:
> > > 
> > >   git://anongit.freedesktop.org/drm/drm-misc tags/topic/amdgpu-dp2.0-
> > > mst-2021-
> > > 10-27
> > 
> > I'm curious, how did you generate and send this pull request? The lines
> > are wrapped with newlines, and you have non-breaking spaces instead of
> > regular spaces there.
> > 
> > So for me this fails with:
> > 
> > Pulling   git://anongit.freedesktop.org/drm/drm-misc tags/topic/amdgpu-
> > dp2.0-mst-2021- 10-27 ...
> > fatal: invalid refspec 'git://anongit.freedesktop.org/drm/drm-misc'
> 
> Fixed manually, but I can't pull this into drm-intel-next directly after
> all, because the baseline is not in drm-intel-next history. The diffstat
> for drm-intel-next is:
> 
> 65 files changed, 3656 insertions(+), 780 deletions(-)
> 
> I asked for this to be a topic branch so I could pull it into
> drm-intel-next. I guess I'll just have to do a drm-next backmerge
> instead.
> 
> BR,
> Jani.
> 
> 
> > 
> > BR,
> > Jani.
> > 
> > 
> > > 
> > > for you to fetch changes up to 00f965e700ef5aa2d889e7e65c7458531d2a4bcf:
> > > 
> > >   drm/amdgpu/display: fix build when CONFIG_DRM_AMD_DC_DCN is not set
> > > (2021-
> > > 10-27 19:50:26 -0400)
> > > 
> > > ----------------------------------------------------------------
> > > UAPI Changes:
> > > Nope!
> > > 
> > > Cross-subsystem Changes:
> > > drm_dp_update_payload_part1() takes a new argument for specifying what
> > > the
> > > VCPI slot start is
> > > 
> > > Core Changes:
> > > Make the DP MST helpers aware of the current starting VCPI slot/VCPI
> > > total
> > > slot count...
> > > 
> > > Driver Changes:
> > > ...and then add support for taking advantage of this for 128b/132b links
> > > on DP
> > > 2.0 for amdgpu
> > > 
> > > ----------------------------------------------------------------
> > > Alex Deucher (1):
> > >       drm/amdgpu/display: fix build when CONFIG_DRM_AMD_DC_DCN is not
> > > set
> > > 
> > > Bhawanpreet Lakha (3):
> > >       drm: Remove slot checks in dp mst topology during commit
> > >       drm: Update MST First Link Slot Information Based on Encoding
> > > Format
> > >       drm/amd/display: Add DP 2.0 MST DM Support
> > > 
> > > Fangzhi Zuo (1):
> > >       drm/amd/display: Add DP 2.0 MST DC Support
> > > 
> > >  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c  |  29 ++
> > >  .../drm/amd/display/amdgpu_dm/amdgpu_dm_debugfs.c  |   3 +
> > >  .../drm/amd/display/amdgpu_dm/amdgpu_dm_helpers.c  |   7 +-
> > >  drivers/gpu/drm/amd/display/dc/core/dc.c           |  14 +
> > >  drivers/gpu/drm/amd/display/dc/core/dc_link.c      | 292
> > > +++++++++++++++++++++
> > >  drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c   |  19 ++
> > >  drivers/gpu/drm/amd/display/dc/dc_link.h           |   7 +
> > >  drivers/gpu/drm/amd/display/dc/dc_stream.h         |  13 +
> > >  drivers/gpu/drm/drm_dp_mst_topology.c              |  42 ++-
> > >  drivers/gpu/drm/i915/display/intel_dp_mst.c        |   4 +-
> > >  drivers/gpu/drm/nouveau/dispnv50/disp.c            |   2 +-
> > >  drivers/gpu/drm/radeon/radeon_dp_mst.c             |   4 +-
> > >  include/drm/drm_dp_mst_helper.h                    |   5 +-
> > >  13 files changed, 425 insertions(+), 16 deletions(-)
>