From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Lyude Paul <lyude@redhat.com>
Cc: dim-tools@lists.freedesktop.org,
Daniel Vetter <daniel.vetter@ffwll.ch>,
intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
Ben Skeggs <bskeggs@redhat.com>
Subject: Re: [Intel-gfx] [PULL] topic/nouveau-i915-dp-helpers-and-cleanup
Date: Fri, 4 Sep 2020 09:24:34 -0400 [thread overview]
Message-ID: <20200904132434.GA586737@intel.com> (raw)
In-Reply-To: <11e59ebdea7ee4f46803a21fe9b21443d2b9c401.camel@redhat.com>
On Mon, Aug 31, 2020 at 07:38:57PM -0400, Lyude Paul wrote:
> topic/nouveau-i915-dp-helpers-and-cleanup-2020-08-31-1:
> UAPI Changes:
>
> None
>
> Cross-subsystem Changes:
>
> * Moves a bunch of miscellaneous DP code from the i915 driver into a set
> of shared DRM DP helpers
>
> Core Changes:
>
> * New DRM DP helpers (see above)
>
> Driver Changes:
>
> * Implements usage of the aforementioned DP helpers in the nouveau
> driver, along with some other various HPD related cleanup for nouveau
was this picked-up on the nouveau side already?
whenever that happens, please ping me so I can pull this to dinq.
But a reminder that it has my ack to go to drm-misc or only to nouveau
directly.
> The following changes since commit bfacb84993eb173c0ab53ca4dd6180f76f4dc176:
>
> drm: virtio: fix kconfig dependency warning (2020-08-31 08:55:02 +0200)
>
> are available in the Git repository at:
>
> git://anongit.freedesktop.org/drm/drm-misc tags/topic/nouveau-i915-dp-helpers-and-cleanup-2020-08-31-1
>
> for you to fetch changes up to 79416e97dda0118b137302575a70a14259a27d7d:
>
> drm/nouveau/kms: Start using drm_dp_read_dpcd_caps() (2020-08-31 19:10:09 -0400)
>
> ----------------------------------------------------------------
> UAPI Changes:
>
> None
>
> Cross-subsystem Changes:
>
> * Moves a bunch of miscellaneous DP code from the i915 driver into a set
> of shared DRM DP helpers
>
> Core Changes:
>
> * New DRM DP helpers (see above)
>
> Driver Changes:
>
> * Implements usage of the aforementioned DP helpers in the nouveau
> driver, along with some other various HPD related cleanup for nouveau
>
> ----------------------------------------------------------------
> Lyude Paul (20):
> drm/nouveau/kms: Fix some indenting in nouveau_dp_detect()
> drm/nouveau/kms/nv50-: Remove open-coded drm_dp_read_desc()
> drm/nouveau/kms/nv50-: Just use drm_dp_dpcd_read() in nouveau_dp.c
> drm/nouveau/kms/nv50-: Use macros for DP registers in nouveau_dp.c
> drm/nouveau/kms: Don't clear DP_MST_CTRL DPCD in nv50_mstm_new()
> drm/nouveau/kms: Search for encoders' connectors properly
> drm/nouveau/kms/nv50-: Use drm_dp_dpcd_(readb|writeb)() in nv50_sor_disable()
> drm/nouveau/kms/nv50-: Refactor and cleanup DP HPD handling
> drm/i915/dp: Extract drm_dp_read_mst_cap()
> drm/nouveau/kms: Use new drm_dp_read_mst_cap() helper for checking MST caps
> drm/nouveau/kms: Move drm_dp_cec_unset_edid() into nouveau_connector_detect()
> drm/nouveau/kms: Only use hpd_work for reprobing in HPD paths
> drm/i915/dp: Extract drm_dp_read_downstream_info()
> drm/nouveau/kms/nv50-: Use downstream DP clock limits for mode validation
> drm/i915/dp: Extract drm_dp_read_sink_count_cap()
> drm/i915/dp: Extract drm_dp_read_sink_count()
> drm/nouveau/kms/nv50-: Add support for DP_SINK_COUNT
> drm/nouveau/kms: Don't change EDID when it hasn't actually changed
> drm/i915/dp: Extract drm_dp_read_dpcd_caps()
> drm/nouveau/kms: Start using drm_dp_read_dpcd_caps()
>
> drivers/gpu/drm/drm_dp_helper.c | 187 ++++++++++++++++-
> drivers/gpu/drm/drm_dp_mst_topology.c | 22 ++
> drivers/gpu/drm/i915/display/intel_dp.c | 124 +++--------
> drivers/gpu/drm/i915/display/intel_dp.h | 1 -
> drivers/gpu/drm/i915/display/intel_lspcon.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv04/dac.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv04/dfp.c | 7 +-
> drivers/gpu/drm/nouveau/dispnv04/disp.c | 24 ++-
> drivers/gpu/drm/nouveau/dispnv04/disp.h | 4 +
> drivers/gpu/drm/nouveau/dispnv04/tvnv04.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv04/tvnv17.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv50/disp.c | 305 ++++++++++++++++------------
> drivers/gpu/drm/nouveau/nouveau_connector.c | 132 +++++-------
> drivers/gpu/drm/nouveau/nouveau_connector.h | 1 +
> drivers/gpu/drm/nouveau/nouveau_display.c | 72 ++++++-
> drivers/gpu/drm/nouveau/nouveau_display.h | 3 +-
> drivers/gpu/drm/nouveau/nouveau_dp.c | 210 +++++++++++++++----
> drivers/gpu/drm/nouveau/nouveau_drm.c | 4 +-
> drivers/gpu/drm/nouveau/nouveau_drv.h | 2 +
> drivers/gpu/drm/nouveau/nouveau_encoder.h | 48 ++++-
> include/drm/drm_dp_helper.h | 15 +-
> include/drm/drm_dp_mst_helper.h | 3 +-
> 22 files changed, 779 insertions(+), 393 deletions(-)
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
WARNING: multiple messages have this Message-ID (diff)
From: Rodrigo Vivi <rodrigo.vivi@intel.com>
To: Lyude Paul <lyude@redhat.com>
Cc: dim-tools@lists.freedesktop.org,
Daniel Vetter <daniel.vetter@ffwll.ch>,
intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
Ben Skeggs <bskeggs@redhat.com>, Sean Paul <sean@poorly.run>
Subject: Re: [PULL] topic/nouveau-i915-dp-helpers-and-cleanup
Date: Fri, 4 Sep 2020 09:24:34 -0400 [thread overview]
Message-ID: <20200904132434.GA586737@intel.com> (raw)
In-Reply-To: <11e59ebdea7ee4f46803a21fe9b21443d2b9c401.camel@redhat.com>
On Mon, Aug 31, 2020 at 07:38:57PM -0400, Lyude Paul wrote:
> topic/nouveau-i915-dp-helpers-and-cleanup-2020-08-31-1:
> UAPI Changes:
>
> None
>
> Cross-subsystem Changes:
>
> * Moves a bunch of miscellaneous DP code from the i915 driver into a set
> of shared DRM DP helpers
>
> Core Changes:
>
> * New DRM DP helpers (see above)
>
> Driver Changes:
>
> * Implements usage of the aforementioned DP helpers in the nouveau
> driver, along with some other various HPD related cleanup for nouveau
was this picked-up on the nouveau side already?
whenever that happens, please ping me so I can pull this to dinq.
But a reminder that it has my ack to go to drm-misc or only to nouveau
directly.
> The following changes since commit bfacb84993eb173c0ab53ca4dd6180f76f4dc176:
>
> drm: virtio: fix kconfig dependency warning (2020-08-31 08:55:02 +0200)
>
> are available in the Git repository at:
>
> git://anongit.freedesktop.org/drm/drm-misc tags/topic/nouveau-i915-dp-helpers-and-cleanup-2020-08-31-1
>
> for you to fetch changes up to 79416e97dda0118b137302575a70a14259a27d7d:
>
> drm/nouveau/kms: Start using drm_dp_read_dpcd_caps() (2020-08-31 19:10:09 -0400)
>
> ----------------------------------------------------------------
> UAPI Changes:
>
> None
>
> Cross-subsystem Changes:
>
> * Moves a bunch of miscellaneous DP code from the i915 driver into a set
> of shared DRM DP helpers
>
> Core Changes:
>
> * New DRM DP helpers (see above)
>
> Driver Changes:
>
> * Implements usage of the aforementioned DP helpers in the nouveau
> driver, along with some other various HPD related cleanup for nouveau
>
> ----------------------------------------------------------------
> Lyude Paul (20):
> drm/nouveau/kms: Fix some indenting in nouveau_dp_detect()
> drm/nouveau/kms/nv50-: Remove open-coded drm_dp_read_desc()
> drm/nouveau/kms/nv50-: Just use drm_dp_dpcd_read() in nouveau_dp.c
> drm/nouveau/kms/nv50-: Use macros for DP registers in nouveau_dp.c
> drm/nouveau/kms: Don't clear DP_MST_CTRL DPCD in nv50_mstm_new()
> drm/nouveau/kms: Search for encoders' connectors properly
> drm/nouveau/kms/nv50-: Use drm_dp_dpcd_(readb|writeb)() in nv50_sor_disable()
> drm/nouveau/kms/nv50-: Refactor and cleanup DP HPD handling
> drm/i915/dp: Extract drm_dp_read_mst_cap()
> drm/nouveau/kms: Use new drm_dp_read_mst_cap() helper for checking MST caps
> drm/nouveau/kms: Move drm_dp_cec_unset_edid() into nouveau_connector_detect()
> drm/nouveau/kms: Only use hpd_work for reprobing in HPD paths
> drm/i915/dp: Extract drm_dp_read_downstream_info()
> drm/nouveau/kms/nv50-: Use downstream DP clock limits for mode validation
> drm/i915/dp: Extract drm_dp_read_sink_count_cap()
> drm/i915/dp: Extract drm_dp_read_sink_count()
> drm/nouveau/kms/nv50-: Add support for DP_SINK_COUNT
> drm/nouveau/kms: Don't change EDID when it hasn't actually changed
> drm/i915/dp: Extract drm_dp_read_dpcd_caps()
> drm/nouveau/kms: Start using drm_dp_read_dpcd_caps()
>
> drivers/gpu/drm/drm_dp_helper.c | 187 ++++++++++++++++-
> drivers/gpu/drm/drm_dp_mst_topology.c | 22 ++
> drivers/gpu/drm/i915/display/intel_dp.c | 124 +++--------
> drivers/gpu/drm/i915/display/intel_dp.h | 1 -
> drivers/gpu/drm/i915/display/intel_lspcon.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv04/dac.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv04/dfp.c | 7 +-
> drivers/gpu/drm/nouveau/dispnv04/disp.c | 24 ++-
> drivers/gpu/drm/nouveau/dispnv04/disp.h | 4 +
> drivers/gpu/drm/nouveau/dispnv04/tvnv04.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv04/tvnv17.c | 2 +-
> drivers/gpu/drm/nouveau/dispnv50/disp.c | 305 ++++++++++++++++------------
> drivers/gpu/drm/nouveau/nouveau_connector.c | 132 +++++-------
> drivers/gpu/drm/nouveau/nouveau_connector.h | 1 +
> drivers/gpu/drm/nouveau/nouveau_display.c | 72 ++++++-
> drivers/gpu/drm/nouveau/nouveau_display.h | 3 +-
> drivers/gpu/drm/nouveau/nouveau_dp.c | 210 +++++++++++++++----
> drivers/gpu/drm/nouveau/nouveau_drm.c | 4 +-
> drivers/gpu/drm/nouveau/nouveau_drv.h | 2 +
> drivers/gpu/drm/nouveau/nouveau_encoder.h | 48 ++++-
> include/drm/drm_dp_helper.h | 15 +-
> include/drm/drm_dp_mst_helper.h | 3 +-
> 22 files changed, 779 insertions(+), 393 deletions(-)
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-09-04 13:23 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-31 23:38 [Intel-gfx] [PULL] topic/nouveau-i915-dp-helpers-and-cleanup Lyude Paul
2020-08-31 23:38 ` Lyude Paul
2020-09-04 13:24 ` Rodrigo Vivi [this message]
2020-09-04 13:24 ` Rodrigo Vivi
2020-09-04 18:12 ` [Intel-gfx] " Lyude Paul
2020-09-04 18:12 ` 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=20200904132434.GA586737@intel.com \
--to=rodrigo.vivi@intel.com \
--cc=bskeggs@redhat.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=lyude@redhat.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.