Received: by 2002:ab2:3141:0:b0:1ed:23cc:44d1 with SMTP id i1csp2003427lqg; Mon, 4 Mar 2024 09:50:28 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCUq8IRmU4G5q7yHu0WZaVLGCNGxPq8J20DOgjyD71JiSNKqPNaK/cMzrks4K0u43Skg2G5Fmr04aarqETYwUNflWkj5IMbwOedd+0No+A== X-Google-Smtp-Source: AGHT+IFvcNQWpnbmQVlcXkKr2Zn5T1vNWbJV3znyx3iy3fvVSo9qT228RHRoclFg5t12gcj0+sh/ X-Received: by 2002:a05:6a00:13a1:b0:6e4:59b3:928c with SMTP id t33-20020a056a0013a100b006e459b3928cmr9874473pfg.10.1709574628556; Mon, 04 Mar 2024 09:50:28 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709574628; cv=pass; d=google.com; s=arc-20160816; b=ARifKmEroA1nVj/b/NFark7PQBzcqb7HDKyfrEIB7FjLqp6KHXxlh1EbOlbQKkxNb4 4WMVIzjtDWbjg1VIrIegkawpdAfwjflCjz1tE2bcIb2KXKnqbBGbow1R+lyyqlAKYlfB CMjLp/m0iwXo8OcqxycxD//rkZ9FkrOyjw/Sw8j9wKQbLXHAgXr2LwxXV9Q16/mPZmOi UMuNvV0ooGfl16P5uVeZ5IKCqxaYPpLR4fOO4+M1qmNRxit+wy/aYie6L1fwVU++kCVQ S16GDSpPZbsIgyDFlYf4rWqkh57AJFN3coj79Lg21MbMQumd/5+L+yp5kegN/5015sLf 51Rg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=ui-outboundreport:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:date:message-id :dkim-signature; bh=7K3bP1+oTQ+p1gZWNk1OVZkWGD2ZtvOsntwZZ5+E8kw=; fh=OukfR1qheCNLw0+69fiFRnvfWtTpFYWohbwSretOZuc=; b=sCZO+14g21IuTKszRCHNrTGgNjR35XjAmsz0zkE57Mdz+Op3AD4xw05G8Yovsxsh9W N69Cho0DdYLMEyTG4v/TtM3cWHZAdqEmLZzgIqp9eZ9NkC9IYiyQ9epshUUjn6TcqAg+ vmX2x4dUs/fmiNkhmaE+LQnTtSunBDKeQfTi05OYj1Q8LRy5U6aTnWHlBFVxjn1yrTyF 5GAyBjNR/XGXdioZwV+QW1ntJwRF0r2EZzPXR+NGmQ6wSPJ77r0xgH8fABVcNFTN5bCe IWfcejkSQ2QMh/9z1tt7crNjVVIDLS7YVBfMPKBeyJua2NdG3BYPmo1Oq0drzi3scNbo 0mhQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=CflVt1jW; arc=pass (i=1 spf=pass spfdomain=web.de dkim=pass dkdomain=web.de dmarc=pass fromdomain=web.de); spf=pass (google.com: domain of linux-kernel+bounces-91027-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-91027-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id q26-20020a63f95a000000b005dc8ccd36d9si8775116pgk.250.2024.03.04.09.50.28 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 04 Mar 2024 09:50:28 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-91027-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=CflVt1jW; arc=pass (i=1 spf=pass spfdomain=web.de dkim=pass dkdomain=web.de dmarc=pass fromdomain=web.de); spf=pass (google.com: domain of linux-kernel+bounces-91027-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-91027-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id AC265281E1B for ; Mon, 4 Mar 2024 17:50:27 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id CD5306167B; Mon, 4 Mar 2024 17:50:07 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=web.de header.i=markus.elfring@web.de header.b="CflVt1jW" Received: from mout.web.de (mout.web.de [212.227.15.3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 94590612FF; Mon, 4 Mar 2024 17:50:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=212.227.15.3 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709574607; cv=none; b=uBl0Mp5qot0b3l7PO+fAFSoSZA2X6Z4JRKEax7xk1uGXyJZqRksMOk+M7WaMvOFhdw/8PPMyRzc7wLPLeTjRYRMDU220ggzGn9qQPTrKGUmBgKpgKU6XbgmIapEel6VNas/1wTdIutRmughk5KssHptIPXbFxi+nuCUu+168lik= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709574607; c=relaxed/simple; bh=7K3bP1+oTQ+p1gZWNk1OVZkWGD2ZtvOsntwZZ5+E8kw=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=odWbaUQJyRP4GVAlgzeEOA7bYTMg12MYYzWjSht0MuEbm8S40ys1TiZRX12gyRzf+3HyDERsdb6o6nfyTDlLfra9fSOm+o+bW/TAx0iMLrLY71ZWmVZGXLb1SEUG/K3EpbQp3Y/Rd4yKBOgsjKm6OmMJrSZTIZF81bGfTBdaw7A= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=web.de; spf=pass smtp.mailfrom=web.de; dkim=pass (2048-bit key) header.d=web.de header.i=markus.elfring@web.de header.b=CflVt1jW; arc=none smtp.client-ip=212.227.15.3 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=web.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=web.de DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=web.de; s=s29768273; t=1709574554; x=1710179354; i=markus.elfring@web.de; bh=7K3bP1+oTQ+p1gZWNk1OVZkWGD2ZtvOsntwZZ5+E8kw=; h=X-UI-Sender-Class:Date:Subject:To:Cc:References:From: In-Reply-To; b=CflVt1jWgMV6mfTL5qSrUp61G3mhxbyohnpj2KyHsZv3/puovr5NygeyA6ACfBQd mvP5K8Sk7muKGyMRb8hUwg6Skamz/liyXcBOQqXrKPtlTlBGpOnqaFPIc/joZ+mrD +h28OH3fAFq0Ltsm5qYcrxhjwl+zXEm18jXeVFNFrx8zHzUEX3JjqCzvpbY5SUflE 0Simsc2SRZHJjrPhnmnn7dwxpM3mqWwEG6h/U3p+sBLH1ehoqZeBFyk6uY1CG8KDW lYeOzlfw2yDaFwMB7204MSG6z5QjIbLuKkFe9Xz1jgyXkvksg/+YkXTNh0Yiukhku neqQ4dB47fvFYfis9A== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.86.95]) by smtp.web.de (mrweb005 [213.165.67.108]) with ESMTPSA (Nemesis) id 1MFs1z-1rdfnh1i2R-00HCYi; Mon, 04 Mar 2024 18:49:14 +0100 Message-ID: Date: Mon, 4 Mar 2024 18:48:58 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [v2] Input: iqs269a - Use scope-based resource management in iqs269_parse_chan() Content-Language: en-GB To: Dmitry Torokhov , Jeff LaBundy , linux-input@vger.kernel.org, kernel-janitors@vger.kernel.org Cc: Mattijs Korpershoek , Rob Herring , =?UTF-8?Q?Uwe_Kleine-K=C3=B6nig?= , ye xingchen , LKML , Jonathan Cameron References: <6bf9f962-cf75-459d-89f4-2546063fc154@web.de> From: Markus Elfring In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:W4V1XEZEso69HFCLMu/ebTwwAh/2Q6Ps1nvxXQYecLV4Ex4Y4tq Zf83KIuZFNK7Owqdj48cyayM67viuzP62V4soZSVINueMLayTAwkRLM91eh5Qmk98JNCdi6 OOqY5dyy0oEsjQPDOVcl0PBbiVzyXw/10vcifUnJxXYoIp/NYbXPjiyrpHeB/HggtHc6KaF cCYNMJ6RbXPfgw4yGJ0EA== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:PKqMxeqTb/s=;xhcINlzZEoVtVbkCJ5JeDg5kP6h KU9RJoGyHBHBHTnGrX6Mjs9P94H5vTapipJRq14O2WtFN/irotdXmULfASsnimS6WWsVuO+Bc AH0lUuB4FvdfNmaqyvB3dXZO3Y5NB1gNYfEpLIAeoel7fl0bNNWvxH8xGsiQcFJ4yTGs9OfuI vIe49oTQoUXrjDgIjoWqbSDFtb4U3EgxX8panHo0OZDkIEwVsqzleDiBtoBE2XzIMUAVeTBoL VXX9qM26hw2RBM+WPjBwV2EwWa8ZKVP1lHaZTv/0OX1tAqwyqRPJPX1nGY0Vbh34y2jiuvpb+ vmNsTzF2OZHkBR6FKZjtkv28jMcIMR7owa2jg32HGvn/xIuBwkojO14vE+HP0fAA0A1tpv+lu HaZ1dExdoJmtrrREucwIz8K7GGaoLlLNMAL08or0DFWMaRuXafBrlpwbdtF259ePy4hPPBPat f1eniLxKcpmwhAqOOINSiUwsetp38FiSoyDO1wbB9xUJfqlIuOigeniInQafapP4T64effSxB nEEylcJml141mA3bTm1VU+9R/WjEyZXE66a3e1JXTvdAXqxV7rwHmZm6x6FpLWIiYtswuawJc IIpLCoPEi75CDAAiVnR+/CX+bR8C40ia0tTZNvDP+dCc7MLCQ3EwlTMiO7eVywLQw4KRnNmm/ ckNmYCho2qttMnoBHoS9fNYRfpED3ma622ZpiIWyNxysdm+/QkzKnJ6GT5BaMuYYN7+rY0dqs 8aS4SvDIi8wvHba/d5vKBmBgYWwsjx4lCQZjfk/xDeCx8mgide6xtQ/ybrsYYWyTcahYrcQ9w 0CGup+Lxhs0q1sHNL/dF9fw1quFn7DiPrkW9nHPEJYuAE= > The extra curly braces are absolutely not needed. The for loop's body > already defines scope, __cleanup()s should be called at the end of the b= ody. I present an other development opinion here. I got the impression that the required scope should be smaller for the adjusted local variable =E2=80=9Cev_node=E2=80=9D (according to the pr= evious function implementation). Otherwise: How do you think about to move any source code part from the loop into a separate function? Regards, Markus