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

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

 



On 2020-03-30 9:02 a.m., Simon Ser wrote:
On Monday, March 30, 2020 2:59 PM, Kazlauskas, Nicholas <nicholas.kazlauskas@xxxxxxx> 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@xxxxxxx>

Regards,
Nicholas Kazlauskas
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx



[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux