From patchwork Sat Mar 2 08:24:53 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Markus Elfring X-Patchwork-Id: 209197 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:fa17:b0:10a:f01:a869 with SMTP id ju23csp373400dyc; Sat, 2 Mar 2024 00:25:51 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCVZRSlNNwWBGtyL0TQgUtUbByaSMxXZcvPhebnJFftQiA27g18NYF8Y560Z30kmyBWdTDZaM0oPIVoDuSYAeHlF4LJqug== X-Google-Smtp-Source: AGHT+IHxOH6V5CERoJeEBT455wTVSoif8aaW5EU7ZId1s01kFRZ/RMAGTV8VSANd/wk1fC2t4qvg X-Received: by 2002:ac8:7fce:0:b0:42e:c168:4a1b with SMTP id b14-20020ac87fce000000b0042ec1684a1bmr4278405qtk.7.1709367950930; Sat, 02 Mar 2024 00:25:50 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709367950; cv=pass; d=google.com; s=arc-20160816; b=U4vnFsV1AX46bL18pMyMx4bbUoP+3YZGU3JjUALTgetAuGMn29NlieiKNWSzJ0drAQ ukcZSPewwehbqJIcGObagCc06ZLFPH3sfwPRZwtbAPsg5ER3s5G2YaEanGBlK3vjAUrR S4/L4D5VSrZLjuDQJ4doji8GaRJAC339v57XA5XQDXSKHOJJ/cVcBLs7a1G7ZFx0pqWI szwfMuIfpwPfEmOnCr5mEDSOWEG58pWLoTyuGabmaCxvju6wtMAhD2BZlZm2DT5u0tdS tR1ifGVP4RUO25eUTVDY8oGDq7xxmgMGR5sIofP+QWcPYq26qPGhYQP8RT8cVkSd+bUX c1XQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=ui-outboundreport:content-transfer-encoding:subject:from:cc :content-language:to:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id:dkim-signature; bh=tNP4D00ZzEjG53nwR2NWIYvT9xWDDmphRKHFRMRXODs=; fh=HA1d9so1rcp0ijZEu8doZQdurCxeUXSOimNwkyLYic0=; b=FKv0Rj2kGQ0qj2JptIBL51Qjii2c6ykydF8CaRcoWGP89sUTa6k0UUIIhoZWH06Y1t FwXiSLPOeZq2j+PUay9a5HBiKCfsFNZOy1XaUP3LHug1wYt6JCH2a4GeOQyt0/XSMxLm XrCjM7qHFjOFxQpXhNw7kv6HIRmU86VdQvlYpFyyvMUPld9u+MMNSPbQwW0PlJXrXeuv BmHR7itwIjigkv9t/0G/wMEzRm6c2sdDE7H8w29YqkOzPpHAaLRtqk66PdcDrH92JiBF O4UfmgUIg6piuM+xJVxuI/kT5YFsVRYaSP69LpEqpp/HyJxDUOGE3J6P7ilSuiBKkpHQ BTZQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=ssCxMlfw; 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-89370-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-89370-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=web.de Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id t9-20020ac87389000000b0042eb6fdb69dsi4960894qtp.335.2024.03.02.00.25.50 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 02 Mar 2024 00:25:50 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-89370-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@web.de header.s=s29768273 header.b=ssCxMlfw; 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-89370-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-89370-ouuuleilei=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 ny.mirrors.kernel.org (Postfix) with ESMTPS id A72F41C2172D for ; Sat, 2 Mar 2024 08:25:50 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5DA0A15AF6; Sat, 2 Mar 2024 08:25:34 +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="ssCxMlfw" Received: from mout.web.de (mout.web.de [217.72.192.78]) (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 CF967134C0; Sat, 2 Mar 2024 08:25:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.72.192.78 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709367931; cv=none; b=EsKtz6kWuZBNa2173+7Jc5AuYUBrzZ4HBanX4v6IfGSfucP29Ykj/Fa/Pt7xV2lYjxqXlbt0YRkUocaHq0KocytlAumHMjyPRymysuqdtCPeBYmCBLdAnoS04jiJg5h+4No/t4mF5hTyl1ed2G//2Bm1wUGwpT/dkVwA4yg8DO0= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709367931; c=relaxed/simple; bh=/IG9/MvSXI2w4ZqCioEYLm/AwTPThoGcnx9Is/OAFXM=; h=Message-ID:Date:MIME-Version:To:Cc:From:Subject:Content-Type; b=urofCfqKh/MsYIgU5PWrC0Y9yMyaSD8TEr/9S5ygWICbT+OwGo+av7YwrI9TuVX1SOTqVo+R5c+txhy5kt/00rI2WZ0EN0PIt7dWopucbOqFYt7eyQr6PtSA1syaoXhKqpTED/pOxXjcxFCeHjZSPzr4gsHy1qMSpS1oFn16qZI= 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=ssCxMlfw; arc=none smtp.client-ip=217.72.192.78 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=1709367911; x=1709972711; i=markus.elfring@web.de; bh=/IG9/MvSXI2w4ZqCioEYLm/AwTPThoGcnx9Is/OAFXM=; h=X-UI-Sender-Class:Date:To:Cc:From:Subject; b=ssCxMlfwCti+lOYRr21QF1mgHtF/RJ2XEgyrgSRDdTolNlWxeYRylRsMCZwpUutg 4yEfrW5hdcg24pVXOCLhX1CiSekUjtHJ+Lg1mgPcplrEL3TPwSUvHgrIYxllbDme1 cui/uo9Ext4bJQ/QjHrJFUGwWZa0eNhtf6t6+jGrnsTqn+mw/yY25qZWF+HExrKFO DJ+YXBVKR5lXTJKntIJYcVVJFms0L8myBq+CcvEMUTdhTOnNZyz3VkJtJ6WVdzVMj GWSFRIvFbVsnraGwXBzKjZ3TbpnljoF3b5jAaQr/Rk0gjCocKZGO9i6ae0WCBFNAM vrkCAji9FxFdgxD59w== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.86.95]) by smtp.web.de (mrweb105 [213.165.67.124]) with ESMTPSA (Nemesis) id 1MkVsa-1r0vNZ35Re-00lx1U; Sat, 02 Mar 2024 09:25:11 +0100 Message-ID: <6bf9f962-cf75-459d-89f4-2546063fc154@web.de> Date: Sat, 2 Mar 2024 09:24:53 +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 To: linux-input@vger.kernel.org, kernel-janitors@vger.kernel.org, Dmitry Torokhov , Jeff LaBundy , Mattijs Korpershoek , =?utf-8?q?Uwe_Kleine-K?= =?utf-8?q?=C3=B6nig?= , ye xingchen Content-Language: en-GB Cc: LKML From: Markus Elfring Subject: [PATCH] Input: iqs269a - Use common error handling code in iqs269_parse_chan() X-Provags-ID: V03:K1:iU4w52N3qhKoRJLZZSPtf+QHPTsIuBjeHh8AJHJA3BiJ68hGjUg OnTVLq4wHVnSswAktXNc6dfsXtZ6yA+MUmif4fJ6HWmKukEu+vAd4rMkWVcUIlQruvyQUdo X93656kXivda+FvLfXLBYRmORhXDo7N1QGteUQpqZckHV0+KOsIBPG0DQ6qxuq7JDQGCac4 I4j0Y3JEHv1YIntrt0MdQ== X-Spam-Flag: NO UI-OutboundReport: notjunk:1;M01:P0:FRSmwSz1bmE=;ENul1wobOmr+CBbTNQETJkUvNDM AsjLTij0F36cjs1ms8NK65I/gPpWSEUtZhMuvEoqUDRgm5CImCgra+4bb7tK8eLxwxjyiYHDO Jfa7MfMo54QvXFn1F+2bkNES2YyNdx3jdq/YxodC84JrpKj28EaIUy7gv6E8DZMYVXSaaOTyT Hab9fOV/ftFhhIk/FATVYsJCxL75lT+qj31R3qLHLkCre5nbFqcAc0Wpd4FN9ygu58J2b6yxm YEAK9Y655F4JMCLpLzJzHJeJqirQhMyddilrhP166oW4j+iFCH3q+3kqNkw3f/uYmXhhrrHeU gYG4++xz45Ij7JZ5tdmSe83k/1MffxdLL7mdBsh3FOUWYsRamDwlElYkFgFxM4aZ98hsw62s9 TQgPKTrNV3CqPHdqyfRPozGgIJDttHD1s9BuMXl2Ple2hFxsS44flAXAhsEJzeF3xfqo8rN5H evYRggqUkOu4adB4QoRutX1PNzDSO82evjnpAQcK9dBW44NFjLbfFIX5zLfRTLeulDzIhQG4I jkjZBXfr9+hdzjfv61YjKb9v/Ercao0q7pim6XhM4IJ6PtEiEsp0s+9ELz0mnDwQRGSFkO/n3 WPcdDKxbAzVqhDfdCze4tKbWST9xjH7Tiohyebk+54BFbKNgRjYD4wG8kYXFWNzhO87nkBNJ3 4NDiUjxpVeolTzqynZfMJceAdHoxK1m5qYxkXvaLBTuveA6k7pHHplf21Tv+Dzj3f4coJJG20 Ozi+XuxQdJyWQYYYAbgdwRcYzBQfvBXnrU2qlFQniW9adSC9hNdZmFv0VJsinPbOKr//J1b0i iYK4aAhd0HDgJzeEvnLO2kJZgyAp+z82vimTokAt3mzM4= X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1792402208862772244 X-GMAIL-MSGID: 1792402208862772244 From: Markus Elfring Date: Sat, 2 Mar 2024 09:15:20 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function implementation. Signed-off-by: Markus Elfring --- drivers/input/misc/iqs269a.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) -- 2.44.0 diff --git a/drivers/input/misc/iqs269a.c b/drivers/input/misc/iqs269a.c index cd14ff9f57cf..1379f80c00e2 100644 --- a/drivers/input/misc/iqs269a.c +++ b/drivers/input/misc/iqs269a.c @@ -744,8 +744,7 @@ static int iqs269_parse_chan(struct iqs269_private *iqs269, dev_err(&client->dev, "Invalid channel %u threshold: %u\n", reg, val); - fwnode_handle_put(ev_node); - return -EINVAL; + goto put_fwnode; } ch_reg->thresh[iqs269_events[i].th_offs] = val; @@ -758,8 +757,7 @@ static int iqs269_parse_chan(struct iqs269_private *iqs269, dev_err(&client->dev, "Invalid channel %u hysteresis: %u\n", reg, val); - fwnode_handle_put(ev_node); - return -EINVAL; + goto put_fwnode; } if (i == IQS269_EVENT_DEEP_DN || @@ -805,6 +803,10 @@ static int iqs269_parse_chan(struct iqs269_private *iqs269, } return 0; + +put_fwnode: + fwnode_handle_put(ev_node); + return -EINVAL; } static int iqs269_parse_prop(struct iqs269_private *iqs269)