linux-kernel - RE: [PATCH 1/2] freescale/dts: add pcie aer interrupt-name property in the dts

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <VI1PR0401MB1709DFD5A437FD01D724FF9F92580@VI1PR0401MB1709.eurprd04.prod.outlook.com>
Date:	Thu, 2 Jun 2016 01:17:24 +0000
From:	Po Liu <po.liu@....com>
To:	Rob Herring <robh@...nel.org>
CC:	"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	Shawn Guo <shawnguo@...nel.org>,
	Marc Zyngier <marc.zyngier@....com>,
	Roy Zang <roy.zang@....com>, Mingkai Hu <mingkai.hu@....com>,
	Stuart Yoder <stuart.yoder@....com>,
	Yang-Leo Li <leoyang.li@....com>,
	Arnd Bergmann <arnd@...db.de>,
	Bjorn Helgaas <bhelgaas@...gle.com>,
	Minghuan Lian <minghuan.lian@....com>
Subject: RE: [PATCH 1/2] freescale/dts: add pcie aer interrupt-name property
 in the dts


>  -----Original Message-----
>  From: Rob Herring [mailto:robh@...nel.org]
>  Sent: Wednesday, June 01, 2016 10:40 PM
>  To: Po Liu
>  Cc: linux-pci@...r.kernel.org; linux-arm-kernel@...ts.infradead.org;
>  linux-kernel@...r.kernel.org; devicetree@...r.kernel.org; Shawn Guo;
>  Marc Zyngier; Roy Zang; Mingkai Hu; Stuart Yoder; Yang-Leo Li; Arnd
>  Bergmann; Bjorn Helgaas; Minghuan Lian
>  Subject: Re: [PATCH 1/2] freescale/dts: add pcie aer interrupt-name
>  property in the dts
>  
>  On Thu, May 26, 2016 at 01:59:09PM +0800, Po Liu wrote:
>  > Freescale some platforms aer interrupt was not MSI/MSI-X/INTx
>  
>  some or all?

For all Layerscape platforms, that should be clear, sorry.

>  
>  > but using interrupt line independently. This patch add a "aer"
>  > interrupt-names for aer interrupt.
>  
>  You are not adding, but replacing the interrupt. This needs a better
>  explanation as you are breaking compatibility.

We used to the "intr" name for general concept, but no use. So removed.
In the Layerscape platform, the aer and pme interrupt lines are independent, 
not in the Complex Root by MSI or MSIX, so need to read from the DTS before
MSIX and MSI searching for pci service driver.
 
>  
>  Rob