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
