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 :-) -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
