Re: [Intel-gfx] [PATCH 0/5] Update VSC SDP / HDR Metadata SDP states on pipe updates.

2019-10-25 Thread Ville Syrjälä
On Thu, Oct 24, 2019 at 09:24:18PM +0300, Gwan-gyeong Mun wrote:
> It calls intel_dp_vsc_enable() and intel_dp_hdr_metadata_enable() on pipe
> updates to make sure that we enable sending of VSC SDP and HDR Metadata
> Infoframe SDP packet (when applicable) on fastsets.

I think we first need to move the stuff into the crtc state and add
real precompute+readout+state check. Then we could start thinking about
optimizing things for fastsets.

> In order to set an enabled state of VSC SDP and HDR Metadata Infoframe SDP,
> It adds intel_enable_infoframe() function to handle enabling/disabling of
> each Video DIP. And it add intel_infoframe_enabled() function to get an
> enabled state of a specific infoframe.
> 
> Gwan-gyeong Mun (5):
>   drm/i915: Add whether or not to enable an each of Video DIP
>   drm/i915: Add checking a specific Video DIP is enabled or not
>   drm/i915/dp: Stop sending of VSC SDP when it is not needed
>   drm/i915/dp: Stop sending of HDR Metadata Infoframe when it is not
> needed
>   drm/i915/dp: Call dp_vsc_enable() / dp_hdr_metata_enable() on pipe
> updates
> 
>  drivers/gpu/drm/i915/display/intel_ddi.c  |  2 +
>  .../drm/i915/display/intel_display_types.h|  4 ++
>  drivers/gpu/drm/i915/display/intel_dp.c   | 20 ++-
>  drivers/gpu/drm/i915/display/intel_hdmi.c | 56 +++
>  drivers/gpu/drm/i915/display/intel_hdmi.h |  6 ++
>  5 files changed, 86 insertions(+), 2 deletions(-)
> 
> -- 
> 2.23.0

-- 
Ville Syrjälä
Intel
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

[Intel-gfx] [PATCH 0/5] Update VSC SDP / HDR Metadata SDP states on pipe updates.

2019-10-24 Thread Gwan-gyeong Mun
It calls intel_dp_vsc_enable() and intel_dp_hdr_metadata_enable() on pipe
updates to make sure that we enable sending of VSC SDP and HDR Metadata
Infoframe SDP packet (when applicable) on fastsets.
In order to set an enabled state of VSC SDP and HDR Metadata Infoframe SDP,
It adds intel_enable_infoframe() function to handle enabling/disabling of
each Video DIP. And it add intel_infoframe_enabled() function to get an
enabled state of a specific infoframe.

Gwan-gyeong Mun (5):
  drm/i915: Add whether or not to enable an each of Video DIP
  drm/i915: Add checking a specific Video DIP is enabled or not
  drm/i915/dp: Stop sending of VSC SDP when it is not needed
  drm/i915/dp: Stop sending of HDR Metadata Infoframe when it is not
needed
  drm/i915/dp: Call dp_vsc_enable() / dp_hdr_metata_enable() on pipe
updates

 drivers/gpu/drm/i915/display/intel_ddi.c  |  2 +
 .../drm/i915/display/intel_display_types.h|  4 ++
 drivers/gpu/drm/i915/display/intel_dp.c   | 20 ++-
 drivers/gpu/drm/i915/display/intel_hdmi.c | 56 +++
 drivers/gpu/drm/i915/display/intel_hdmi.h |  6 ++
 5 files changed, 86 insertions(+), 2 deletions(-)

-- 
2.23.0

___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx