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
next prev parent 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: linkBe 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.