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 551E7CD6E5D for ; Wed, 11 Oct 2023 11:08:01 +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-ID:Content-Type: MIME-Version:References:Message-ID:In-Reply-To:Subject:cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=s7Ez+OTDX31hgafa4m2xjctNov31XcGhCgjkGzRcRHU=; b=VYp2osh/20JwymMNRNgbjVycKy hazVMBp2HSHTQSXon74oGwe3TbSvhP7V0ovApsaSscnkP0braAMZoypVgnLgRbGaIoqFjir0dQBuU RwFbMfzSM4E1djgzSVey1v+B4JeYOuAxLoQ8h23/gvl4gIbLn4Lu3ASNexUsLJaa3YjpmQiw6ZSPB j3QjWv4tTsovosowMLxWEvrdQ6cH/XmxYb0H1brfusnW1E+HXs5BpAQt14+F4Dwbej9k1OYms3tnm 4Tcj5RjE04Pwb5Dv27rmIfS7t9UJ+b05AdnFeoINKFrD7JxE6x/ykRw0ezxV+haFrgm8MKtQc0Yqq 2+cH38QQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qqX3t-00FhXM-2u; Wed, 11 Oct 2023 11:07:33 +0000 Received: from mgamail.intel.com ([134.134.136.126]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qqX3q-00FhTl-1p for linux-arm-kernel@lists.infradead.org; Wed, 11 Oct 2023 11:07:32 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1697022450; x=1728558450; h=date:from:to:cc:subject:in-reply-to:message-id: references:mime-version:content-id; bh=ckRPG2JUL38SKDbKOkmm+WWH+26uhzkNuPJSIGu2OfI=; b=K2sxKecc1DJXwVXBXDuiXQr5tz5sykAUcWPD1Y8BLkx4qfpB/N7mqS9k Rti9bCT3ap6WGMftlC1Y1nxtpnigpOgd9ciAZr7BQMozlBo4QDAxRAp6e nDW4NTnabsmSBDFJSvj+cj2u4Di7/gP2zLeaWmYAVAbgnIKx01GHJOWSb rOYiE20U9FPXtR6+kL0EAav3G1VhNUObXOE25Fpfe0P/mv2fFP0qClE9E x3ZrXm3dRDYBT2Bc1DBSCkvtdZ5KKugCnMIBeRqdFlVEuVhiuWOhPBWl3 EjXSaMxx0L9RB/UNwI4+vt799UtIXcHIm3xxFN/ol3Usv2rKbzfKbBTF9 Q==; X-IronPort-AV: E=McAfee;i="6600,9927,10859"; a="369698104" X-IronPort-AV: E=Sophos;i="6.03,214,1694761200"; d="scan'208";a="369698104" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Oct 2023 04:07:27 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10859"; a="788961825" X-IronPort-AV: E=Sophos;i="6.03,214,1694761200"; d="scan'208";a="788961825" Received: from opipikin-mobl2.ger.corp.intel.com ([10.252.57.154]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Oct 2023 04:07:20 -0700 Date: Wed, 11 Oct 2023 14:07:18 +0300 (EEST) From: =?ISO-8859-15?Q?Ilpo_J=E4rvinen?= To: Bjorn Helgaas cc: Rob Herring , linux-pci@vger.kernel.org, LKML , 3chas3@gmail.com, brking@us.ibm.com, dalias@libc.org, glaubitz@physik.fu-berlin.de, ink@jurassic.park.msu.ru, jejb@linux.ibm.com, kw@linux.com, linux-alpha@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-atm-general@lists.sourceforge.net, linux-scsi@vger.kernel.org, linux-sh@vger.kernel.org, lpieralisi@kernel.org, martin.petersen@oracle.com, mattst88@gmail.com, Netdev , richard.henderson@linaro.org, toan@os.amperecomputing.com, ysato@users.sourceforge.jp, Tadeusz Struk Subject: Re: [PATCH v3 0/6] PCI/treewide: Cleanup/streamline PCI error code handling In-Reply-To: <20231010223530.GA1005985@bhelgaas> Message-ID: <667e1068-20c3-938f-2d87-fb1d6c44a45e@linux.intel.com> References: <20231010223530.GA1005985@bhelgaas> MIME-Version: 1.0 Content-Type: multipart/mixed; BOUNDARY="8323329-1666151866-1697020845=:1977" Content-ID: <98898f16-3a16-7cb9-26e4-d9e9776916fe@linux.intel.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231011_040730_883319_AAB4EDB0 X-CRM114-Status: GOOD ( 23.20 ) 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 This message is in MIME format. The first part should be readable text, while the remaining parts are likely unreadable without MIME-aware tools. --8323329-1666151866-1697020845=:1977 Content-Type: text/plain; CHARSET=ISO-8859-1 Content-Transfer-Encoding: 8BIT Content-ID: <64336ba8-d7c9-4fae-879-59b88302220@linux.intel.com> On Tue, 10 Oct 2023, Bjorn Helgaas wrote: > [+cc Tadeusz; updates to quirk_intel_qat_vf_cap()] > > On Mon, Sep 11, 2023 at 03:53:48PM +0300, Ilpo Järvinen wrote: > > As the first step towards converting PCI accessor function return codes > > into normal errnos this series cleans up related code paths which have > > complicated multi-line construct to handle the PCI error checking. > > > > I'd prefer these (the remaining ones) to be routed through PCI tree due > > to PCI accessor function return code conversion being built on top of > > them. > > > > v3: > > - Return pci_generic_config_read32()'s error code directly > > - Removed already accepted patches > > > > v2: > > - Moved ret local var to the inner block (I2C: ali15x3) > > - Removed already accepted patches > > > > > > Ilpo Järvinen (6): > > alpha: Streamline convoluted PCI error handling > > sh: pci: Do PCI error check on own line > > atm: iphase: Do PCI error checks on own line > > PCI: Do error check on own line to split long if conditions > > PCI: xgene: Do PCI error check on own line & keep return value > > scsi: ipr: Do PCI error checks on own line > > > > arch/alpha/kernel/sys_miata.c | 17 +++++++++-------- > > arch/sh/drivers/pci/common.c | 7 ++++--- > > drivers/atm/iphase.c | 20 +++++++++++--------- > > drivers/pci/controller/pci-xgene.c | 7 ++++--- > > drivers/pci/pci.c | 9 ++++++--- > > drivers/pci/probe.c | 6 +++--- > > drivers/pci/quirks.c | 6 +++--- > > drivers/scsi/ipr.c | 12 ++++++++---- > > 8 files changed, 48 insertions(+), 36 deletions(-) > > Applied all to pci/config-errs for v6.7, thanks! > > I made the tweaks below; heads-up to John Paul and Tadeusz. > > Oh, and weird experience applying these via b4, git am: the > Signed-off-by was corrupted on these patches: > > https://lore.kernel.org/r/20230911125354.25501-7-ilpo.jarvinen@linux.intel.com https://lore.kernel.org/r/20230911125354.25501-6-ilpo.jarvinen@linux.intel.com https://lore.kernel.org/r/20230911125354.25501-3-ilpo.jarvinen@linux.intel.com > > It looked like this: > > Signed-off-by: Ilpo Järvinen > > Not sure why this happened; maybe one of the mailing lists screwed it > up and the order of arrival determines which one b4 uses? The ones > from linux-alpha look like: > > Signed-off-by: Ilpo J=C3=A4rvinen > > which I think corresponds to the bad rendering. I think I fixed them > all. Thanks for letting me know. It seems that copies of the same mail https://lore.kernel.org/linux-alpha/... https://lore.kernel.org/linux-pci/... do indeed differ for some reason. I'll probably have to experiment to see if I can reproduce problem with the linux-alpha list. -- i. --8323329-1666151866-1697020845=:1977 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --8323329-1666151866-1697020845=:1977--