Re: [PATCH] pci: Add and use PCI_GENERIC_SETUP Kconfig entry

2017-06-27 Thread Bjorn Helgaas
[+cc Lorenzo] Hi Palmer, On Fri, Jun 23, 2017 at 02:45:38PM -0700, Palmer Dabbelt wrote: > We wanted to add RISC-V to the list of architectures that used the > generic PCI setup-irq.o inside the Makefile and it was suggested that > instead we define a Kconfig entry and use that. > > I've done ve

Re: alpha runtime warnings due to commit 0e4c2eeb758 ("alpha/PCI: Replace pci_fixup_irqs() call with host ...")

2017-09-27 Thread Bjorn Helgaas
[+cc linux-pci, linux-ide, DaveM] On Wed, Sep 27, 2017 at 11:30:34AM +0100, Lorenzo Pieralisi wrote: > Bjorn, Guenter, > > On Wed, Sep 20, 2017 at 12:31:04PM +0100, Lorenzo Pieralisi wrote: > > On Tue, Sep 19, 2017 at 11:02:33AM -0700, Guenter Roeck wrote: > > > On Tue, Sep 19, 2017 at 10:12:32AM

Re: alpha runtime warnings due to commit 0e4c2eeb758 ("alpha/PCI: Replace pci_fixup_irqs() call with host ...")

2017-09-28 Thread Bjorn Helgaas
On Thu, Sep 28, 2017 at 10:25:33AM +0100, Lorenzo Pieralisi wrote: > On Wed, Sep 27, 2017 at 02:55:02PM -0500, Bjorn Helgaas wrote: > > On Wed, Sep 27, 2017 at 11:30:34AM +0100, Lorenzo Pieralisi wrote: > > > On Wed, Sep 20, 2017 at 12:31:04PM +0100, Lorenzo Pieralisi wrote: >

[PATCH 0/4] PCI: Cleanup unused stuff

2017-10-05 Thread Bjorn Helgaas
Sorry for the long cc list. These are pretty trivial; they just remove some unnecessary declarations across several arches. --- Bjorn Helgaas (4): PCI: Remove redundant pcibios_set_master() declarations PCI: Remove redundant pci_dev, pci_bus, resource declarations PCI: Remove

[PATCH 2/4] PCI: Remove redundant pci_dev, pci_bus, resource declarations

2017-10-05 Thread Bjorn Helgaas
From: Bjorn Helgaas defines struct pci_bus and struct pci_dev and includes the struct resource definition before including . Nobody includes directly, so they don't need their own declarations. Remove the redundant struct pci_dev, pci_bus, resource declarations. Signed-off-by: Bjorn He

[PATCH 1/4] PCI: Remove redundant pcibios_set_master() declarations

2017-10-05 Thread Bjorn Helgaas
From: Bjorn Helgaas All users of pcibios_set_master() include , which already has a declaration. Remove the unnecessary declarations from the files. Signed-off-by: Bjorn Helgaas --- arch/alpha/include/asm/pci.h |2 -- arch/cris/include/asm/pci.h|1 - arch/frv/include/asm/pci.h

[PATCH 3/4] PCI: Remove unused declarations

2017-10-05 Thread Bjorn Helgaas
From: Bjorn Helgaas Remove these unused declarations: pcibios_config_init() # never defined anywhere pcibios_scan_root()# only defined by x86 pcibios_get_irq_routing_table()# only defined by x86 pcibios_set_irq_routing() # only defined by x86

[PATCH 4/4] alpha/PCI: Make pdev_save_srm_config() static

2017-10-05 Thread Bjorn Helgaas
From: Bjorn Helgaas pdev_save_srm_config() and struct pdev_srm_saved_conf are only used in arch/alpha/kernel/pci.c, so make them static there. Signed-off-by: Bjorn Helgaas --- arch/alpha/kernel/pci.c | 11 ++- arch/alpha/kernel/pci_impl.h |8 2 files changed, 10

