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 9A477C6FD18 for ; Wed, 29 Mar 2023 16:09:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=3VsBIJ+Ro2LY1EuCK9padyVywjI7B0cIWDnqfPNvPVI=; b=SmTa/Re7cZtAPhn+diJdlqMPUH m09J4Ns9tM6EfKQGRl0zhp3OJ9sNovb5v7uGCsbt4Sr7t2gTl7rf6f0Ba0gbNPeew03rm2rdbe2i4 r8NN5pbM5gEKPsT/B+JB3KuH2RgGkrzQDmh3Hw8SP7lIMEsHiSPwZh0wF14ZcYR3Mg3DQBWU0anP3 5j+WkSCGuxJu3ka9EjqXSxPHSX+LkB9I8sr2ILmt5xgUz+iDbQtNijwOvcmOOvzzKBRGM7bUsqSFV pqjPsGejrBg9RoVSGgwZbgqXJiy46HoQHRH3S49cCqQg+8e7VPZH7Phb3zQyCplvffKjNdcQrO19+ YZU9we0g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1phYLW-0011Dp-0l; Wed, 29 Mar 2023 16:08:22 +0000 Received: from new2-smtp.messagingengine.com ([66.111.4.224]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1phYLT-0011CU-2F for linux-arm-kernel@lists.infradead.org; Wed, 29 Mar 2023 16:08:21 +0000 Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailnew.nyi.internal (Postfix) with ESMTP id D4EE8581FA9; Wed, 29 Mar 2023 12:08:18 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute6.internal (MEProxy); Wed, 29 Mar 2023 12:08:18 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h=cc :cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm3; t=1680106098; x=1680113298; bh=MY XYHXP9TA6RiIrd7ebeit6HRgTdNhlockaHZW3ryv4=; b=ercciHkF5NtsqnR3G9 2kbtpTFdZP2CT+pcXjXMhpd4PlmXkhcooAR2s7NxrWuYoFP67mCyJEPbtq402sXs gxsH13YLzW6ltHcKYW2tpJNEy29LgUOEovFwT9EONOLF+ERt1N8osO6McMdkQ95Y bdoDJMxZUON5NEH7xPxY2YeAxjYv0UNqWKFef0h6LD6c1KAltSESaWNyCKlkHOnM FsTCb8RpJ9Kvr3/yJnBA+sYB5KoHYh30TolGycHbQhYhEimBMMvTLDeePDKCS9U4 KeMS6spq/lbjTIBIlNBdFKqxVHBCvoNRlHKDQeQPVpLojkevzgtvPzxllmWPSkaR /pmg== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm2; t=1680106098; x=1680113298; bh=MYXYHXP9TA6Ri Ird7ebeit6HRgTdNhlockaHZW3ryv4=; b=CH2r4RgmXy/7IqLWVhWn8y+T/fcNk KkjYeUa3p4K4zQHAuy0gkIWkKzHgsXINbPT4Te4D6ZVupzfifr6WuQWEcA67POJv ePtGv39f38YhzPkbqq+MQeDQOE4dei56PB6zZYXjnIbgku6IPj4uyJ+ohN9r4J3Z MGHCSYd7AedbgStvEN7RIiRgZCMeICB2nxDEnEvbDljZMOKqWlZnEH7G1hj7SW96 ep0heiIR9Z5x7CGFEfwHsKl0Zb5SLwU093tDswxxyESBICTTS57qKEQfY5VdEdly i7s1PqNFZDqd88p+hfi7DoGU6nHfFVBOZKa1fRMarbj1UOai8fNZm/j0w== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvhedrvdehiedgleekucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvfevuffkfhggtggujgesghdtreertddtjeenucfhrhhomhepofgrgihi mhgvucftihhprghrugcuoehmrgigihhmvgestggvrhhnohdrthgvtghhqeenucggtffrrg htthgvrhhnpeejgeeiveevvdfgffeftdevuefhheduveejieefgeejveeuhfetkeelgfev teefffenucevlhhushhtvghrufhiiigvpedunecurfgrrhgrmhepmhgrihhlfhhrohhmpe hmrgigihhmvgestggvrhhnohdrthgvtghh X-ME-Proxy: Feedback-ID: i8771445c:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 29 Mar 2023 12:08:17 -0400 (EDT) Date: Wed, 29 Mar 2023 18:08:15 +0200 From: Maxime Ripard To: Jagan Teki Cc: Dave Stevenson , Maarten Lankhorst , Thomas Zimmermann , David Airlie , Daniel Vetter , Andrzej Hajda , Neil Armstrong , Robert Foss , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Sam Ravnborg , Rob Herring , Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org, Marek Vasut , linux-amarula Subject: Re: [PATCH v7 02/12] drm: bridge: panel: Implement drmm_of_dsi_get_bridge helper Message-ID: <20230329160815.ql6ctqoxae7n65b7@penduick> References: <20230329131615.1328366-1-jagan@amarulasolutions.com> <20230329131615.1328366-3-jagan@amarulasolutions.com> <20230329150337.ua6qz7rvupk6vizl@penduick> MIME-Version: 1.0 In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230329_090819_802789_DB184C04 X-CRM114-Status: GOOD ( 23.53 ) 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: , Content-Type: multipart/mixed; boundary="===============1379155582030793102==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============1379155582030793102== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="4c4xrdmgsh5fx3jk" Content-Disposition: inline --4c4xrdmgsh5fx3jk Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Mar 29, 2023 at 08:56:29PM +0530, Jagan Teki wrote: > On Wed, Mar 29, 2023 at 8:33=E2=80=AFPM Maxime Ripard = wrote: > > > > On Wed, Mar 29, 2023 at 06:46:08PM +0530, Jagan Teki wrote: > > > Implement a DRM-managed action helper that returns the next DSI bridge > > > in the chain. > > > > > > Unlike general bridge return helper drmm_of_get_bridge, this helper > > > uses the dsi specific panel_or_bridge helper to find the next DSI > > > device in the pipeline. > > > > > > Helper lookup a given downstream DSI device that has been added via > > > child or OF-graph port or ports node. > > > > > > Upstream DSI looks for downstream devices using drm pointer, port and > > > endpoint number. Downstream devices added via child node don't affect > > > the port and endpoint number arguments. > > > > > > Signed-off-by: Jagan Teki > > > > If you intend to use it with the sun4i driver, then don't. The sun4i > > driver is barely using drm-managed resources, so chances are all its > > resources will be freed by the time this action will run. >=20 > Any idea how to check all resources will be freed before this > DRM-managed action helper runs? I did test the panel insertion and > removal at runtime and I can see that the action helper > (drmm_drm_panel_bridge_release) is triggered at removal time. >=20 > Here is the log for it. > # insmod panel-bananapi-s070wv20-icn6211.ko > [ 11.411894] sun4i-drm display-engine: bound > 1e00000.display-frontend (ops 0xc0951eb4) > [ 11.420253] sun4i-drm display-engine: bound 1e60000.display-backend > (ops 0xc0951620) > [ 11.428128] sun4i-drm display-engine: bound 1e70000.drc (ops 0xc095115= 0) > [ 11.435574] sun4i-drm display-engine: No panel or bridge found... > RGB output disabled > [ 11.443484] sun4i-drm display-engine: bound 1c0c000.lcd-controller > (ops 0xc094fdb0) > [ 11.451264] sun4i-drm display-engine: bound 1ca0000.dsi (ops 0xc0952f0= 4) > [ 11.462479] [drm] Initialized sun4i-drm 1.0.0 20150629 for > display-engine on minor 1 > [ 11.963246] Console: switching to colour frame buffer device 100x30 > [ 12.046329] sun4i-drm display-engine: [drm] fb0: sun4i-drmdrmfb > frame buffer device > [ 12.082641] sun6i-mipi-dsi 1ca0000.dsi: Attached device s070wv20-ct16-= icn62 > # rmmod panel-bananapi-s070wv20-icn6211.ko > [ 16.306388] Console: switching to colour dummy device 80x30 > [ 16.466963] drmm_drm_panel_bridge_release: In Enable KASAN, unload the DSI controller module or unbind its device, and watch things go boom :) Maxime --4c4xrdmgsh5fx3jk Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCZCRibwAKCRDj7w1vZxhR xUC0AP9QtTmenppZSOyUQNhOSmr4OauGpP3gr8K2Kuh7RleiRQEAkbq8MRKgyzwI PKywDo9KTOF2N/FYWK0hSjkyjA5DFgw= =bdW5 -----END PGP SIGNATURE----- --4c4xrdmgsh5fx3jk-- --===============1379155582030793102== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============1379155582030793102==--