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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 5F30BC2BD09 for ; Wed, 3 Jul 2024 18:46:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:Subject:References:Cc:To:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Mp/RJoiuH5JiTbBnMyafiYmkXjelrGaI3F8h2g29e0c=; b=GoLBvpttuFIBkPFfBZxTTxrPHn xuKSCXluxIwNXNBRPpcnsDETQNVZ37VNtbUFMg9aDsICcA4g5NXc4Dbh4m1OXYaTuvOWYJw7iTfkX Wq7dB0iDwZvTEKpMhkcOFzWKiXoF3QknkWHT52sySYWhOFhAgwVgx0qHrcIXmqXuH6ksu8VRjHS5u BMhKyrnu5tz9rTTlasGQc3HrBG5RmalpuX9inqVlAP1PVyXMCYvgzbl71iLJh2vZfg87aqBJmeqFE 7EqvWsE57E9srEvJfHNBYLhUm9u+pFkRJf6nMrS6BiA93gRB3Ia6vLY48aZUI5OCbWKkmHzeVaJ6a a7MNeiRg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sP4ze-0000000BBtS-2V4D; Wed, 03 Jul 2024 18:46:14 +0000 Received: from mout.web.de ([212.227.17.11]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sP4zR-0000000BBrk-1vEp; Wed, 03 Jul 2024 18:46:03 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=web.de; s=s29768273; t=1720032342; x=1720637142; i=markus.elfring@web.de; bh=Mp/RJoiuH5JiTbBnMyafiYmkXjelrGaI3F8h2g29e0c=; h=X-UI-Sender-Class:Message-ID:Date:MIME-Version:To:Cc:References: Subject:From:In-Reply-To:Content-Type:Content-Transfer-Encoding: cc:content-transfer-encoding:content-type:date:from:message-id: mime-version:reply-to:subject:to; b=DLfBixoZfSzfhAqQhaJRZxKMEWBeWNTs/I4tZy8GLj92Uz4CuoE5BaMRKt0w8YB2 kdfa2rSjMsn+COXpIqnbYMeODqSvQ0ghCP/51fvflpO4HKp2XtTlPWFZ+u8ELzDZq U/2lhOAG4vjFLanfW9RDCFQyCqFYAw6ywSq6kiV2LQodarS/jfks/cQdRlirzCyVQ 3l0pZS3yOlct3FokXtKQbWK/2q/g6L5xsTQtoHuGQfwc7CDDVZbHY9UbBWkw3A3rn NWcpF3SK5EAur9DKh3eOzvl5ZSl0Z80JDR9pMPRlewAEMU+p+mVBngtvZcV/q1NvT xUi05W/X9Dzmk4hO1g== X-UI-Sender-Class: 814a7b36-bfc1-4dae-8640-3722d8ec6cd6 Received: from [192.168.178.21] ([94.31.91.95]) by smtp.web.de (mrweb106 [213.165.67.124]) with ESMTPSA (Nemesis) id 1MCGWe-1sYSv93Y2s-003ArN; Wed, 03 Jul 2024 20:45:41 +0200 Message-ID: Date: Wed, 3 Jul 2024 20:45:34 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird To: Jim Quinlan , linux-pci@vger.kernel.org, bcm-kernel-feedback-list@broadcom.com, linux-arm-kernel@lists.infradead.org, linux-rpi-kernel@lists.infradead.org, Bjorn Helgaas , Cyril Brulebois , Lorenzo Pieralisi , Nicolas Saenz Julienne , Stanimir Varbanov Cc: LKML , Florian Fainelli , =?UTF-8?Q?Krzysztof_Wilczy=C5=84ski?= , Lorenzo Pieralisi , Rob Herring References: <20240628205430.24775-3-james.quinlan@broadcom.com> Subject: Re: [PATCH 2/8] PCI: brcmstb: Use "clk_out" error path label Content-Language: en-GB From: Markus Elfring In-Reply-To: <20240628205430.24775-3-james.quinlan@broadcom.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Provags-ID: V03:K1:36kGwr4RXLOs7Zyisb/7KuH3H8Liv9r7JcDTXD8/jLwpnjgF0hH ol9kafgktAzGSLXOI3u5n7KJwetN6a6FrYF/Jlck0Kaqr7ZIa3D2cxTdXotpITGEgELkEbx Ok9BtNlxZWTd7R5H4q8//sqNHFG5vJ+9F0lZSo/ZhRSfhcGi+DcEVOrSsy1m/UXQsdITxE3 7yGSt0qlFVPzxzuxYp49g== UI-OutboundReport: notjunk:1;M01:P0:dIV3an7U1FA=;Q2jgiRBuBnX9GKqS3FEqNVzp5+N C/B5fQgLI3Dq6OBVYywgdc8zmgY8Tbshr6GC5ABogV2TiRu8LV0IoWb0qODMaFmZHeP3SElDo Odz6hJ2XS6X9ej5kETmc7DSstPmdOzjaFe4YhElbJkzqXogvXPbXcRc0X/RjhLmiyRoDBA3MD zwS7xgsGla/hcSKFgCZ2hA0NX+Uxc6BfG9TW3zZ+kQ//UvoIyrajsDg2j8ZqMk+bqxEeO0x9d RPxru1IVjrJAJPij7i0nkk9qx1zviVPBKUBZNErlY9PG3sRs3jl8AksGvqpd67Ot3IFrDPJTb NBa/Xv87Go/P5Xciu9GsVtOtkWxExZj5NLJKMBulNQLKd1oVTUXVwLCunBrR9p005jy18W9D5 81Yb/H2X1B3IV8/fH3eziMG4ZMIfQ984jZhbUIWazfg90CNdW8VON+/a5RTpGRMiBYqUghlZA /JPMw062527JCKPcHS/Dj2MUlue883ilzlStgUNlzwa1G1lVv6KtxQ0fBV0cmqc5MLb6tizhK DdsowquqwXFn1wp7VYm72yz3wE8E+DOz7lQ8nx+xW55t4DKl55Mstitt9wr4WCauuOCcDBBqh gmx1dz3aans2fVrAjZZV/H70EmV/AcAefhFkf1aUYRYj7GRSOWPAb+0gfi/RLzDnGgND34ij1 fif0V37vWJMlWk6oOE2Bz72qsDziw3125EIwPx7d7Y5koPLphLLTnyFIBOQ/+yBy7poCCYMsw JLZKnjDxWCSYGzBakul3szcE1vDTYjql48Cpu7xXeo7K47FqIA8TmavQvsLFZnGYNUnpBToMh sQmL+nRzgiuBMOIc6HDaTdEUfhU5vBdLOEMm2fCkk7C9s= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240703_114601_892429_8072AABC X-CRM114-Status: GOOD ( 10.89 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org > [-- Attachment #1: Type: text/plain, Size: 1600 bytes --] Can improved adjustments be provided as regular diff data (without an extr= a attachment)? > Instead of invoking "clk_disable_unprepare(pcie->clk)" in > a number of error paths. * Can a wording approach (like the following) be a better change descripti= on? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/= Documentation/process/submitting-patches.rst?h=3Dv6.10-rc6#n45 Add a jump target so that a bit of exception handling can be better reus= ed at the end of this function implementation. * How do you think about to use a summary phrase like =E2=80=9CUse more common error handling code in brcm_pcie_probe()=E2=80= =9D? =E2=80=A6 > +++ b/drivers/pci/controller/pcie-brcmstb.c =E2=80=A6 > ret =3D reset_control_reset(pcie->rescal); > - if (ret) > + if (ret) { > dev_err(&pdev->dev, "failed to deassert 'rescal'\n"); > + goto clk_out; > + } > > ret =3D brcm_phy_start(pcie); =E2=80=A6 Does this software update complete the exception handling? Would you like to add any tags (like =E2=80=9CFixes=E2=80=9D and =E2=80=9C= Cc=E2=80=9D) accordingly? =E2=80=A6 > @@ -1676,6 +1677,9 @@ static int brcm_pcie_probe(struct platform_device = *pdev) > > return 0; > > +clk_out: > + clk_disable_unprepare(pcie->clk); > + return ret; > fail: =E2=80=A6 I suggest to add a blank line before the second label. Regards, Markus