Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1411288 > unrolled thread

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

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH 1/2] freescale/dts: add pcie aer interrupt-name property  in the dts Rob Herring <robh@kernel.org> - 2016-06-01 16:50 +0200
    RE: [PATCH 1/2] freescale/dts: add pcie aer interrupt-name property  in the dts Po Liu <po.liu@nxp.com> - 2016-06-02 04:50 +0200

#1411288 — 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?

> 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.

Rob

[toc] | [next] | [standalone]

#1411783

>  -----Original Message-----
>  From: Rob Herring [mailto:robh@kernel.org]
>  Sent: Wednesday, June 01, 2016 10:40 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; 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

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web