From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.5 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 40E32C433FE for ; Thu, 10 Dec 2020 12:20:04 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id D93F623B70 for ; Thu, 10 Dec 2020 12:20:03 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D93F623B70 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ideasonboard.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 220B36E1A5; Thu, 10 Dec 2020 12:20:03 +0000 (UTC) Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647]) by gabe.freedesktop.org (Postfix) with ESMTPS id 425FC6E1A5 for ; Thu, 10 Dec 2020 12:20:02 +0000 (UTC) Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 0614B96; Thu, 10 Dec 2020 13:19:59 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1607602800; bh=o1IfTzuQWBi4Lg6aRvcu8MSyMSn7kF0Cfohjw0AKcxk=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=f/L78g2lh/rPVOElq8tddcx18s/bIOXTagNk0MolM2AACYv6ZrJtOKHbeGvlCjnQf IQmA92bO7Oa0+Fj9byFo5h2B2nE6icNSMnz1zuTzRhiRWrKW1xjVbXev40Te6So86z uNumjC14DuOJ2T/8b5XWF0iTwQPbiuefz35bzBRY= Date: Thu, 10 Dec 2020 14:19:54 +0200 From: Laurent Pinchart To: Daniel Vetter Subject: Re: [PATCH v4 00/19] drm: managed encoder/plane/crtc allocation Message-ID: References: <20201208155451.8421-1-p.zabel@pengutronix.de> <4dbb86ffa92f7e1ab0a628a5ab09b91a265e64fc.camel@pengutronix.de> <20201209211047.GA401619@phenom.ffwll.local> <20201209211354.GB401619@phenom.ffwll.local> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20201209211354.GB401619@phenom.ffwll.local> X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kernel@pengutronix.de, dri-devel@lists.freedesktop.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Hi Daniel, On Wed, Dec 09, 2020 at 10:13:54PM +0100, Daniel Vetter wrote: > 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). I've sent my rcar-du fixes without depending on this series, to ease backporting. I'll then submit additional cleanups using the new managed allocators when these patches will hit the drm tree. There's thus no need for a feature branch on my side. -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel