[PATCH] drm/amd/display: add basic atomic check for cursor plane

Kazlauskas, Nicholas nicholas.kazlauskas at amd.com
Mon Mar 30 13:11:17 UTC 2020


On 2020-03-30 9:02 a.m., Simon Ser wrote:
> On Monday, March 30, 2020 2:59 PM, Kazlauskas, Nicholas <nicholas.kazlauskas at amd.com> wrote:
> 
>> We've been doing these checks for position before but I don't think we
>> really need them. DC should be disabling the cursor when we ask for a
>> position completely off the screen.
>>
>> I think that's better than rejecting the commit entirely at least.
> 
> I agree DC should be disabling the cursor in this case, however that's
> not yet implemented right? I think implementing this feature is
> orthogonal and should be done in a separate patch.
> 
> This patch simply copies over the cursor checks in the atomic check
> function.
> 

It's implemented on DCN but I don't remember if we're doing it on DCE.

I guess the drop can be in a separate patch.

Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas at amd.com>

Regards,
Nicholas Kazlauskas


More information about the amd-gfx mailing list