From patchwork Thu Sep 8 06:25:50 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "M.h. Lian" X-Patchwork-Id: 9320405 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id D47726077F for ; Thu, 8 Sep 2016 06:26:50 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id CC3F8295AD for ; Thu, 8 Sep 2016 06:26:50 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id C0957295AF; Thu, 8 Sep 2016 06:26:50 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAD_ENC_HEADER,BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 42923295AD for ; Thu, 8 Sep 2016 06:26:50 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.85_2 #1 (Red Hat Linux)) id 1bhslb-0005Pa-9Y; Thu, 08 Sep 2016 06:24:55 +0000 Received: from mail-co1nam03on0081.outbound.protection.outlook.com ([104.47.40.81] helo=NAM03-CO1-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.85_2 #1 (Red Hat Linux)) id 1bhsl3-0004lW-Lg for linux-arm-kernel@lists.infradead.org; Thu, 08 Sep 2016 06:24:25 +0000 Received: from BN3PR0301CA0039.namprd03.prod.outlook.com (10.160.180.177) by MWHPR03MB2447.namprd03.prod.outlook.com (10.169.200.141) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA_P384) id 15.1.609.9; Thu, 8 Sep 2016 06:23:58 +0000 Received: from BN1AFFO11FD035.protection.gbl (2a01:111:f400:7c10::116) by BN3PR0301CA0039.outlook.office365.com (2a01:111:e400:4000::49) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA_P384) id 15.1.609.9 via Frontend Transport; Thu, 8 Sep 2016 06:23:58 +0000 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=nxp.com; nxp.com; dkim=none (message not signed) header.d=none; nxp.com; dmarc=fail action=none header.from=nxp.com; nxp.com; dkim=none (message not signed) header.d=none; Received-SPF: Fail (protection.outlook.com: domain of nxp.com does not designate 192.88.168.50 as permitted sender) receiver=protection.outlook.com; client-ip=192.88.168.50; helo=tx30smr01.am.freescale.net; Received: from tx30smr01.am.freescale.net (192.88.168.50) by BN1AFFO11FD035.mail.protection.outlook.com (10.58.52.159) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_RSA_WITH_AES_256_CBC_SHA) id 15.1.587.6 via Frontend Transport; Thu, 8 Sep 2016 06:23:58 +0000 Received: from lmh.ap.freescale.net (lmh.ap.freescale.net [10.193.20.63]) by tx30smr01.am.freescale.net (8.14.3/8.14.0) with ESMTP id u886No2R022045; Wed, 7 Sep 2016 23:23:54 -0700 From: Minghuan Lian To: Subject: [PATCH 2/2] pci/layerscape: fix error response behavior Date: Thu, 8 Sep 2016 14:25:50 +0800 Message-ID: <1473315950-6396-2-git-send-email-Minghuan.Lian@nxp.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1473315950-6396-1-git-send-email-Minghuan.Lian@nxp.com> References: <1473315950-6396-1-git-send-email-Minghuan.Lian@nxp.com> X-EOPAttributedMessage: 0 X-Matching-Connectors: 131177894385772383; (91ab9b29-cfa4-454e-5278-08d120cd25b8); () X-Forefront-Antispam-Report: CIP:192.88.168.50; IPV:NLI; CTRY:US; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(7916002)(2980300002)(1109001)(1110001)(339900001)(199003)(189002)(2906002)(8936002)(68736007)(4326007)(50226002)(105606002)(11100500001)(586003)(85426001)(76176999)(106466001)(50986999)(229853001)(36756003)(2351001)(110136002)(97736004)(305945005)(47776003)(48376002)(189998001)(5003940100001)(77096005)(87936001)(2950100001)(92566002)(86362001)(8676002)(50466002)(19580405001)(19580395003)(5660300001)(8666005)(7416002)(81166006)(81156014)(104016004)(356003)(7846002)(626004)(551934003)(7059030); DIR:OUT; SFP:1101; SCL:1; SRVR:MWHPR03MB2447; H:tx30smr01.am.freescale.net; FPR:; SPF:Fail; PTR:InfoDomainNonexistent; A:1; MX:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; BN1AFFO11FD035; 1:W7tDKueWzXJzUS5gmGqZisfPJkG4q37CJ9YiyV4GpUWBI057/xqrvjLISSMfrJAq4sS6j6N7WdHqWqf9v/b8w5O8ZMHu2G9iCZcEHnRXCSfesd4LJAxdaB3vtSP1gtpMGKlMql34c2ncoFZ07nyFcOnKZOwPPD+3vPHRplaBVREaBROFXtq6uDpLgGOZK8YT1xU3+llU5lt2wLhqHDKzglR1QDEB1T449EaeDKl7XBW3ZoMoKDEVV8l3GkO2e95y732EhrWYgFUzvLjgIVYwCqSkNNuaCxP8f3ld216tEv98asfWGDaBnAyb9kCldfIlHZQ2ulEN1s4RjD+13YT5E2msn9hIhsZ0HDUN+eSPkRlGyFbbAy3x4t3wssh4syfLjVIDUZYNAXpe4K4e+C42ZFl0vWXlW1XhP8UK+LwQdI2F94mMphVeFzP8hXnDmZNYmCNo1FPz9FzqhwrC6MtvVofaDAIe0Ay6MBIL95Xk7047kozlYTmnzbt3N0mpRKL2u1gxKKg29EvtDUdLrGpaeixMLiJ4H6FXUeZJfpkEfoA6nZVNqvJtZwXRWd5BFVE56bG2CD5AdFm0DpuarMlbboMMPBy+3BiR3S+xOLvY4+abY7oh6EKkmAiESescpPxrXCJyY3zQiuYYZaezmG0/dIAFdr56qEeO7atFJhYTAwfaGD4aQQ4qpK3CqbrBcCkEXmyyiyhVEjCqEKdxPFZS1Ges41gQoIE6PDMT6sN43H0= MIME-Version: 1.0 X-MS-Office365-Filtering-Correlation-Id: c7b9dae6-bdf7-467c-05fe-08d3d7b0b7b4 X-Microsoft-Exchange-Diagnostics: 1; MWHPR03MB2447; 2:Ta16TWeJ2jHq0EGvMHH+B88eWc0OsJp7MOkSBhSnoQysOorLVgW/sI6eRnC25Y/O9tZTOPDRNE3rT4ikPT9gSNB1y0mVavjHqhsfRsYg5EPItls5nYMgw9Pn1LSpssn9qcYmtPM3QOOayuSay20THJKBSbgq3FDvwmN6RCt2naO4dC/sqjTpmGTiwRSLPuCs; 3:vGPlrppSXNf9atPotBkQ8K7j7Vw+qFc68qTVFuTzc3/Lzud2w5EMSVA1H/KlWlYeLEWjWeKxW/V0yPGTtlxTzWy7zAS6zDyLBIFVCK8hNTN1YmyN2evSAyAEk6GcYGOePZxay05WRLxw3opUYX5d3B+wHCWooKecBoE6DOfqqcHmtaa/bHpUhznxD8dbBTvn1QRZSU8P/impCc6ahmuwPRGgeaiqlSqEm5rnu3Y7RMQ=; 25:1xVrnmbpEe7BXxrWroFgFOvTy2tu8gEmIXc/m8pIFRtRoN8EgvV0FMttN75GiZkVEQrut7OMpT/LIF7DOWo0IEbViMRmvepuhLII//qoSaeX2RhAmkqPrrtgX11F9XFphkM2IsK5wEkTQwCL2LtE99qis+og/JIzQqghgqCALNMu7zdmzJmie8vzw+wTKCiBT+U+MWKdfXiz6pev7kANscyHRED2zJK9z/OmH5yz9ZexSV1gw8He9yY4YY/nivIyBJz33arzzpJ1UTFZTIZ7/7+trj/V9XODwsvXl/WEB1AXKMzccTt0rIig33Nk4IfOXsS6LnJjdpP96DIe+gyb5WCMVqxjUyTgY6cpoM4Ed47EC+uz2n3rmdckWMQ3Uf6LCiOmeEsKyGjZEjUA/1pYf0wg+4Au10A55TS+rFThHKw= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:MWHPR03MB2447; X-Microsoft-Exchange-Diagnostics: 1; MWHPR03MB2447; 31:i5ZlQB/wAdUrtS3Nmd8Da7JKXv2SsCYkv3PJvhtu9tr56/DhySyY1nNXCADjgyvoaxP6lB48ATKlwfJJnfy6/ay617cLImUAYMDfjHz64VwIeyaCQ6vhrdP0azV+B+Q7lcjiBXwDah9mGkV0fTKnjBOWn0tezIOzb2wCMJpeuMzXvmXPjocFCKvC9uPuwi1917hItINPFyoxxJqlLk9/i3t0bagPZ2cA8uyB3Xh7I6Y=; 4:LZhSV8yO/Mp74aYu+2g/6qOyVGAuQQ2k9+ck9NvjOQPwidg/GejY811BiwqdHb3iscEAq1T6pdFwtu0skA92dzTt9X/nKtW8p0QftsFmqFmGkGqDQKxosB/0by8tV1wxqEXJA8/VT+/O+D0npjxUJkqORgwXzTP4e/p/d0j7JmhBudYSOS9pNYY9WCugwnYHHdW1v33cW3eQ8ipdTyWjG0e4a7L+kjob1Ttg50tw6oAtDKFtNFsoi+VK7inOD9RzEbdfd2MYROaqwx6h7xRg6KDGuLxf0IddFTBSiJV2eDB7mX/eqReZ9/OLo5Pgo6C1/IIlGEeo+9XyyIcUUR1+B/2g95s7/ka9hvqkaM4jAHFLOa6zjKy05fMWFsUnomTlBYCJ8a7CP+kHvPcCDYzE1GBYOm1bZmxjZsUuEGQXcKOjDTB1ok/cbGAa0yGPfiy3Vcyz/+5Be8f+2Zu9GOOFyzMZjWqtDgmw5S23zUiWIYePcjKF/ijaAs3a3skQ4a2D6qPUFdhuyiNLAUMKGguoqZQVk9qi6sHMoJ9O1JsU91s= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(185117386973197); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040176)(601004)(2401047)(13017025)(13015025)(13024025)(13023025)(13018025)(8121501046)(5005006)(10201501046)(3002001)(6055026); SRVR:MWHPR03MB2447; BCL:0; PCL:0; RULEID:(400006); SRVR:MWHPR03MB2447; X-Forefront-PRVS: 00594E8DBA X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1; MWHPR03MB2447; 23:xO1mL/RCo54/JWZe+fG72yN8naTgJg2KmZMujsWiL?= =?us-ascii?Q?LFDiXCR8ivbotvEbVWoRiJQeRhwOQzNkjv3KY9u16MPTmintnFF/4/lGmlmm?= =?us-ascii?Q?rhsSxRBYIJxXxg2lmn1gaImrOSUwCPoXo5dvAlO0PgrsnD8gbDr17DdQKl0b?= =?us-ascii?Q?eYM0y57tyBR5TBGSkqfeEoqhQJSUsFf3OyQ5ui/XlxS/dV+uGleanAPLJQNa?= =?us-ascii?Q?WEW+CZNcr/WO1Etp/3mO4w9+0XxpSg8iVDzQ80Fy1iUPZLk5BmqGvLCDU8Lk?= =?us-ascii?Q?KySN6GF4Hyn8mIothPfm6qOgjx0VQuKUYli8+K2k7iCjh19TwOo8/9CYZNAH?= =?us-ascii?Q?38mZUadAOBaSHNVASsHMg6OicdnczbAnwP9/lUFQXzAdq3oPrzHhyefy3iQm?= =?us-ascii?Q?GtCBZNS8PvHSXlkIF6FzUp5+Mb5AcKiC4CV4bYlovPsXJBOEHD+EN9XstUfu?= =?us-ascii?Q?M/4NciRikMhyxGk5el0HEQ/SW2LhwO2qrg0OJ6UfNl4vf2yD24JQ+hV3bOGD?= =?us-ascii?Q?W2PRd24Eo0G5hGkHctc7IkHshZLw8I/OfFEeh/bLbRCMTgSOEMRxHjHGw5Sl?= =?us-ascii?Q?pxrJ/DBc6kIlMzV9b+FKfy+9w+EjpXGApL+bxwk1pMtwuFNQ314m3v89g7CZ?= =?us-ascii?Q?KiuOHhSNxz1Xjp0H5VDqTm3nrlzl2rkXTSE1B2Y+wGzzjUBKf5Std1xYHjkV?= =?us-ascii?Q?xk4ysY8gYo7VwseT8f5N2iSgYXPXm9ph1Cr4Y+S/RXLx8T+5aCSVb9mW6YdF?= =?us-ascii?Q?GKHuur0scw8NqnMvJ/Xrf3rbwW3/pBdQD2jSxDMfbSaPtl0cBckq1eBPA7sh?= =?us-ascii?Q?bS/B0JMz5x1AyV5hR3/BwFSmVsJYy9MVIDLTORPV2e0KBj3EptvlJVVTREYX?= =?us-ascii?Q?d/ugL2lx7rvxspJJHigNkCrdSmKRastpTPwR2c7O12/X/iEAojNgAgpo1wj3?= =?us-ascii?Q?71Wsi0YGDSG66Yyf35CWdHZJcHUBccv0/poFfnxn+pc1YBvrGvPp8xamVQzp?= =?us-ascii?Q?Id/n4iaEbVcq1l1V4TxpnEGrXJrjlZwN4COWG/jGTt1gCXBthj7wymB4H5r5?= =?us-ascii?Q?h+HrCeQMgYk4mkRdWQSDRunt4ZrEuDCjjVE5x0pdaFE60XF89R7wOuDy5/bt?= =?us-ascii?Q?86wH80+wkf10Nx3gt7G1FfuLT4ZTdPP6UQIU/I+lBzyVfEaLrH2RL4fQkIG+?= =?us-ascii?Q?80DJakd0pEk0FMtNgOWWRZ1CQ1AsEishLQr?= X-Microsoft-Exchange-Diagnostics: 1; MWHPR03MB2447; 6:aTpz+E2x3QbjUxCW0w7F99wnRKN7JiwhRZ8tM7AIbDla+Bs38fkBu13pVFg9pOpUQmnzwtYBoxYRI2j4DDsBeHaGlesVscKUz4v66lnB0okfNsUaLT/7jVhqyQT2ahaR+YiMExTTv54DgfSn5KMS1jLAyDlbLZlKZZCbi/T2tadFAh+9v1C43FMYWa12n0QSpBMb1lv+1NdQDZB2z8lUZfOGV7GGizpN/AChmiltG8/k2LoqyZqXexrQH/DADFMcSixJo8cDDI7ds2twFeyF758FGorn5jtjzo/pM708x14=; 5:cvZWAKxcq/LXO/uqpxSkjoYUxc8wWq2r38DN//I+RxAGDD3mqVyVgAIPoGRYXLEnfD9evTDMY0ekLWXHDkneNdbjILVmJsWDe6EyBsKH3vV0eRhhati4d8qqcUlWD27h+JSlAhfgMNp2gNtRrN6IPcVrk5nXAwb+6yV9ngVnqFg=; 24:bENhQVFRdd3qQ95Frf4rufOTWCWI2aFMDRJJoF760tHedTXC7lYUfuduiipOdawxaxfZszvtp/sD4h7t/r2eyRvg9Yprh2FLEfQZTLqVXqg=; 7:0uoKTfBV1aKJ3U3lUW2cpMN2YG8fM9wHvht61MstOdsSN2PaE7aUzL+PS0iaavgqswhW1RQ0AQa1zx3WaJ1RylhUc0/vrgq3YWb32aKU6nfBwZy/3MVgSPTQe2cYZ/5tMnTE3R5wqSWoWiM3KRDxYyU5hL76sxK1HoDWsiEViV9rghEughF9gRo8tsW6A7jbccasyrNl4xFYW1PAw7Q5Slpaql7wwVbUBmIxLOGr/YJ8F9CvwdSjQLuBAU8S/6PD SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Sep 2016 06:23:58.3588 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e; Ip=[192.88.168.50]; Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR03MB2447 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20160907_232421_924055_5093BDB5 X-CRM114-Status: GOOD ( 13.96 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Roy Zang , Arnd Bergmann , Jingoo Han , Stuart Yoder , Yang-Leo Li , Minghuan Lian , linux-arm-kernel@lists.infradead.org, Bjorn Helgaas , Mingkai Hu Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP By default, when the PCIe controller experiences an erroneous completion from an external completer for its outbound non-posted request, it always sends an OKAY response to the device's internal AXI slave system interface. However, such default system error response behavior cannot be used for other types of outbound non-posted requests. For example, the outbound memory read transaction requires an actual ERROR response, like UR completion or completion timeout. The patch is to fix it by forwarding the error response of the non-posted request. Signed-off-by: Minghuan Lian --- drivers/pci/host/pci-layerscape.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/drivers/pci/host/pci-layerscape.c b/drivers/pci/host/pci-layerscape.c index cf783ad..873447a 100644 --- a/drivers/pci/host/pci-layerscape.c +++ b/drivers/pci/host/pci-layerscape.c @@ -34,6 +34,8 @@ /* PEX Internal Configuration Registers */ #define PCIE_STRFMR1 0x71c /* Symbol Timer & Filter Mask Register1 */ #define PCIE_DBI_RO_WR_EN 0x8bc /* DBI Read-Only Write Enable Register */ +#define PCIE_ABSERR 0x8d0 /* Bridge Slave Error Response Register */ +#define PCIE_ABSERR_SETTING 0x9401 /* Forward error of non-posted request */ /* PEX LUT registers */ #define PCIE_LUT_DBG 0x7FC /* PEX LUT Debug Register */ @@ -99,6 +101,12 @@ static void ls_pcie_disable_outbound_atus(struct ls_pcie *pcie) dw_pcie_disable_outbound_atu(&pcie->pp, i); } +/* Forward error response of outbound non-posted requests */ +static void ls_pcie_fix_error_response(struct ls_pcie *pcie) +{ + iowrite32(PCIE_ABSERR_SETTING, pcie->dbi + PCIE_ABSERR); +} + static int ls1021_pcie_link_up(struct pcie_port *pp) { u32 state; @@ -166,6 +174,7 @@ static void ls_pcie_host_init(struct pcie_port *pp) iowrite32(0, pcie->dbi + PCIE_DBI_RO_WR_EN); ls_pcie_disable_outbound_atus(pcie); + ls_pcie_fix_error_response(pcie); } static int ls_pcie_msi_host_init(struct pcie_port *pp,