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 lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (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 DFBB3CA5FC5 for ; Thu, 1 Oct 2026 01:01:55 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [127.0.0.1]) by lists.ozlabs.org (Postfix) with ESMTP id 4hwDBw0jDVz309L; Thu, 01 Oct 2026 11:01:44 +1000 (AEST) Authentication-Results: lists.ozlabs.org; arc=none smtp.remote-ip=213.167.242.64 ARC-Seal: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1790784498; cv=none; b=iUF7ubuc4tsGE6l1Mqf8F4k3rXpMKmUImcXTJX3bjQnJYVS8TnmLzZRe34WMhIztg0yDTiY3n7pTdymZr2P0WBAq9Y9uSKU/C20Fh6eQdKB/8SMcWTiNp40Q4dclqbIGHeVo8Ui6UCJFvIjWrNgsbeWQbqvIIPMDMfmsipF58ggHgNLoZKVy1v87KLXvtpNB411rZIxID8owZgSYKXRGo4eJOMXoo6n5TA+DgzIwQpS7nqgby5F9wUEDqJDqIqmFGIjRYz2QhWhJUbQ4l1U8exONEUYfp4Peu1rDiEbumFSlvWUryAypw4atCL3/PVpg8Ri2l9KfoxB1QtMvsIfp3A== ARC-Message-Signature: i=1; a=rsa-sha256; d=lists.ozlabs.org; s=201707; t=1790784498; c=relaxed/relaxed; bh=/C94Ajvktw2W9OEYMvowHEDIuF2bz9HTNYBffl1uOP4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=jKnVfHQbkn1Srh3JreAMdp/bw2LaYu0wYwd0H8aSG6Z4x6Tpp9D2MCC16g02PU0nrMYR48uur0iDGAaQ5ZQ4926fdlbFD1xPGCYLPfSTA/Qo07fNUT7N2xJ4OtflsM6IM9g1YBugqE1VWCw7MJQrGPN1QkWXOwA5jqzd+5ExouJr5IgOhJesQeQ1YxE5N2ub1WX/eoXeZ+u6ptLDdo/z1kTwEcTgKh+BRg7V8ktYC0W0ucxy3DubLNtyJr3rVxTmzJTMOypQalNQK0PqFxQi2aBPYg8DAcw9fRXvIMdrecPuD0LSDfM/uvacCE+QrKWPivI92lv43Svr8O4k+XQhbQ== ARC-Authentication-Results: i=1; lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.a=rsa-sha256 header.s=mail header.b=L2tfF9HC; dkim-atps=neutral; spf=pass (client-ip=213.167.242.64; helo=perceval.ideasonboard.com; envelope-from=jacopo.mondi@ideasonboard.com; receiver=lists.ozlabs.org) smtp.mailfrom=ideasonboard.com Authentication-Results: lists.ozlabs.org; dmarc=pass (p=none dis=none) header.from=ideasonboard.com Authentication-Results: lists.ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.a=rsa-sha256 header.s=mail header.b=L2tfF9HC; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=ideasonboard.com (client-ip=213.167.242.64; helo=perceval.ideasonboard.com; envelope-from=jacopo.mondi@ideasonboard.com; receiver=lists.ozlabs.org) X-Greylist: delayed 589 seconds by postgrey-1.37 at boromir; Thu, 01 Oct 2026 02:08:16 AEST Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange x25519) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4hw0MN1RFrz2xqM; Thu, 01 Oct 2026 02:08:16 +1000 (AEST) Received: from ideasonboard.com (unknown [93.65.100.155]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 34D4C34; Wed, 30 Sep 2026 17:56:26 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1790783786; bh=7+uOAdwvRA5Phs71HOWvHd2WnABdLy2EzB8eI2nyFFE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=L2tfF9HC3qurjOKRXYDMqOKASmD+Fr11uViI6pruu4IK+X2gjb+Qrc0LPbo2ClC/i pQ50KvP2bVEDVqeebyd/BBPmVqkW68rD+oEjTdYgMuso89FKaU5zYqVN2gKEIaX8Kh 049B+Gb1N/HvapsKch/NcZ4isnk/qWbGR2u6qN6Y= Date: Wed, 30 Sep 2026 17:58:14 +0200 From: Jacopo Mondi To: Mukesh Ojha Cc: Bjorn Andersson , Konrad Dybcio , Liviu Dudau , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Joel Stanley , Andrew Jeffery , Paul Cercueil , Anitha Chrisanthus , Paul Kocialkowski , Linus Walleij , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Alexey Brodkin , Laurent Pinchart , Tomi Valkeinen , Michal Simek , Daniel Scally , Jacopo Mondi , Mauro Carvalho Chehab , Eddie James , Tiffany Lin , Andrew-CT Chen , Yunfei Dong , Minghsiu Tsai , Houlong Wei , Matthias Brugger , AngeloGioacchino Del Regno , Joseph Liu , Marvin Lin , Dmitry Osipenko , Krzysztof Kozlowski , Thierry Reding , Jonathan Hunter , Srinivas Kandagatla , Arnd Bergmann , Greg Kroah-Hartman , Ge Gordon , Adrian Hunter , Ulf Hansson , Rob Herring , Saravana Kannan , Mathieu Poirier , Jaroslav Kysela , Takashi Iwai , Shengjiu Wang , Xiubo Li , Liam Girdwood , Mark Brown , Frank Li , Sascha Hauer , Peter Ujfalusi , Bard Liao , Daniel Baluta , Orson Zhai , Baolin Wang , Peter Chen , Fugang Duan , Ekansh Gupta , BST Linux Kernel Upstream Group , Fabio Estevam , Nicolin Chen , Pengutronix Kernel Team , Kai Vehmanen , Pierre-Louis Bossart , Vijendar Mukunda , Chunyan Zhang , CIX Linux Kernel Upstream Group , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-aspeed@lists.ozlabs.org, linux-arm-kernel@lists.infradead.org, linux-mips@vger.kernel.org, linux-sunxi@lists.linux.dev, linux-media@vger.kernel.org, openbmc@lists.ozlabs.org, linux-mediatek@lists.infradead.org, kernel@collabora.com, linux-tegra@vger.kernel.org, linux-mmc@vger.kernel.org, devicetree@vger.kernel.org, linux-remoteproc@vger.kernel.org, linux-staging@lists.linux.dev, linux-sound@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, imx@lists.linux.dev, sound-open-firmware@alsa-project.org Subject: Re: [PATCH 21/42] media: arm: mali-c55: Use devm_of_reserved_mem_device_init() Message-ID: References: <20260703193855.110619-1-mukesh.ojha@oss.qualcomm.com> <20260703193855.110619-22-mukesh.ojha@oss.qualcomm.com> X-Mailing-List: openbmc@lists.ozlabs.org List-Id: List-Help: List-Owner: List-Post: List-Subscribe: , , List-Unsubscribe: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20260703193855.110619-22-mukesh.ojha@oss.qualcomm.com> Hi Mukesh On Sat, Jul 04, 2026 at 01:08:34AM +0530, Mukesh Ojha wrote: > Use the devres-managed devm_of_reserved_mem_device_init() instead of > the manual of_reserved_mem_device_init()/of_reserved_mem_device_release() > pair, letting the device resource manager handle cleanup automatically. > > Signed-off-by: Mukesh Ojha Reviewed-by: Jacopo Mondi I'll collect this single patch for the Mali-C55 PR for v7.4 > --- > drivers/media/platform/arm/mali-c55/mali-c55-core.c | 12 +++--------- > 1 file changed, 3 insertions(+), 9 deletions(-) > > diff --git a/drivers/media/platform/arm/mali-c55/mali-c55-core.c b/drivers/media/platform/arm/mali-c55/mali-c55-core.c > index 94a389b3f833..afbec11f69f8 100644 > --- a/drivers/media/platform/arm/mali-c55/mali-c55-core.c > +++ b/drivers/media/platform/arm/mali-c55/mali-c55-core.c > @@ -801,14 +801,12 @@ static int mali_c55_probe(struct platform_device *pdev) > if (ret) > return dev_err_probe(dev, ret, "failed to acquire resets\n"); > > - of_reserved_mem_device_init(dev); > + devm_of_reserved_mem_device_init(dev); > vb2_dma_contig_set_max_seg_size(dev, UINT_MAX); > > ret = __mali_c55_power_on(mali_c55); > - if (ret) { > - dev_err_probe(dev, ret, "failed to power on\n"); > - goto err_release_mem; > - } > + if (ret) > + return dev_err_probe(dev, ret, "failed to power on\n"); > > ret = mali_c55_check_hwcfg(mali_c55); > if (ret) > @@ -847,9 +845,6 @@ static int mali_c55_probe(struct platform_device *pdev) > kfree(mali_c55->context.registers); > err_power_off: > __mali_c55_power_off(mali_c55); > -err_release_mem: > - of_reserved_mem_device_release(dev); > - > return ret; > } > > @@ -864,7 +859,6 @@ static void mali_c55_remove(struct platform_device *pdev) > } > pm_runtime_disable(&pdev->dev); > kfree(mali_c55->context.registers); > - of_reserved_mem_device_release(&pdev->dev); > } > > static const struct of_device_id mali_c55_of_match[] = { > -- > 2.53.0 >