On Wed, Dec 09, 2020 at 10:10:47PM +0100, Daniel Vetter wrote: > On Tue, Dec 08, 2020 at 04:59:16PM +0100, Philipp Zabel wrote: > > On Tue, 2020-12-08 at 16:54 +0100, Philipp Zabel wrote: > > > Hi, > > > > > > this is an update of the drmm_(simple_)encoder_alloc(), > > > drmm_universal_plane_alloc(), and drmm_crtc_alloc_with_plane() > > > functions in v3 [1] together with the imx-drm managed allocation > > > conversion from [2] as an example usage. > > > a bit. > > ^^^^^^ > > this is a left-over of ", reordered an squashed a bit." before I decided > > to move it into the list of changes below. > > > > > > > > Changes since v3: > > [...] > > > - Reorder and squash the imx-drm managed allocation conversion patches > > > to use the new functions directly. > > imx parts all look good. One thing I spotted is that you could use > devm_drm_dev_alloc instead of drm_dev_alloc, at least my tree here doesn't > have that one yet. Feel free to add a-b: me on top of the imx patches too, > but probably not worth much :-) Oh also for merging, ack for merging through whatever tree you feel like. Since 5.11 merge window is done already if you go through imx please make sure to send a feature pull soon after -rc1 so it's not holding up Laurent. Or coordinate with Laurent (and maybe others). -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
