Re: [PATCH v8 2/2] add new platform driver for PCI RC

On Thu, Feb 04, 2016 at 06:31:05PM +, Joao Pinto wrote: > On 2/4/2016 6:19 PM, Bjorn Helgaas wrote: > > On Thu, Feb 04, 2016 at 03:52:10PM +, Joao Pinto wrote: > >> This patch adds a new driver that will be the reference platform driver > >> for all PCI RC IP Protoyping Kits based on ARC SD

Re: [PATCH v8 2/2] add new platform driver for PCI RC

Hi Bjorn, On 2/4/2016 6:19 PM, Bjorn Helgaas wrote: > On Thu, Feb 04, 2016 at 03:52:10PM +, Joao Pinto wrote: >> This patch adds a new driver that will be the reference platform driver >> for all PCI RC IP Protoyping Kits based on ARC SDP. >> >> Signed-off-by: Joao Pinto >> --- >> Change v7 -

Re: [PATCH v8 2/2] add new platform driver for PCI RC

On Thu, Feb 04, 2016 at 03:52:10PM +, Joao Pinto wrote: > This patch adds a new driver that will be the reference platform driver > for all PCI RC IP Protoyping Kits based on ARC SDP. > > Signed-off-by: Joao Pinto > --- > Change v7 -> v8 (Bjorn Helgaas and Arnd Bergmann): > - driver name was

[PATCH v8 0/2] adding PCI support to AXS10x

This patch set has the goal to add suppport for DesignWare PCIe RC in ARC AXS10x. It includes the necessary tweaks to the ARC architecture, necessary tweaks to the PCI subsystem and a new driver (pcie-synopsys). This new driver will be used extensively in the PCIe RC Prototyping Kit. The patches

[PATCH v8 2/2] add new platform driver for PCI RC

This patch adds a new driver that will be the reference platform driver for all PCI RC IP Protoyping Kits based on ARC SDP. Signed-off-by: Joao Pinto --- Change v7 -> v8 (Bjorn Helgaas and Arnd Bergmann): - driver name was changed from pcie-synopsys to pcie-dw-pltfm - mdelay() replaced for msleep

[PATCH v8 1/2] PCI support added to ARC

This patch adds PCI support to ARC and updates drivers/pci Makefile enabling the ARC arch to use the generic PCI setup functions. Signed-off-by: Joao Pinto --- Change v7 -> v8: - Nothing changed (just to keep up with patch set version). Change v6 -> v7 (Bjorn Helgaas): - Parenthesis removed from

Re: [PATCH v7 2/2] add new platform driver for PCI RC

On Thursday 04 February 2016 14:09:26 Joao Pinto wrote: > Hi Bjorn and Arnd, > > Removing the irq_handler causes the irq request to fail because in > request_threaded_irq() both handler and thread_fn are NULL. > > What's the typical procedure for this? > Don't call request_irq? ;-) Arn

Re: [PATCH v7 2/2] add new platform driver for PCI RC

Hi Bjorn and Arnd, Removing the irq_handler causes the irq request to fail because in request_threaded_irq() both handler and thread_fn are NULL. What's the typical procedure for this? Joao On 2/4/2016 11:14 AM, Joao Pinto wrote: > Hi Bjorn, > > On 2/3/2016 6:38 PM, Bjorn Helgaas wrote: >> On

Re: [PATCH v7 2/2] add new platform driver for PCI RC

On Thursday 04 February 2016 11:10:55 Joao Pinto wrote: > > The "synopsys" can go away, it's already in the vendor field of the > > string. "ipk" is still a bit unspecific, I was hoping to see a specific > > chip and/or version of the PCIe part. Something like > > > > compatible = "snps,ipk2

Re: [PATCH v7 2/2] add new platform driver for PCI RC

Hi Bjorn, On 2/3/2016 6:38 PM, Bjorn Helgaas wrote: > On Wed, Feb 03, 2016 at 06:12:00PM +, Joao Pinto wrote: >> Hi Bjorn, >> >> Are these tasks enough? >> >> - replace mdelay() for msleep() >> - remove synopsys_pcie_irq_handler() > > Above are fine with me. Ok, I'll to that. Thanks. _

Re: [PATCH v7 2/2] add new platform driver for PCI RC

Hi Arnd and Bjorn, On 2/3/2016 9:01 PM, Arnd Bergmann wrote: > On Wednesday 03 February 2016 12:38:44 Bjorn Helgaas wrote: >> On Wed, Feb 03, 2016 at 06:12:00PM +, Joao Pinto wrote: >> >>> - replace "snps,pcie-synopsys" for "snps,pcie-synopsys-ipk"? >> >> This is a question for Arnd. >> >>> -