Re: [PATCH] alpha/pci: Fix pci_map_irq()/pci_swizzle() functions section markers

2017-10-26 Thread Bjorn Helgaas
t; Fixes: 0e4c2eeb758a ("alpha/PCI: Replace pci_fixup_irqs() call with host > bridge IRQ mapping hooks") > Link: http://lkml.kernel.org/r/alpine.lrh.2.21.1710251043170.7...@math.ut.ee > Reported-by: Meelis Roos > Signed-off-by: Lorenzo Pieralisi > Cc: Bjorn Helgaas > Cc: Ri

Re: the patch "alpha/PCI: Replace pci_fixup_irqs()" breaks networking

2018-01-02 Thread Bjorn Helgaas
[+cc linux-pci] Thanks for the report! Lorenzo has been on vacation and is probably buried in email, but I'm sure he'll be on this as soon as he digs out. On Tue, Jan 2, 2018 at 3:32 PM, Mikulas Patocka wrote: > Hi > > The patch 0e4c2eeb758a91e68b9eaf7a4bee9bd5ed97ff2b ("alpha/PCI: Replace > pc

Re: Some Alphas broken by f75b99d5a77d (PCI: Enforce bus address limits in resource allocation)

2018-04-16 Thread Bjorn Helgaas
Hi Matt, First of all, sorry about breaking Nautilus, and thanks very much for tracking it down to this commit. On Mon, Apr 16, 2018 at 07:33:57AM -0700, Matt Turner wrote: > Commit f75b99d5a77d63f20e07bd276d5a427808ac8ef6 (PCI: Enforce bus > address limits in resource allocation) broke Alpha sys

Re: Some Alphas broken by f75b99d5a77d (PCI: Enforce bus address limits in resource allocation)

2018-04-17 Thread Bjorn Helgaas
On Mon, Apr 16, 2018 at 09:43:42PM -0700, Matt Turner wrote: > On Mon, Apr 16, 2018 at 2:50 PM, Bjorn Helgaas wrote: > > Hi Matt, > > > > First of all, sorry about breaking Nautilus, and thanks very much for > > tracking it down to this commit. > > It's a p

Re: Some Alphas broken by f75b99d5a77d (PCI: Enforce bus address limits in resource allocation)

2018-04-20 Thread Bjorn Helgaas
On Wed, Apr 18, 2018 at 09:48:08PM +0100, Ivan Kokshaysky wrote: > On Tue, Apr 17, 2018 at 02:43:44PM -0500, Bjorn Helgaas wrote: > > On Mon, Apr 16, 2018 at 09:43:42PM -0700, Matt Turner wrote: > > > On Mon, Apr 16, 2018 at 2:50 PM, Bjorn Helgaas wrote: > > > > Hi

Re: Some Alphas broken by f75b99d5a77d (PCI: Enforce bus address limits in resource allocation)

2018-05-02 Thread Bjorn Helgaas
On Mon, Apr 23, 2018 at 06:34:23PM +0100, Ivan Kokshaysky wrote: > On Sun, Apr 22, 2018 at 01:07:38PM -0700, Matt Turner wrote: > > On Wed, Apr 18, 2018 at 1:48 PM, Ivan Kokshaysky > > wrote: > > > On Tue, Apr 17, 2018 at 02:43:44PM -0500, Bjorn Helgaas wrote: > > &g

Re: [PATCH RESEND v3 00/26] Add definition for the number of standard PCI BARs

2019-09-30 Thread Bjorn Helgaas
On Sat, Sep 28, 2019 at 02:40:26AM +0300, Denis Efremov wrote: > Code that iterates over all standard PCI BARs typically uses > PCI_STD_RESOURCE_END, but this is error-prone because it requires > "i <= PCI_STD_RESOURCE_END" rather than something like > "i < PCI_STD_NUM_BARS". We could add such a de