All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Sharma, Shashank" <shashank.sharma@intel.com>
To: Jyri Sarha <jsarha@ti.com>, dri-devel@lists.freedesktop.org
Cc: Liviu.Dudau@arm.com, tomi.valkeinen@ti.com,
	laurent.pinchart@ideasonboard.com
Subject: Re: [PATCH RFC 0/6] drm: Add properties to control YCbCr to RGB conversion
Date: Fri, 21 Apr 2017 18:23:03 +0530	[thread overview]
Message-ID: <c5bc8231-d1e5-b74a-8958-f5d3939603c4@intel.com> (raw)
In-Reply-To: <cover.1492768073.git.jsarha@ti.com>

Regards

Shashank


On 4/21/2017 3:21 PM, Jyri Sarha wrote:
> The series adds plane specific atomic properties to control YCbCr to
> RGB conversions. My intention was to try to implement the plane
> specific (before DEGAMMA) part of the suggestion in this dri-devel
> post:
I would probably extend this series to have a bigger view. Instead of 
addressing RGB->YCBCR conversion, the actual target here should be 
blending of various framebuffers considering:
- their color spaces (Rec 601/709/2020)
- their format (YCBCR->RGB or RGB->YCBCR)
- their linear state (Linear/Non-linear) or if they are ready for Gamut 
mapping/color conversion or not.

Also, we need to have the sequence of properties, so that, it would 
match all(most of the) HWs.
I will add my comments in the upcoming patches accordingly.

- Shashank
> https://lists.freedesktop.org/archives/dri-devel/2017-March/135870.html
>
> This series may not be ready as such. At least the kernel doc parts
> should be more detailed and carefully written. The purpose is merely
> to move the discussion to a more concrete level.
>
> The series also includes drm/omap patches that implement the standard
> properties for OMAP DSS in omapdrm driver.
>
> Best regards,
> Jyri
>
> Jyri Sarha (4):
>    drm: drm_color_mgmt.h needs struct drm_crtc declaration
>    drm: Make drm_atomic_replace_property_blob_from_id() more generic
>    drm: Plane YCbCr to RGB conversion related properties
>    drm/omap: Enable ycbcr_to_rgb_properties for omapdrm planes REVISIT
>
> Tomi Valkeinen (2):
>    drm/omap: cleanup color space conversion
>    drm/omap: csc full range support
>
>   drivers/gpu/drm/drm_atomic.c          |  36 +++++++--
>   drivers/gpu/drm/drm_atomic_helper.c   |   9 +++
>   drivers/gpu/drm/drm_color_mgmt.c      | 136 +++++++++++++++++++++++++++++++-
>   drivers/gpu/drm/drm_plane.c           |  10 +++
>   drivers/gpu/drm/omapdrm/dss/dispc.c   | 141 +++++++++++++++++++++++++++++-----
>   drivers/gpu/drm/omapdrm/dss/omapdss.h |  14 ++++
>   drivers/gpu/drm/omapdrm/omap_plane.c  |  41 ++++++++++
>   include/drm/drm_color_mgmt.h          |  25 ++++++
>   include/drm/drm_plane.h               |  10 +++
>   include/uapi/drm/drm_mode.h           |  12 +++
>   10 files changed, 408 insertions(+), 26 deletions(-)
>

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  parent reply	other threads:[~2017-04-21 12:53 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-21  9:51 [PATCH RFC 0/6] drm: Add properties to control YCbCr to RGB conversion Jyri Sarha
2017-04-21  9:51 ` [PATCH RFC 1/6] drm: drm_color_mgmt.h needs struct drm_crtc declaration Jyri Sarha
2017-04-21 11:46   ` Laurent Pinchart
2017-04-21  9:51 ` [PATCH RFC 2/6] drm: Make drm_atomic_replace_property_blob_from_id() more generic Jyri Sarha
2017-04-21 11:47   ` Laurent Pinchart
2017-05-02  8:31     ` Daniel Vetter
2017-04-21  9:51 ` [PATCH RFC 3/6] drm: Plane YCbCr to RGB conversion related properties Jyri Sarha
2017-04-21 11:17   ` Ville Syrjälä
2017-04-21 12:10     ` Laurent Pinchart
2017-04-21 13:08       ` Ville Syrjälä
2017-04-21 12:58     ` Sharma, Shashank
2017-04-21 13:39     ` Jyri Sarha
2017-04-21 13:52       ` Ville Syrjälä
2017-04-21 14:53         ` Brian Starkey
2017-04-21 15:21           ` Ville Syrjälä
2017-04-21 15:34             ` Brian Starkey
2017-04-21 16:49               ` Ville Syrjälä
2017-04-24 12:58                 ` Brian Starkey
2017-04-24  9:02         ` [PATCH] RFC: Design: DRM: Blending pipeline using DRM plane properties Shashank Sharma
2017-04-24 13:21         ` [PATCH RFC 3/6] drm: Plane YCbCr to RGB conversion related properties Jyri Sarha
2017-04-24 15:13           ` Ville Syrjälä
2017-04-24 15:49             ` Jyri Sarha
2017-04-24 16:55               ` Ville Syrjälä
2017-04-26 12:56                 ` Jyri Sarha
2017-04-26 13:28                   ` Sharma, Shashank
2017-04-26 17:22                   ` Ville Syrjälä
2017-05-02  8:33   ` Daniel Vetter
2017-05-02  9:29     ` Jyri Sarha
2017-04-21  9:51 ` [PATCH RFC 4/6] drm/omap: cleanup color space conversion Jyri Sarha
2017-04-21 11:53   ` Laurent Pinchart
2017-04-21  9:51 ` [PATCH RFC 5/6] drm/omap: csc full range support Jyri Sarha
2017-04-21 11:55   ` Laurent Pinchart
2017-04-21  9:51 ` [PATCH RFC 6/6] drm/omap: Enable ycbcr_to_rgb_properties for omapdrm planes REVISIT Jyri Sarha
2017-04-21 12:53 ` Sharma, Shashank [this message]
2017-04-21 13:50 ` [PATCH RFC 0/6] drm: Add properties to control YCbCr to RGB conversion Liviu Dudau
2017-04-24  9:18 ` ✓ Fi.CI.BAT: success for RFC: Design: DRM: Blending pipeline using DRM plane properties Patchwork

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=c5bc8231-d1e5-b74a-8958-f5d3939603c4@intel.com \
    --to=shashank.sharma@intel.com \
    --cc=Liviu.Dudau@arm.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jsarha@ti.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=tomi.valkeinen@ti.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.