All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@linux.intel.com>
To: Lyude Paul <lyude@redhat.com>, Dave Airlie <airlied@gmail.com>,
	Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
	Rodrigo Vivi <rodrigo.vivi@intel.com>,
	Sean Paul <sean@poorly.run>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Maxime Ripard <mripard@kernel.org>,
	dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org,
	dim-tools@lists.freedesktop.org
Subject: Re: [PULL] topic/amdgpu-dp2.0-mst
Date: Fri, 29 Oct 2021 13:23:43 +0300	[thread overview]
Message-ID: <87sfwkdt8w.fsf@intel.com> (raw)
In-Reply-To: <bf8e724cc0c8803d58a8d730fd6883c991376a76.camel@redhat.com>

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, Intel Open Source Graphics Center

WARNING: multiple messages have this Message-ID (diff)
From: Jani Nikula <jani.nikula@linux.intel.com>
To: Lyude Paul <lyude@redhat.com>, Dave Airlie <airlied@gmail.com>,
	Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
	Rodrigo Vivi <rodrigo.vivi@intel.com>,
	Sean Paul <sean@poorly.run>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Maxime Ripard <mripard@kernel.org>,
	dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org,
	dim-tools@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PULL] topic/amdgpu-dp2.0-mst
Date: Fri, 29 Oct 2021 13:23:43 +0300	[thread overview]
Message-ID: <87sfwkdt8w.fsf@intel.com> (raw)
In-Reply-To: <bf8e724cc0c8803d58a8d730fd6883c991376a76.camel@redhat.com>

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, Intel Open Source Graphics Center

  reply	other threads:[~2021-10-29 10:23 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-28  0:26 [PULL] topic/amdgpu-dp2.0-mst Lyude Paul
2021-10-28  0:26 ` [Intel-gfx] " Lyude Paul
2021-10-29 10:23 ` Jani Nikula [this message]
2021-10-29 10:23   ` Jani Nikula
2021-10-29 10:35   ` Jani Nikula
2021-10-29 10:35     ` [Intel-gfx] " Jani Nikula
2021-10-29 17:20     ` Lyude Paul
2021-10-29 17:20       ` [Intel-gfx] " Lyude Paul
  -- strict thread matches above, loose matches on Subject: below --
2021-10-26  1:30 Lyude Paul
2021-10-27  7:18 ` Maxime Ripard
2021-10-27 16:43   ` Lyude Paul

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=87sfwkdt8w.fsf@intel.com \
    --to=jani.nikula@linux.intel.com \
    --cc=airlied@gmail.com \
    --cc=daniel.vetter@ffwll.ch \
    --cc=dim-tools@lists.freedesktop.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=joonas.lahtinen@linux.intel.com \
    --cc=lyude@redhat.com \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=mripard@kernel.org \
    --cc=rodrigo.vivi@intel.com \
    --cc=sean@poorly.run \
    /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.