[PATCH 2/3] drm/vc4: Check if plane requires background fill

[Eric Anholt]

Ville Syrjälä <ville.syrjala at linux.intel.com> writes:

> On Tue, Mar 06, 2018 at 02:48:38AM +0100, Stefan Schake wrote:
>> Considering a single plane only, we have to enable background color
>> when the plane has an alpha format and could be blending from the
>> background or when it doesn't cover the entire screen.
>> 
>> Signed-off-by: Stefan Schake <stschake at gmail.com>
>> ---
>>  drivers/gpu/drm/vc4/vc4_drv.h   |  6 ++++++
>>  drivers/gpu/drm/vc4/vc4_plane.c | 15 ++++++++++++++-
>>  2 files changed, 20 insertions(+), 1 deletion(-)
>> 
>> diff --git a/drivers/gpu/drm/vc4/vc4_drv.h b/drivers/gpu/drm/vc4/vc4_drv.h
>> index fefa166..7cc6390 100644
>> --- a/drivers/gpu/drm/vc4/vc4_drv.h
>> +++ b/drivers/gpu/drm/vc4/vc4_drv.h
>> @@ -302,6 +302,12 @@ struct vc4_hvs {
>>  
>>  struct vc4_plane {
>>  	struct drm_plane base;
>> +
>> +	/* Set when the plane has per-pixel alpha content or does not cover
>> +	 * the entire screen. This is a hint to the CRTC that it might need
>> +	 * to enable background color fill.
>> +	 */
>> +	bool needs_bg_fill;
>
> Looks to me like that should really be a bitmask (or something similar)
> in the crtc state.

Why?

In particular, VC4 really doesn't have a fixed number of planes, and the
fact that we're exposing just a handful so far is probably a bug.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20180306/6dd96db6/attachment-0001.sig>