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