From: "Hogander, Jouni" <jouni.hogander@intel.com>
To: "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org>
Cc: "Vivi, Rodrigo" <rodrigo.vivi@intel.com>
Subject: Re: [PATCH v2 0/7] VSC SDP rework
Date: Fri, 22 Dec 2023 06:34:45 +0000 [thread overview]
Message-ID: <eecbcb79129f861ce6041b73b3a3262216155ec4.camel@intel.com> (raw)
In-Reply-To: <20231220103609.1384523-1-jouni.hogander@intel.com>
Hello,
These are now pushed to drm-intel-next. There are BAT and IGT results
available in patchwork:
https://patchwork.freedesktop.org/series/127803/
Due to problems with our CI result mails were not sent and not all
checks were run on version 2. I checked IGT failures and they are not
related to my patches. Also IGT was passing on version 1. Rest of the
checks were run on version 2. Changes I made in version 2. were minor
so I decided trust on results over version 1. and merge this set.
Thank you Rodrigo, Mika and Shawn for your help with reviewing and
testing my patches.
BR,
Jouni Högander
On Wed, 2023-12-20 at 12:36 +0200, Jouni Högander wrote:
> Current VSC SDP configuration code is broken:
>
> 1. Driver configured header and data is not used at all
> 2. Colorimetry data is not added into VSC SDP if PSR1 is used
>
> This patch set fixes these problems and makes VSC SDP configuration a
> bit more simpler by unifying PSR and non-PSR cases.
>
> v2: s/-colorimetry/without colorimetry/
>
> Jouni Högander (7):
> drm/i915/display: Remove intel_crtc_state->psr_vsc
> drm/i915/display: Move colorimetry_support from intel_psr to
> intel_dp
> drm/i915/display: Unify VSC SPD preparation
> drm/i915/display: Fix vsc_sdp computation
> drm/i915/display: Ignore only psr specific part of vsc sdp
> drm/i915/display: Read PSR configuration before VSC SDP
> drm/i915/display: Take care of VSC select field in video dip ctl
> register
>
> drivers/gpu/drm/i915/display/intel_ddi.c | 4 +-
> drivers/gpu/drm/i915/display/intel_display.c | 9 +-
> .../drm/i915/display/intel_display_types.h | 4 +-
> drivers/gpu/drm/i915/display/intel_dp.c | 97 ++++++-----------
> --
> drivers/gpu/drm/i915/display/intel_dp.h | 7 --
> drivers/gpu/drm/i915/display/intel_hdmi.c | 8 +-
> drivers/gpu/drm/i915/display/intel_psr.c | 11 +--
> 7 files changed, 44 insertions(+), 96 deletions(-)
>
prev parent reply other threads:[~2023-12-22 6:34 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-20 10:36 [PATCH v2 0/7] VSC SDP rework Jouni Högander
2023-12-20 10:36 ` [PATCH v2 1/7] drm/i915/display: Remove intel_crtc_state->psr_vsc Jouni Högander
2023-12-20 10:36 ` [PATCH v2 2/7] drm/i915/display: Move colorimetry_support from intel_psr to intel_dp Jouni Högander
2023-12-20 10:36 ` [PATCH v2 3/7] drm/i915/display: Unify VSC SPD preparation Jouni Högander
2023-12-20 10:36 ` [PATCH v2 4/7] drm/i915/display: Fix vsc_sdp computation Jouni Högander
2023-12-20 10:36 ` [PATCH v2 5/7] drm/i915/display: Ignore only psr specific part of vsc sdp Jouni Högander
2023-12-20 10:36 ` [PATCH v2 6/7] drm/i915/display: Read PSR configuration before VSC SDP Jouni Högander
2023-12-20 10:36 ` [PATCH v2 7/7] drm/i915/display: Take care of VSC select field in video dip ctl register Jouni Högander
2023-12-22 6:34 ` Hogander, Jouni [this message]
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=eecbcb79129f861ce6041b73b3a3262216155ec4.camel@intel.com \
--to=jouni.hogander@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=rodrigo.vivi@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox