[AMD Official Use Only - General]
Thanks Chandan for the comment, will revise the patch.
DZ
From: VURDIGERENATARAJ, CHANDAN <chandan.vurdigerenata...@amd.com>
Sent: Monday, May 9, 2022 4:26 AM
To: Zhang, Dingchen (David) <dingchen.zh...@amd.com>;
amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>
Cc: Wang, Chao-kai (Stylon) <stylon.w...@amd.com>; Li, Sun peng (Leo)
<sunpeng...@amd.com>; Wentland, Harry <harry.wentl...@amd.com>; Zhuo, Qingqing
(Lillian) <qingqing.z...@amd.com>; Siqueira, Rodrigo
<rodrigo.sique...@amd.com>; Li, Roman <roman...@amd.com>; Chiu, Solomon
<solomon.c...@amd.com>; Zuo, Jerry <jerry....@amd.com>; Pillai, Aurabindo
<aurabindo.pil...@amd.com>; Lin, Wayne <wayne....@amd.com>; Lakha, Bhawanpreet
<bhawanpreet.la...@amd.com>; Gutierrez, Agustin <agustin.gutier...@amd.com>;
Kotarac, Pavle <pavle.kota...@amd.com>
Subject: RE: [PATCH 05/17] drm/amd/display: revise Start/End SDP data
Hi,
Why not set "vsc_packet_rev2" when "stream->link->psr_settings.psr_version ==
DC_PSR_VERSION_1"?
This would be safer for future PSR versions also.
BR,
Chandan V N
>[why & how]
>We need to implement the VSC packet rev4 that is required by PSRSU.
>
>Follow the eDP 1.5 spec pg. 257
>
>Signed-off-by: David Zhang <dingchen.zh...@amd.com>
>---
> .../display/modules/info_packet/info_packet.c | 29 +++++++++++++++++--
> 1 file changed, 27 insertions(+), 2 deletions(-)
>
>diff --git a/drivers/gpu/drm/amd/display/modules/info_packet/info_packet.c
>b/drivers/gpu/drm/amd/display/modules/info_packet/info_packet.c
>index 79bc207415bc..c7d1b48c0478 100644
>--- a/drivers/gpu/drm/amd/display/modules/info_packet/info_packet.c
>+++ b/drivers/gpu/drm/amd/display/modules/info_packet/info_packet.c
>@@ -145,8 +145,10 @@ void mod_build_vsc_infopacket(const struct
>dc_stream_state *stream,
> stereo3dSupport = true;
> }
>
>- /*VSC packet set to 2 when DP revision >= 1.2*/
>- if (stream->link->psr_settings.psr_version !=
>DC_PSR_VERSION_UNSUPPORTED)
>+ /*VSC packet set to 4 for PSR-SU, or 2 for all other PSR support cases*/
>+ if (stream->link->psr_settings.psr_version == DC_PSR_VERSION_SU_1)
>+ vsc_packet_revision = vsc_packet_rev4;
>+ else if (stream->link->psr_settings.psr_version !=
>+DC_PSR_VERSION_UNSUPPORTED)
> vsc_packet_revision = vsc_packet_rev2;
>
> /* Update to revision 5 for extended colorimetry support */ @@ -159,6
> +161,29 @@ void mod_build_vsc_infopacket(const struct >dc_stream_state
> *stream,
> if (vsc_packet_revision == vsc_packet_undefined)
> return;
>
>+ if (vsc_packet_revision == vsc_packet_rev4) {
>+ /* Secondary-data Packet ID = 0*/
>+ info_packet->hb0 = 0x00;
>+ /* 07h - Packet Type Value indicating Video
>+ * Stream Configuration packet
>+ */
>+ info_packet->hb1 = 0x07;
>+ /* 04h = VSC SDP supporting 3D stereo + PSR/PSR2 + Y-coordinate
>+ * (applies to eDP v1.4 or higher).
>+ */
>+ info_packet->hb2 = 0x04;
>+ /* 0Eh = VSC SDP supporting 3D stereo + PSR2
>+ * (HB2 = 04h), with Y-coordinate of first scan
>+ * line of the SU region
>+ */
>+ info_packet->hb3 = 0x0E;
>+
>+ for (i = 0; i < 28; i++)
>+ info_packet->sb[i] = 0;
>+
>+ info_packet->valid = true;
>+ }
>+
> if (vsc_packet_revision == vsc_packet_rev2) {
> /* Secondary-data Packet ID = 0*/
> info_packet->hb0 = 0x00;
>--
>2.25.1
