RE: Overlay issues

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

 



[AMD Official Use Only - Approved for External Use]

Thanks for the suggestion, Simon.

Improved the patch, now in the process of getting it reviewed internally. It'll be posted for upstream thereafter.


-----Original Message-----
From: Simon Ser <contact@xxxxxxxxxxx> 
Sent: Monday, February 22, 2021 5:31 PM
To: Cornij, Nikola <Nikola.Cornij@xxxxxxx>
Cc: Alex Deucher <alexdeucher@xxxxxxxxx>; Kazlauskas, Nicholas <Nicholas.Kazlauskas@xxxxxxx>; Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Wentland, Harry <Harry.Wentland@xxxxxxx>; amd-gfx list <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>
Subject: RE: Overlay issues

On Monday, February 22nd, 2021 at 9:59 PM, Cornij, Nikola <Nikola.Cornij@xxxxxxx> wrote:

> [AMD Official Use Only - Approved for External Use]
>
> Hi Simon,
>
> Yes, I did have a chance to wrap this up, indeed :-)
>
> It turned out this and other similar setup was hitting a legit HW 
> limitation. I added a patch (please see attached) that'd fail this 
> config at validation time. The patch has been merged for upstreaming 
> at the beginning of February time-frame, not sure if it made it to the 
> public repo by now.

Excellent! I don't see the patch in agd5f/drm-next yet, but it'll probably show up soon.

Thanks for fixing this!

One small nit: if possible, add some drm_dbg_atomic() calls explaining the error right before returning -EINVAL. This is very valuable when debugging user-space applications and trying to figure out why the kernel rejects an atomic commit.
_______________________________________________
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