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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 0AFE7CCA464 for ; Fri, 16 Oct 2026 11:46:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type: Content-Transfer-Encoding:MIME-Version:References:In-Reply-To:Message-ID:Date :Subject:Cc:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=0xf9aqDEB2ABb/PUaNcWQ+D8Pgss+XS8jNScelABQvc=; b=GlLXgqU/+82TDhyICrvLr6VL+g X0NBbD4PNIjxizX1xftrrHmwTmlp+FzL1wLJIrn3Ssw1v+3uakfSJ7rRKCyJd9q7MvuuT7Za7wXgg eDp0OMuf6VUZwEM9TGLVkD1EQJ+rUEvy/8qj1Y6VgcBBURfiOM9dyw3kGVCkaRZjhRn0XF26ozh2/ NJPv5PUBkf0t53Wi5M7xNIFJGW0RkRyD+VLE1wWBzEDNuKgxpeKLeNaEXGL7uqYqZLyqznmlnK6Y4 m58ITCV/Dr+g8FpevRTwyQqee+1RMIy4T37yLhbuZ0fMRsIRFz18ZoNl1V3AckCE0m9ZksK8RN3FC uUqTkkRA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xHgNx-00000002nHz-3QJa; Fri, 16 Oct 2026 11:46:05 +0000 Received: from gloria.sntech.de ([185.11.138.130]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xHgNv-00000002nHJ-2KCG; Fri, 16 Oct 2026 11:46:04 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sntech.de; s=gloria202408; h=Content-Type:Content-Transfer-Encoding:MIME-Version: References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From:Reply-To; bh=0xf9aqDEB2ABb/PUaNcWQ+D8Pgss+XS8jNScelABQvc=; b=wwlo6iHZCJAUeOFDpzHHQ37CHT utNFt53Nbnz0vB2LQNSwuV/fzBTLIPmlwPr27mVmZtCgNk7RFhO0D04ecYNRAm4kzay40ZdjONpWT dmczjPfCoRQyFOgQOAjBKBvLS1rfmbGDeNBoCKDdXO1gpKgJi37q10cwwDM+PbdTpCUu2t0ZBc88d UFEVQH2YhPy6Bh5NiHTKoYk2HYdfMpLozrAewFwbOIrP1AOih195/PUXsiX/MJxkQWV06igSTa0o5 eVkaYULwETuyawXTNcJx8nErl6dbn1xtJBjy+kmj97tO9cA+gKq7URZWaEPjtjP6UV/Z2uvowyXt/ KtMysznQ==; From: Heiko =?UTF-8?B?U3TDvGJuZXI=?= To: Luca Ceresoli , Sandy Huang , Andy Yan , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Cristian Ciocaltea , Huseyin BIYIK , Johan Jonker Cc: kernel@collabora.com, dri-devel@lists.freedesktop.org, linux-rockchip@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/2] drm: devm_drm_bridge_alloc() conversion and cleanup Date: Fri, 16 Oct 2026 13:45:22 +0200 Message-ID: <2372538.iZASKD2KPV@diego> In-Reply-To: References: <20261001-bridge-alloc-conv-v1-0-f0a1fe040a86@collabora.com> <179213353665.3058528.14437637653539699061.b4-ty@b4> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20261016_044603_617762_BDA8B221 X-CRM114-Status: GOOD ( 15.25 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Johan, Am Freitag, 16. Oktober 2026, 10:11:19 Mitteleurop=C3=A4ische Sommerzeit sc= hrieb Johan Jonker: > If any decency or Code of Conduct left in the DRM group then also credit = the original author. > Also remove driver_private. after letting that stew in my mind over a lunch break: =2D yes, your patch was earlier =2D yes, I should've gotten around to it earlier - I am sorry about that =2D so yes, this got mishandled Nevertheless, this in no way justifies attacking the "drm-group", which consists of a big number of individuals (most of which had no contact with either variant of the patch at all) on a personal level :-( . The change itself of moving to devm_drm_bridge_alloc, also is somewhat straight-forward that two different people could easily just create the same change separately, as we saw here. Heiko > =3D=3D=3D >=20 > [PATCH v3] drm/rockchip: rk3066_hdmi: convert to devm_drm_bridge_alloc() = API > https://lore.kernel.org/linux-rockchip/20260904-drm-v3-1-5b36532a3b72@gma= il.com/ =20 >=20 > [PATCH v2] drm/rockchip: rk3066_hdmi: convert to devm_drm_bridge_alloc() = API > https://lore.kernel.org/linux-rockchip/20260824-drm-v2-1-7f9273c6020b@gma= il.com/ >=20 > https://lore.kernel.org/linux-rockchip/dea8a038-577b-45da-bff3-703e9099b7= e1@gmail.com/ > [PATCH v1] drm/rockchip: rk3066_hdmi: convert to devm_drm_bridge_alloc() = API >=20 > On 10/16/26 08:52, Luca Ceresoli wrote: > >=20 > > On Thu, 01 Oct 2026 11:25:10 +0300, Cristian Ciocaltea wrote: > >> Bridge drivers are expected to allocate their private data via > >> devm_drm_bridge_alloc(), which also initializes the embedded struct > >> drm_bridge, including its funcs pointer. Most drivers were converted = by > >> commit 9c399719cfb9 ("drm: convert many bridge drivers from > >> devm_kzalloc() to devm_drm_bridge_alloc() API"), but a couple of > >> leftovers remain. > >> > >> [...] > >=20 > > Applied, thanks! > >=20 > > [1/2] drm/rockchip: rk3066_hdmi: Convert to devm_drm_bridge_alloc() API > > commit: 5c0cdbee4fa4a621f871d6dc825e759db08a1a4a > > [2/2] drm/bridge: lontium-lt8713sx: Drop redundant bridge funcs assignm= ent > > commit: 382c1e8ba409d1611cd256dccebdfa828c832bb1 > >=20 > > Best regards, >=20 >=20