[PATCH 03/36] drm/omap: add crtc background property

[Laurent Pinchart]

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