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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux