From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Tomi Valkeinen <tomi.valkeinen@ti.com>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 03/36] drm/omap: add crtc background property
Date: Wed, 30 Nov 2016 15:34:58 +0200 [thread overview]
Message-ID: <2108677.Uzi5NIM7cU@avalon> (raw)
In-Reply-To: <42d2a0c5-6a55-7861-dce3-eab3acdcf3d0@ti.com>
On Wednesday 30 Nov 2016 14:51:26 Tomi Valkeinen wrote:
> On 30/11/16 13:25, Laurent Pinchart wrote:
> > Hi Tomi,
> >
> > Thank you for the patch.
> >
> > On Wednesday 30 Nov 2016 13:17:05 Tomi Valkeinen wrote:
> >> Add DRM property for crtc background color property. Background
> >> color is shown on areas where there are no planes.
> >
> > The background property is useful for the rcar-du driver as well, and I
> > assume for a bunch of other display engines too. Could you please
> > standardize it ? And for the usual bikeshedding required in those
> > circumstances, should it be called background-color or bgcolor instead ?
>
> The possible color values for background color property depend on the
> HW. The current OMAP DSS has 8 bits per component, but other sizes
> should be supported too.
>
> I'm not sure how to implement that in a generic way... Always use 16
> bits per component for the property?
That's one option. Another one would be to decide that 8 bit per component is
enough. The value of the background colour property could also possibly be
device-dependent, for instance if a device specifies it in YUV.
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2016-11-30 13:34 UTC|newest]
Thread overview: 51+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-11-30 11:17 [PATCH 00/36] drm/omap: miscallaneous improvements Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 01/36] drm/omap: work-around for errata i886 Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 02/36] drm/omap: subclass drm_crtc_state Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 03/36] drm/omap: add crtc background property Tomi Valkeinen
2016-11-30 11:25 ` Laurent Pinchart
2016-11-30 12:51 ` Tomi Valkeinen
2016-11-30 13:34 ` Laurent Pinchart [this message]
2016-11-30 13:53 ` Ville Syrjälä
2016-11-30 13:56 ` Laurent Pinchart
2016-11-30 14:40 ` Ville Syrjälä
2016-11-30 14:46 ` Laurent Pinchart
2016-11-30 14:50 ` Ville Syrjälä
2016-11-30 13:32 ` Daniel Vetter
2016-12-09 11:03 ` Tomi Valkeinen
2016-12-09 13:41 ` Daniel Vetter
2016-11-30 11:17 ` [PATCH 04/36] drm/omap: remove divider constraint from hsdiv Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 05/36] drm/omap: decrease min width & height Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 06/36] drm/omap: improve DPI clock selection on DRA7xx Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 07/36] drm/omap: Init fbdev emulation only when we have displays Tomi Valkeinen
2016-11-30 13:46 ` Daniel Vetter
2016-12-09 11:13 ` Tomi Valkeinen
2016-12-09 14:09 ` Daniel Vetter
2016-11-30 11:17 ` [PATCH 08/36] drm/omap: Add support for render nodes Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 09/36] drm/omap: fix HDMI sync polarities Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 10/36] drm/omap: add omapdss_of_get_endpoint() helper Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 11/36] drm/omap: add omapdss-base.ko Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 12/36] drm/omap: move dss_initialized to omapdss-base Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 13/36] drm/omap: output: use dev_err instead of DSSERR Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 14/36] drm/omap: display: don't use dsi_get_pixel_size() Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 15/36] drm/omap: move display, dss-of, output to omapdss-base Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 16/36] drm/omap: move dispc related dss-feat funcs to dispc Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 17/36] drm/omap: add dispc_ops Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 18/36] drm/omap: fill dispc_ops Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 19/36] drm/omap: use dispc_ops Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 20/36] drm/omap: remove all EXPORT_SYMBOLs from dispc.c Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 21/36] drm/omap: remove unused dispc_wb_enable & dispc_wb_is_enabled Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 22/36] drm/omap: fix replication logic Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 23/36] drm/omap: dss: Functions to check components in the display/output list Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 24/36] drm/omap: dss: Support for detecting display stack readiness Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 25/36] drm/omap: Use omapdss_stack_is_ready() to check that the display stack is up Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 26/36] drm/omap: fix plane update warning when crtc is disabled Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 27/36] drm/omap: display: Add displays in sorted order to the panel_list Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 28/36] drm/omap: Create fbdev emulation only for the first DRM connector Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 29/36] drm/omap: fix max fb width Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 30/36] drm/omap: poll only connectors where the connect/disconnect can be checked Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 31/36] drm/omap: Support for HDMI hot plug detection Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 32/36] drm/omap: displays: connector-hdmi: Support for " Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 33/36] drm/omap: displays: encoder-tpd12s015: " Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 34/36] drm/omap: displays: panel-dpi: Support for handling backlight devices Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 35/36] drm/omap: dispc: improve debug print of display flags Tomi Valkeinen
2016-11-30 11:17 ` [PATCH 36/36] drm/omap: fix display SYNC/DE flags Tomi Valkeinen
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=2108677.Uzi5NIM7cU@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=dri-devel@lists.freedesktop.org \
--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.