From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f169.google.com (mail-pg1-f169.google.com [209.85.215.169]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7BB703B9937 for ; Mon, 15 Jun 2026 22:31:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781562673; cv=none; b=XSV4sbuO6U2rAX6uQ4NX7FP5BBRdzX4Qz69zQ9se8lZXfJPdiSYi+BVsbGWdK9np3NNz7TPmEBKuuRdmdvnPUpia/HJT5l3mG8sK8S75RF60w5JWIF9l6kUbSF86ZKdO0zcZvdTHCpdq+Q19WKu4Ygm95Q3gxbybZf08VZUXBGw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781562673; c=relaxed/simple; bh=fd8S9xwvgM4gVaiRo4qfg6c4apMQUtakGh5a9Ly+mUY=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=gpLWxvKerChIStqtlkL3vCWnsZEWpRUnTwSLsyYZe/CC9TcFN1QHP1B0cUmtthrbwfxq5HTyPDHs5OGqRTp/z+SWIXzks2s0ksQJjsC+255+mtzIuVUYVg0O605W/1TcqXTCD417HMqHlQ3YoFfEP3z4sKAWzmZ68DMuL57B0JE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=O2+Wgh5t; arc=none smtp.client-ip=209.85.215.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="O2+Wgh5t" Received: by mail-pg1-f169.google.com with SMTP id 41be03b00d2f7-c88831f28f4so243258a12.2 for ; Mon, 15 Jun 2026 15:31:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1781562671; x=1782167471; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=g3HHVwgTTRBza2h0D/n/Uei1kjwtoCZN60paXjloDr4=; b=O2+Wgh5tQu2B9URRrskOBbWM4qpC5x0KXMo7J+3TaR+4gaMViDUDeQoUcTH5OS8tfV 0ZAv8MwimfgDEn+oq5TwsDm5xN2vtRWTnDoL44Po3nWt7XuvygUiyPEgvEIwzMFpL/An 0jC4pZV4CP+HTugEUup1puKZfu16honX0drtCjZ9HgNrJAuXPKQL2Ns0UkDAEK4GGlqE KEzY7BMhOp6N8aDlokIzUW4q5P4wIezI3O5LuMiTMbb76QP9JBlwhrPyCtx4ddfb2JyF wRERObhLkI+QrS7VIShA/9vbJrOZom7fbIskBoOguwnDeofL4LDUUdlrWbYm0aABI//K ku3A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781562671; x=1782167471; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=g3HHVwgTTRBza2h0D/n/Uei1kjwtoCZN60paXjloDr4=; b=WK5hk2KHeypCeXufTEEAcF2WXlOPjU5kmo6e16KdGlaY7j76eYG4Xy1TWswlRCt/gP lltDN6LIe1hhyWS0xcQwSTTFWO7WjJz30Rju4h3HFxVKEo8kZAD+A3ew20Lj4S3GNEP6 miq3wAHEzCi/DrWmzLBVgxUodSUW7VIl2GJg488l29qngJ+9IgHTPkdt05OGHOkCW/ZD vG/miNjQqaXsLnUa1IYUfwMyOY1UzpEM5qyGB0qD8ahBc3z6wrw1Yo3LKOGOIWo4uzrf IbPwBvfvCnbv+GkYZcVvWEs8mVqPh+gVNYlJ8Yh4OjKvLf5Eg7Iy7FZTwm3veiVgOdam Waqw== X-Forwarded-Encrypted: i=1; AFNElJ93xzvg5r7L5YaNVPXaXsxnFeqIh47F2qTuF4Or3oHUF8GATseh2KitozR+79tWR8ggv1781w60u+BkQoM=@vger.kernel.org X-Gm-Message-State: AOJu0Yx1kJV5DeZYgjDHlj/6a3lm8GWyDiTONFZ35961UERHPL+qgQ8g 0I/Kt/fpBc+rvNuwBjNETDlDrMLyTzs+JrroRWG0C5uQdswjAneybJAKOB0A3imc X-Gm-Gg: Acq92OH07j+hCRngqxLuUQvbl8mgOOVNu3aza13qTLQaXMvdhpeb4MoxMbgXzCYD/Ek MmIQrJN8GR2ilGJ2US7w6rrvXPgiB+OiEw1QdtjtpSC/B7b81+hZsHnN67tsgyT/j79lUDESwFS v31P5A8Z8GsD02s4VZQE+3gO7vLNzGX5EdfhICtNGn+VXdb4tAFrUnj//AWQP1GthC4hhdX24yD h7qRgagmY4nWnVC0AnWdJzHrYgBsyMn5LApRg+fzVHYbQIfp6Xw0NWGTxKTm0YUScHylSNshe1i s8kkjinFZVeE5W/E3R3E+YVCxugkcapX1BM0b2M2x0YlEJwwXGyqUfx1BvDvmeKwRka9lOQGmr2 VM2grJoP6C0tzqZijEoX+zFjPUFHuGhhJMUtfGpObfWji9pt6LNdrchLt1+hDnqoMcRSKzb8RIe /sHwTYc+Vo9640N+7lnd9YAp5gZBXMzioRBDiTpe6gm4sxc7g4ihPfLE2Zy+hIGzg= X-Received: by 2002:a05:6a20:72a3:b0:3b4:85db:1bdc with SMTP id adf61e73a8af0-3b7e4d50026mr919553637.34.1781562670749; Mon, 15 Jun 2026 15:31:10 -0700 (PDT) Received: from localhost.localdomain ([49.207.217.37]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c86651a0090sm9638592a12.26.2026.06.15.15.31.06 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Mon, 15 Jun 2026 15:31:10 -0700 (PDT) From: Biren Pandya To: linux-media@vger.kernel.org Cc: Dave Stevenson , Sakari Ailus , Mauro Carvalho Chehab , Manivannan Sadhasivam , Steve Longerbeam , linux-kernel@vger.kernel.org, Biren Pandya Subject: [PATCH 2/3] media: i2c: imx290: Drop manual fwnode_handle_put() via scope-based cleanup Date: Tue, 16 Jun 2026 04:00:54 +0530 Message-ID: <20260615223055.39953-3-birenpandya@gmail.com> X-Mailer: git-send-email 2.50.1 In-Reply-To: <20260615223055.39953-1-birenpandya@gmail.com> References: <20260615223055.39953-1-birenpandya@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Utilize the __free(fwnode_handle) scoped guard macro from to automate the lifecycle management of the endpoint fwnode in imx290_parse_dt(). This safely ties the release of the fwnode_handle to its compiler scope, allowing us to drop the manual fwnode_handle_put() call and removing the risk of missed puts if new early returns are added in the future. Signed-off-by: Biren Pandya --- drivers/media/i2c/imx290.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/media/i2c/imx290.c b/drivers/media/i2c/imx290.c index 21cbc81cb2ed..b0fd3e1dec2e 100644 --- a/drivers/media/i2c/imx290.c +++ b/drivers/media/i2c/imx290.c @@ -8,6 +8,7 @@ * Author: Manivannan Sadhasivam */ +#include #include #include #include @@ -1514,20 +1515,19 @@ static int imx290_parse_dt(struct imx290 *imx290) struct v4l2_fwnode_endpoint ep = { .bus_type = V4L2_MBUS_CSI2_DPHY }; - struct fwnode_handle *endpoint; int ret; s64 fq; imx290->model = of_device_get_match_data(imx290->dev); - endpoint = fwnode_graph_get_next_endpoint(dev_fwnode(imx290->dev), NULL); + struct fwnode_handle *endpoint __free(fwnode_handle) = + fwnode_graph_get_next_endpoint(dev_fwnode(imx290->dev), NULL); if (!endpoint) { dev_err(imx290->dev, "Endpoint node not found\n"); return -EINVAL; } ret = v4l2_fwnode_endpoint_alloc_parse(endpoint, &ep); - fwnode_handle_put(endpoint); if (ret == -ENXIO) { dev_err(imx290->dev, "Unsupported bus type, should be CSI2\n"); goto done; -- 2.50.1 (Apple Git-155)