On Fri, Apr 05, 2019 at 12:40:14PM +1000, Dave Airlie wrote: > On Thu, 14 Mar 2019 at 18:58, Boris Brezillon > <boris.brezillon@xxxxxxxxxxxxx> wrote: > > > > On Thu, 28 Feb 2019 15:49:03 +0100 > > Daniel Vetter <daniel.vetter@xxxxxxxx> wrote: > > > > > Hi all, > > > > > > Nothing too major, only things I did find in all my igt test extending for > > > drm lease is some corner cases around implicit planes and atomic target > > > crtcs. Review and comments very much appreciated. > > > > > > Cheers, Daniel > > > > > > Test-with: 20190228141918.26043-1-daniel.vetter@xxxxxxxx > > > > > > Daniel Vetter (7): > > > drm/leases: Drop object_id validation for negative ids > > > drm/lease: Drop recursive leads checks > > > drm/leases: Don't init to 0 in drm_master_create > > > drm/lease: Check for lessor outside of locks > > > drm/lease: Make sure implicit planes are leased > > > drm/atomic: Wire file_priv through for property changes > > > drm/atomic: -EACCESS for lease-denied crtc lookup > > > > I had a look at patches 5-7 and they seem to do the right thing, but my > > knowledge on DRM leases is quite limited, and given all the corner > > cases of legacy CRTC/plane updates, I'm not comfortable adding my R-b > > on these. > > For the series: > > Reviewed-by: Dave Airlie <airlied@xxxxxxxxxx> Finally gotten around to pulling it all in, thanks for all your reviews. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
