From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f176.google.com (mail-pg1-f176.google.com [209.85.215.176]) (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 464C83C455B for ; Mon, 15 Jun 2026 22:31:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.176 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781562668; cv=none; b=iZCnEJA2iDDcieHtYm4BY5mD//vyzir5RpIpWAG5ct4o15x3S0yjjcWQfZqRI57EO+SqwpxFOBoXA2p+MgATcQWlhi8r+VE8fPxBGpL0rG5965j3XAKHtEeENOWsoh3vxZPMC8+/dQbG6AQ13SjQP/9qUBncFoaJuIEB6YrWT+I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781562668; c=relaxed/simple; bh=fG/DhwJUYK2MFwrgtcaOynoaDnqY5t5HM8w1qMcdcR4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=cWj+csZVTZClGGYl7l3n/FDom77JP9ZQ2gid6XYgysMXEorhRbjho3IiqOq7E8CmvuWhwpHhUOtg6dyNhPhg595f0gk3XB3gIVVsS/YsShKGWEiYQTmjjByafGy+XFFu2QF0AR3s6T1xcEL9DcnQt5GeJaTZyoYoZrqkqpsuAIg= 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=Yeqach+M; arc=none smtp.client-ip=209.85.215.176 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="Yeqach+M" Received: by mail-pg1-f176.google.com with SMTP id 41be03b00d2f7-c858014845aso1537131a12.1 for ; Mon, 15 Jun 2026 15:31:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1781562666; x=1782167466; 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=xLMrQUpzK88XueUv2+MZgJ9sRfs9rpPqqUQZ5Nh/QZg=; b=Yeqach+Ml36AMIHzuGUD+kw6VqEJ0t6SZ3H+3nOPPmKYyUsg3y4OSOGq6DhCDHq7Vi XGut9PjtcxZWTwlGAZpWagCgXw9AsJxliyoC8O3JvkXbbdo9x97pDamH59B43S1Wvc28 +61ep9NQQE37aBtom9oRUlyvTqZKRTW0LehOnehSgqnyWEEQmo8D18ihAd4eM0Z519tZ iQfDxBCw1Vnvc3pG9XhXvTA5gQWME8apVj1DOATo7kQIBmkxiy29piaTJoS2D58ov2GO 1HIjpbksZF+8usTHKs3FcCSjiId09BCkrRFo6/3NdJGoMLzTa09LVJ6FyunL01LZhLUk WdTw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781562666; x=1782167466; 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=xLMrQUpzK88XueUv2+MZgJ9sRfs9rpPqqUQZ5Nh/QZg=; b=icDTLJm9eaMuvbZx8EnX6opT6C0Ppsd7yE9IevwFDZrB/uhr38Y1n4nl3w7tBOv0xD 7sJc+OV4MY/fg34WNnBdm/FHQxDxoS2sn7Bhis2//tMrf8BAAX8UEjUbsYLjNy2e0fXT 9iprk4KxPmfGZvvuUnR9uRL9xHjvyOuCNRfn3grErYt/jeZ1q/FKNnX40PumSHtxfV/O VB/VyogL5LqedlvXQDs80b0JFxpZH9oy2qBVuP5TMdRNgrUXQ7+QR5qhn7vNWmeCHzKr 1h/zbpWW6vyhHMpxuP+jE3rGZu0BJkyl3/sBHrmNGv85wDG+6d2d9iBbS37zK6T6Z/pJ 15XA== X-Forwarded-Encrypted: i=1; AFNElJ+cM0jnsiJqZhpA8Nryozr45+FE6kXUcC85cU6cDhzcYBJRXvKHWIWOhzAyjq7hC+G4Yt92u1ZiBfusgIk=@vger.kernel.org X-Gm-Message-State: AOJu0Yz1Wws3+X+eF/cgt/t0ZDYGKaSpsDgwr4MSTz8IHZjPwvW2PjBT pDUgPnWb+vi++KBQGC8o0HhfLSQyuPMaf+f3bm7y6eno74oRNEJ+Sijk X-Gm-Gg: Acq92OHhMLktFNhrBbiSCb5So83iKgCRJ+SLvmgUYgIn3Y7QyyFXWClYGdP7hCy/3Pw BfH/oYzc8o8kAYgKQkFlliZ0OfTs8ZdMKqc2j/wJ2mxw4ptMg8AUFS+TGuvlD2O9Qpmfl7TnSzY Jjg3nYjLql8vZ/uDu3RjGRafYvKVW4Tkfh+9tzsv8olqBnpUd2mOxYHMRw/VapoQSwFD/Um4nOB NDBOJsDLiv5Q9+X+Vngx4w/Jjj3YaHQ7rd2MkVk5obn8anvE8DAbVm04Sft0tsyfiadVdXgNNCL 0EsH5Vpew6g+bntA/+4QbFt66+wVvfEQtBIT4/k48GCUCr4TJAfoirF8XP+uDP1hw/PCjke0Fnw Ypw1uN8/D0CMZi5Xk9Kof5+vttIgf/cojjBg8kImESRAlMs/IEDNvb+EUL1A9szLSaCXEhPmcO0 ASIHnmL7XypxDP1E0Yx+pNEMH5DfXq7SEv0N64A7xx6VNcAa18uXZO X-Received: by 2002:a05:6a21:7103:b0:3a2:c9a1:2c22 with SMTP id adf61e73a8af0-3b7e225ae47mr1101624637.6.1781562666482; Mon, 15 Jun 2026 15:31:06 -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.03 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Mon, 15 Jun 2026 15:31:05 -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 1/3] media: i2c: ov5640: Drop manual fwnode_handle_put() via scope-based cleanup Date: Tue, 16 Jun 2026 04:00:53 +0530 Message-ID: <20260615223055.39953-2-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 ov5640_probe(). This eliminates the need for the manual fwnode_handle_put() call, preventing potential memory leaks in error paths and simplifying the probe routine. Signed-off-by: Biren Pandya --- drivers/media/i2c/ov5640.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/media/i2c/ov5640.c b/drivers/media/i2c/ov5640.c index ce6b0724afa1..7795f2de0f75 100644 --- a/drivers/media/i2c/ov5640.c +++ b/drivers/media/i2c/ov5640.c @@ -4,6 +4,7 @@ * Copyright (C) 2014-2017 Mentor Graphics Inc. */ +#include #include #include #include @@ -3845,7 +3846,6 @@ static int ov5640_check_chip_id(struct ov5640_dev *sensor) static int ov5640_probe(struct i2c_client *client) { struct device *dev = &client->dev; - struct fwnode_handle *endpoint; struct ov5640_dev *sensor; int ret; @@ -3870,15 +3870,14 @@ static int ov5640_probe(struct i2c_client *client) sensor->ae_target = 52; - endpoint = fwnode_graph_get_next_endpoint(dev_fwnode(&client->dev), - NULL); + struct fwnode_handle *endpoint __free(fwnode_handle) = + fwnode_graph_get_next_endpoint(dev_fwnode(&client->dev), NULL); if (!endpoint) { dev_err(dev, "endpoint node not found\n"); return -EINVAL; } ret = v4l2_fwnode_endpoint_parse(endpoint, &sensor->ep); - fwnode_handle_put(endpoint); if (ret) { dev_err(dev, "Could not parse endpoint\n"); return ret; -- 2.50.1 (Apple Git-155)