From: Daniel Vetter <daniel@ffwll.ch>
To: Manasi Navare <manasi.d.navare@intel.com>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 0/4] DRM helpers for Display Stream Compression PPS infoframes
Date: Mon, 14 May 2018 18:50:28 +0200 [thread overview]
Message-ID: <20180514165028.GI28661@phenom.ffwll.local> (raw)
In-Reply-To: <1525903103-19119-1-git-send-email-manasi.d.navare@intel.com>
On Wed, May 09, 2018 at 02:58:23PM -0700, Manasi Navare wrote:
> VESA Display Stream Compression is a specification for visually losless
> video compression over display links. The DSC standard also defines
> a picture parameter set (PPS) which encoder must communicate to decoders.
> This is done by encapsulating PPS header and payload bytes in an infoframe
> that can be sent to the display sink using secondary data packets
> as defined in DP 1.4 spec.
>
> This patch series creates a new files drm_dsc.h and drm_dsc.c
> which define all the DSC related structures and helpers that
> can be called by drivers to form DSC PPS infoframes before
> enabling Display Stream compression on eDP/DP/MIPI
Just an aside: At least here the threading for this patch series seems
off. And patchwork also didn't pick up your cover letter:
https://patchwork.freedesktop.org/series/42969/
>
> These files can be used to add more DSC spec related helpers
> common to all drivers.
>
> Gaurav K Singh (1):
> drm/dsc: Define VESA Display Stream Compression Capabilities
>
> Manasi Navare (3):
> drm/dp: Define payload size for DP SDP PPS packet
> drm/dsc: Define Display Stream Compression PPS infoframe
> drm/dsc: Add helpers for DSC picture parameter set infoframes
>
> drivers/gpu/drm/Makefile | 2 +-
> drivers/gpu/drm/drm_dsc.c | 222 ++++++++++++++++++
> include/drm/drm_dp_helper.h | 1 +
> include/drm/drm_dsc.h | 539 ++++++++++++++++++++++++++++++++++++++++++++
Please include these new files into the overall kernel-doc structure (and
make sure stuff is sufficiently documented).
Thanks, Daniel
> 4 files changed, 763 insertions(+), 1 deletion(-)
> create mode 100644 drivers/gpu/drm/drm_dsc.c
> create mode 100644 include/drm/drm_dsc.h
>
> --
> 2.7.4
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2018-05-14 16:50 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-09 21:58 [PATCH 0/4] DRM helpers for Display Stream Compression PPS infoframes Manasi Navare
2018-05-14 16:50 ` Daniel Vetter [this message]
2018-05-14 18:17 ` Manasi Navare
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=20180514165028.GI28661@phenom.ffwll.local \
--to=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=manasi.d.navare@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