From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755242AbcIEOkZ (ORCPT ); Mon, 5 Sep 2016 10:40:25 -0400 Received: from mail-db5eur01on0048.outbound.protection.outlook.com ([104.47.2.48]:27062 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751245AbcIEOkT (ORCPT ); Mon, 5 Sep 2016 10:40:19 -0400 X-Greylist: delayed 10760 seconds by postgrey-1.27 at vger.kernel.org; Mon, 05 Sep 2016 10:40:18 EDT From: Po Liu To: Rob Herring CC: "linux-pci@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Bjorn Helgaas , Shawn Guo , Marc Zyngier , Roy Zang , Mingkai Hu , Stuart Yoder , Yang-Leo Li , Arnd Bergmann , Minghuan Lian , Murali Karicheri Subject: RE: [PATCH v4 2/2] pci:aer: add support aer interrupt with none MSI/MSI-X/INTx mode Thread-Topic: [PATCH v4 2/2] pci:aer: add support aer interrupt with none MSI/MSI-X/INTx mode Thread-Index: AQHSA1QFxtx3Y9yuBUCYqYDkWBXTnKBmU5CAgAQabIA= Date: Mon, 5 Sep 2016 06:05:49 +0000 Message-ID: References: <1465892645-32381-2-git-send-email-po.liu@nxp.com> <1472625442-23309-1-git-send-email-po.liu@nxp.com> <1472625442-23309-2-git-send-email-po.liu@nxp.com> <20160902151725.GA25273@rob-hp-laptop> In-Reply-To: <20160902151725.GA25273@rob-hp-laptop> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: spf=none (sender IP is ) smtp.mailfrom=po.liu@nxp.com; x-originating-ip: [199.59.230.102] x-ms-office365-filtering-correlation-id: d8456b62-a65e-4e18-06f9-08d3d552afaf x-microsoft-exchange-diagnostics: 1;VI1PR0401MB2639;6:R0xDDyUfY9rgNpucEGTLIzbZswGPfAPbbpxD5tKsoC5Op3OtiCFsX0HRdd/ZUuInRv2QSkc9nACBLsKBF8WVVD1wA2EAKjqHMWUwCr/L7l07+L6ZiwLyoXS+JADWRqLT1ZMw3foc/5tY4B+fmPCa/wtvx99E4cRs/LdZPix/hMKVaGBSq99Z02PecaIA6CzURXcGT/MPoYA6M4OMKKTAadWnwK474D0cGopn48kTtcAjI3ssB/8EbfaCud3RUQcFutJiJ6fWYsyX+ZRE6wfEdon45imY4070ezfvBSSIgE8NMC4PESfrcASmDRlR05WSha6rxDRz1rOeb53Q8K4qOg==;5:e6lLJi/UbmOqS8zND5YnsuN3RijFSADKBMYlI9/xGvNS84bsqrd8Ir6MKDXNR0uXaGW4S8kW5FQDYjvFxJ+fPVqyaTV17kMW1y8eg8ztTgMsSy5fyRzztpl6x2wJoauDPxm64/iYf4+LRmh0HP+ekQ==;24:SIEf6he9SI08ah8Fc4SFXpZMGw4DJIuJ79EQ3Aad0m8F6aYKVuBsNBPNtP176sFEcxTdi59plsQEWXP7VU4wL/S5Z+jIhBE2Ad0EXYl3P0k=;7:yO8iYVMy64VmZoyhvtEwpZmGobm+XqCJplGqOyfr6OS3GZIsKqiPA30qe9Xfgz7Laoaz1Mds7CxOEr6waDJrao1j6Erc3ZgzrqISBx334yo9lh2zJHao/LNtlaMJYOijy0RHgaQmF2PgwQ9SB3ckys7/4tP+D1NinOWgM3NCCUw+5MCLzFAZeNie7QBc8Y0lddD4ou6Xv+NWNaSGcjwdkuPNwRFjoSA8WRiXmWMPGCRtth8foQRSAbAL4W13BI9H x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:VI1PR0401MB2639; x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(9452136761055)(185117386973197)(258649278758335); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(6040176)(601004)(2401047)(5005006)(8121501046)(3002001)(10201501046)(6055026);SRVR:VI1PR0401MB2639;BCL:0;PCL:0;RULEID:;SRVR:VI1PR0401MB2639; x-forefront-prvs: 005671E15D x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(7916002)(199003)(189002)(377454003)(13464003)(24454002)(2950100001)(93886004)(2900100001)(74316002)(9686002)(8676002)(3280700002)(3660700001)(50986999)(6116002)(106356001)(3846002)(81166006)(11100500001)(81156014)(105586002)(8936002)(19580395003)(2906002)(102836003)(101416001)(7846002)(110136002)(87936001)(97736004)(5002640100001)(189998001)(7696003)(19580405001)(86362001)(5660300001)(54356999)(7416002)(122556002)(92566002)(10400500002)(4326007)(7736002)(68736007)(66066001)(76176999)(76576001)(586003)(77096005)(305945005)(33656002)(106116001);DIR:OUT;SFP:1101;SCL:1;SRVR:VI1PR0401MB2639;H:VI1PR0401MB1709.eurprd04.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Sep 2016 06:05:49.8758 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0401MB2639 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u85EeTj8000404 Hi Rob, Best regards, Liu Po > -----Original Message----- > From: Rob Herring [mailto:robh@kernel.org] > Sent: Friday, September 02, 2016 11:17 PM > To: Po Liu > Cc: linux-pci@vger.kernel.org; linux-arm-kernel@lists.infradead.org; > linux-kernel@vger.kernel.org; devicetree@vger.kernel.org; Bjorn Helgaas; > Shawn Guo; Marc Zyngier; Roy Zang; Mingkai Hu; Stuart Yoder; Yang-Leo Li; > Arnd Bergmann; Minghuan Lian; Murali Karicheri > Subject: Re: [PATCH v4 2/2] pci:aer: add support aer interrupt with none > MSI/MSI-X/INTx mode > > On Wed, Aug 31, 2016 at 02:37:22PM +0800, Po Liu wrote: > > On some platforms, root port doesn't support MSI/MSI-X/INTx in RC mode. > > When chip support the aer interrupt with none MSI/MSI-X/INTx mode, > > maybe there is interrupt line for aer pme etc. Search the interrupt > > number in the fdt file. Then fixup the dev->irq with it. > > > > Signed-off-by: Po Liu > > --- > > Changes for v4: > > - re-use the patch changes in the root port driver; > > - add binding information; > > > > .../devicetree/bindings/pci/layerscape-pci.txt | 4 +-- > > drivers/pci/pcie/portdrv_core.c | 31 > +++++++++++++++++++--- > > 2 files changed, 30 insertions(+), 5 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/pci/layerscape-pci.txt > > b/Documentation/devicetree/bindings/pci/layerscape-pci.txt > > index 41e9f55..1dfb1da 100644 > > --- a/Documentation/devicetree/bindings/pci/layerscape-pci.txt > > +++ b/Documentation/devicetree/bindings/pci/layerscape-pci.txt > > @@ -19,7 +19,7 @@ Required properties: > > - interrupts: A list of interrupt outputs of the controller. Must > contain an > > entry for each entry in the interrupt-names property. > > - interrupt-names: Must include the following entries: > > - "intr": The interrupt that is asserted for controller interrupts > > + "aer": The interrupt that is asserted for aer interrupts > > You can't just change this. That breaks compatibility with old dts files. > Plus, it is just a name. Why does it even matter? Not to mention having > the name for a single irq is a bit pointless. Sorry, this binding comments maybe not clear. "aer" is just one of the interrupt names from host controller. There are "aer" "pme" and so on for further applications. This patch is to fix the Layerscape owning independent aer irq issue. So the 'aer' is used for this patch. The 'intr' will never be used, so removed. I may remove the 'must include' for mis-understanding. Thanks! Po Liu > > Rob