From mboxrd@z Thu Jan 1 00:00:00 1970 From: maxime.ripard@free-electrons.com (Maxime Ripard) Date: Tue, 9 Jan 2018 14:53:22 +0100 Subject: [PATCH 06/19] drm/blend: Add a generic alpha property In-Reply-To: <20180109123241.GU26573@phenom.ffwll.local> References: <5c765fc730d75cb362dc37bcdb3b3aeacc7bdb30.1515494838.git-series.maxime.ripard@free-electrons.com> <20180109123241.GU26573@phenom.ffwll.local> Message-ID: <20180109135322.7lb3snbxfcezobhz@flea> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Jan 09, 2018 at 01:32:41PM +0100, Daniel Vetter wrote: > On Tue, Jan 09, 2018 at 11:56:25AM +0100, Maxime Ripard wrote: > > Some drivers duplicate the logic to create a property to store a per-plane > > alpha. > > > > Let's create a helper in order to move that to the core. > > > > Cc: Boris Brezillon > > Cc: Laurent Pinchart > > Signed-off-by: Maxime Ripard > > Do we have userspace for this? Wayland seems to be on its way to implement this, with ChromeOS using it: https://lists.freedesktop.org/archives/wayland-devel/2017-August/034741.html and more specifically: https://chromium.googlesource.com/chromium/src/+/master/third_party/wayland-protocols/unstable/alpha-compositing/alpha-compositing-unstable-v1.xml#118 > Is encoding a fixed 0-255 range really the best idea? I don't really know, is there hardware or formats where there is more than 255? Or did you mean less than that? > I know other drivers have skimped on the rules here a bit ... But at least > internally (i.e. within the drm_plane_state) we probably should restrict > ourselves to u8. And this needs real docs (i.e. the full blend equation > drivers are supposed to implement). You mean straight vs premultiplied? Maybe we should implement this as an additional property in read only depending on how the hardware behaves? Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: not available URL: