[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