[-cc Oza, bounced]
On Mon, Sep 30, 2019 at 11:46:18PM +0200, Marek Vasut wrote:
> On 9/30/19 11:40 PM, Bjorn Helgaas wrote:
> > This would follow the convention for subject lines:
> >
> > PCI: OF: Add of_pci_get_dma_ranges() for inbound DMA restrictions
> >
> &
On Fri, Aug 09, 2019 at 07:34:49PM +0200, marek.va...@gmail.com wrote:
> From: Oza Pawandeep
>
> current device framework and OF framework integration assumes
> dma-ranges in a way where memory-mapped devices define their
> dma-ranges. (child-bus-address, parent-bus-address, length).
"Memory-map
This would follow the convention for subject lines:
PCI: OF: Add of_pci_get_dma_ranges() for inbound DMA restrictions
On Fri, Aug 09, 2019 at 07:34:48PM +0200, marek.va...@gmail.com wrote:
> From: Oza Pawandeep
>
> The patch exports interface to PCIe RC drivers so that,
> Drivers can get thei
On Mon, May 27, 2019 at 12:51:51AM +0200, marek.va...@gmail.com wrote:
> From: Marek Vasut
>
> On ARM64 R-Car Gen3 R8A7795 system with Intel NVMe SSD inserted into the
> PCIe slot, with CONFIG_PROVE_LOCKING=y enabled in the kernel config, the
> following lockdep warning can be triggered:
>
> $
On Thu, Mar 28, 2019 at 03:59:11PM +0100, Geert Uytterhoeven wrote:
> On Thu, Mar 28, 2019 at 3:18 PM Bjorn Helgaas wrote:
> > On Mon, Mar 25, 2019 at 08:43:19PM +0100, marek.va...@gmail.com wrote:
> > > From: Kazufumi Ikeda
> > >
> > > Reestablish the PCIe l
On Mon, Mar 25, 2019 at 08:43:19PM +0100, marek.va...@gmail.com wrote:
> From: Kazufumi Ikeda
>
> Reestablish the PCIe link very early in the resume process in case it
> went down to prevent PCI accesses from hanging the bus. Such accesses
> can happen early in the PCI resume process, as early as
[+cc linux-pm, Rafael for SET_NOIRQ_SYSTEM_SLEEP_PM_OPS question at the end]
On Thu, Mar 07, 2019 at 11:49:34PM +0100, Marek Vasut wrote:
> On 3/7/19 9:50 PM, Bjorn Helgaas wrote:
> > On Sun, Feb 17, 2019 at 02:24:41PM +0100, marek.va...@gmail.com wrote:
> >> Fr
On Sun, Feb 17, 2019 at 02:24:41PM +0100, marek.va...@gmail.com wrote:
> From: Kazufumi Ikeda
>
> Reestablish the PCIe link very early in the resume process in case it
> went down to prevent PCI accesses from hanging the bus. Such accesses
> can happen early in the PCI resume process, in the resu
[+cc Bart, Tejun]
On Tue, Nov 06, 2018 at 12:25:00AM +0100, Marek Vasut wrote:
> From: Tho Vu
>
> This patch fixes deadlock warning in removing/rescanning through sysfs
> when CONFIG_PROVE_LOCKING is enabled.
>
> The issue can be reproduced by these steps:
> 1. Enable CONFIG_PROVE_LOCKING via d
Semiconductor PCI host driver")
> Fixes: b7e78170efd4 ("PCI: versatile: Add DT-based ARM Versatile PB PCIe host
> driver")
> Fixes: 5f6b6ccdbe1c ("PCI: xgene: Add APM X-Gene PCIe driver")
> Fixes: 637cfacae96f ("PCI: mediatek: Add MediaTek PCIe host c
On Wed, Jun 13, 2018 at 04:52:52PM +0100, Lorenzo Pieralisi wrote:
> On Wed, Jun 13, 2018 at 08:53:08AM -0500, Bjorn Helgaas wrote:
> > On Wed, Jun 13, 2018 at 01:54:51AM +0200, Marek Vasut wrote:
> > > On 06/11/2018 03:59 PM, Bjorn Helgaas wrote:
> > > > On Sun, Ju
On Wed, Jun 13, 2018 at 01:54:51AM +0200, Marek Vasut wrote:
> On 06/11/2018 03:59 PM, Bjorn Helgaas wrote:
> > On Sun, Jun 10, 2018 at 03:57:10PM +0200, Marek Vasut wrote:
> >> On 11/17/2017 06:49 PM, Lorenzo Pieralisi wrote:
> >>> On Fri, Nov 10, 2017 at 10:58
On Sun, Jun 10, 2018 at 03:57:10PM +0200, Marek Vasut wrote:
> On 11/17/2017 06:49 PM, Lorenzo Pieralisi wrote:
> > On Fri, Nov 10, 2017 at 10:58:42PM +0100, Marek Vasut wrote:
> >> From: Phil Edworthy
> >>
> >> Most PCIe host controllers support L0s and L1 power states via ASPM.
> >> The R-Car ha
On Wed, May 23, 2018 at 07:05:06PM +0200, Marek Vasut wrote:
> On 05/23/2018 06:17 PM, Lorenzo Pieralisi wrote:
> > On Mon, May 21, 2018 at 03:11:20PM +0200, Marek Vasut wrote:
> >> The function name is just too confusing, rename it, no functional change.
> >> Rename the function to rcar_pcie_alloc
On Mon, Dec 11, 2017 at 01:30:37PM +, Biju Das wrote:
> Hi Bjorn Helgaas,
>
> Does this patch look okay to you?
It's OK by me, but Lorenzo (cc'd) would handle this now.
> > -Original Message-
> > From: Biju Das [mailto:biju@bp.renesas.com]
> >
On Thu, Dec 07, 2017 at 11:15:19AM +0100, Geert Uytterhoeven wrote:
> If CONFIG_DEBUG_SLAB=y, and no PCIe card is inserted, the kernel crashes
> during probe on r8a7791/koelsch:
>
> rcar-pcie fe00.pcie: PCIe link down
> Unable to handle kernel paging request at virtual address 6b6b6b6b
On Thu, Dec 07, 2017 at 11:15:18AM +0100, Geert Uytterhoeven wrote:
> Hi Simon, Lorenzo, Bjorn,
>
> This patch series fixes two issues in the error path for the R-Car PCIe
> host bridge driver.
>
> The first issue is triggered easily by not having a PCIe card inserted,
> and may cause a cra
On Wed, Oct 11, 2017 at 03:50:13PM +0200, Geert Uytterhoeven wrote:
> Correct the USB subnodes in the example, cfr. commit f7d569c1e6a6fa73
> ("ARM: dts: r8a779x: Fix PCI bus dtc warnings"):
> 1. Drop the bogus 'device_type = "pci"' properties,
> 2. Correct the unit addresses.
>
> Update other
On Thu, Aug 24, 2017 at 10:35:44AM +0100, Biju Das wrote:
> Add internal PCI bridge support for r8a7743/5 SoC. Renesas RZ/G1[ME]
> (R8A7743/5) internal PCI bridge is identical to the R-Car Gen2 family.
>
> Signed-off-by: Biju Das
Applied with Simon's ack to pci/host-rcar, thanks!
> ---
> This p
On Fri, Aug 04, 2017 at 12:32:53PM +0900, Harunobu Kurokawa wrote:
> When no PCIe card is inserted, there is a memory leak as
> pci_free_resource_list is not called before returning.
>
> v2:
> separate the patch to two files.
>
> Harunobu Kurokawa (1):
> PCI: rcar-pcie: Fix memory leak when no
On Fri, Aug 04, 2017 at 12:32:53PM +0900, Harunobu Kurokawa wrote:
> When no PCIe card is inserted, there is a memory leak as
> pci_free_resource_list is not called before returning.
>
> v2:
> separate the patch to two files.
>
> Harunobu Kurokawa (1):
> PCI: rcar-pcie: Fix memory leak when no
On Fri, Jun 23, 2017 at 02:59:52PM +0530, Arvind Yadav wrote:
> of_device_ids are not supposed to change at runtime. All functions
> working with of_device_ids provided by work with const
> of_device_ids. So mark the non-const structs as const.
>
> Signed-off-by: Arvind Yadav
Applied with acks
On Sun, May 28, 2017 at 11:30:48AM +0200, Wolfram Sang wrote:
> It is 'R-Car', not 'RCar'. No code or binding changes, only descriptive text.
>
> Signed-off-by: Wolfram Sang
> ---
> I suggest this trivial patch should be picked individually per susbsystem.
>
> Documentation/devicetree/bindings/
On Tue, Apr 18, 2017 at 08:44:30PM +0200, Lukas Wunner wrote:
> Laurent Pinchart reported that the Renesas R-Car H2 Lager board
> (r8a7790) crashes during suspend tests. Geert Uytterhoeven managed to
> reproduce the issue on an M2-W Koelsch board (r8a7791):
>
> It occurs when the PME scan runs, o
On Tue, Jan 31, 2017 at 02:19:51PM -0600, Bjorn Helgaas wrote:
> This turned into much ado about nothing, but Shailendra's hisi patch
> prompted similar cleanups in other drivers. It's a minor simplification,
> but I think worth doing consistently across drivers.
>
>
ant to
return -EINVAL, which is easy to ignore. We'd rather take the NULL pointer
dereference so we notice the problem and fix it.
Use of_device_get_match_data() to retrieve the hw_init_fn pointer. No
functional change intended.
Suggested-by: Geert Uytterhoeven
Signed-off-by: Bjorn Helgaas
---
The only way to call iproc_pcie_pltfm_probe() is to match an entry in
iproc_pcie_of_match_table[], so match cannot be NULL.
Use of_device_get_match_data() to retrieve the pcie->type. No functional
change intended.
Based-on-suggestion-from: Geert Uytterhoeven
Signed-off-by: Bjorn Helg
-suggestion-from: Geert Uytterhoeven
Signed-off-by: Shailendra Verma
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-hisi.c |8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/pci/host/pcie-hisi.c b/drivers/pci/host/pcie-hisi.c
index a301a7187b30
The only way to call ls_pcie_probe() is to match an entry in
ls_pcie_of_match[], so match cannot be NULL.
Use of_device_get_match_data() to retrieve the drvdata pointer. No
functional change intended.
Based-on-suggestion-from: Geert Uytterhoeven
Signed-off-by: Bjorn Helgaas
---
drivers/pci
nly as a result of matching something in , a
subsequent of_match_device(, dev) call should never fail.
Therefore, instead of this:
struct of_device_id *match;
match = of_match_device(, dev);
... = match->data;
we can do this:
... = of_device_get_match_data(dev);
---
Bjorn Helg
On Tue, Jan 31, 2017 at 04:33:15PM +0100, Geert Uytterhoeven wrote:
> Hi Bjorn,
>
> On Tue, Jan 31, 2017 at 4:10 PM, Bjorn Helgaas wrote:
> > A match in the rcar_pcie_of_match[] table is required, so check that first,
> > before we start setting up things that need to be un
A match in the rcar_pcie_of_match[] table is required, so check that first,
before we start setting up things that need to be undone if it fails. No
functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c | 10 +-
1 file changed, 5 insertions(+), 5
On Wed, Dec 21, 2016 at 03:37:06AM +0900, Yoshihiro Kaneko wrote:
> From: Harunobu Kurokawa
>
> This patch adds support for r8a7796.
>
> Signed-off-by: Harunobu Kurokawa
> Signed-off-by: Yoshihiro Kaneko
> Reviewed-by: Geert Uytterhoeven
Applied to pci/host-rcar for v4.11 with acks from Simo
obe when no card present
R-Car PCIe does not support hotplug so it is appropriate to treat the
absence of a PCIe card as an -ENODEV error.
Signed-off-by: Harunobu Kurokawa
[simon: updated changelog]
Signed-off-by: Simon Horman
Signed-off-by: Bjorn Helgaas
On Tue, Dec 06, 2016 at 04:51:28PM +0100, Simon Horman wrote:
> Hi,
>
> this short series makes some bindings cleanups to the Renesas PCI drivers.
>
> Changes v2->v3:
> * Reworded changelogs to indicate that re-ordering struct of_device_id
> entries does not effect run-time behaviour
> * Correc
On Wed, Oct 12, 2016 at 07:58:35AM -0500, Bjorn Helgaas wrote:
> - Add local "dev" pointers to reduce repetition of things like
> "&pdev->dev".
>
> - Remove platform drvdata because it appears unused (we called
> platform_set_drvdata() but no
reg/val arguments
PCI: rcar-gen2: Name private struct pointer "rcar" consistently
PCI: rcar-gen2: Add register accessors
---
Bjorn Helgaas (5):
PCI: rcar: Add local struct device pointers
PCI: rcar: Remove unused platform data
PCI: rcar: Remove unused rcar_pci
The DRV_NAME macro is only used once, so there's no real advantage to
having the macro at all. Remove it and use the "rcar-pcie" name directly
in the struct platform_driver. No functional change intended.
Signed-off-by: Bjorn Helgaas
Acked-by: Simon Horman
---
drivers/pci/h
Use a local "struct device *dev" for brevity and consistency with other
drivers. No functional change intended.
Signed-off-by: Bjorn Helgaas
Acked-by: Simon Horman
---
drivers/pci/host/pcie-rcar.c | 88 ++
1 file changed, 47 inserti
Use a local "struct device *dev" for brevity and consistency with other
drivers. No functional change intended.
Signed-off-by: Bjorn Helgaas
Acked-by: Simon Horman
---
drivers/pci/host/pci-rcar-gen2.c | 46 +-
1 file changed, 25 inserti
The rcar driver never uses the platform drvdata pointer, so don't bother
setting it. No functional change intended.
Signed-off-by: Bjorn Helgaas
Acked-by: Simon Horman
---
drivers/pci/host/pcie-rcar.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/pci/host/pcie-rcar.c b/dr
rcar_pcie_get_resources() doesn't use the platform_device pointer passed to
it, so remove it. No functional change intended.
Signed-off-by: Bjorn Helgaas
Acked-by: Simon Horman
---
drivers/pci/host/pcie-rcar.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/dr
Use a local "struct device *dev" for brevity and consistency with other
drivers. No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-rcar-gen2.c | 47 +-
1 file changed, 26 insertions(+), 21 deletions(-)
di
The rcar driver never uses the platform drvdata pointer, so don't bother
setting it. No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/pci/host/pcie-rcar.c b/drivers/pci/host/pcie-r
Rename struct rcar_pci_priv to rcar_pci, along the lines of other drivers.
Use a device-specific name, "rcar", for struct rcar_pci pointers
to hint that this is device-specific information. No functional change
intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-rcar-gen
Add device-specific register accessors for consistency across host drivers.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-rcar-gen2.c | 61 ++
1 file changed, 35 insertions(+), 26 deletions(-)
diff --git a/drivers/pci
The DRV_NAME macro is only used once, so there's no real advantage to
having the macro at all. Remove it and use the "rcar-pcie" name directly
in the struct platform_driver. No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c |
rcar_pcie_get_resources() doesn't use the platform_device pointer passed to
it, so remove it. No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/pci/host/pcie-rca
Swap order of rcar_writel() arguments to match the "dev, pos, val" order
used by pci_write_config_word() and other drivers. No functional change
intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c | 90 +-
1 file c
Use a local "struct device *dev" for brevity and consistency with other
drivers. No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c | 92 +-
1 file changed, 47 insertions(+), 45 deletions(-)
di
Use a device-specific name, "rcar", for struct rcar_pcie pointers
to hint that this is device-specific information. No functional change
intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c | 388 +-
1 file changed, 194
Rename rcar_pci_read_reg() to rcar_readl() and rcar_pci_write_reg() to
rcar_writel() for consistency with other drivers. No functional change
intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c | 142 --
1 file changed, 68 insertions
ff-by: Sergei Shtylyov
I addressed Simon's comment for you, added his ack, and applied this to
pci/host-rcar for v4.9.
> ---
> The patch is against the 'next' branch of Bjorn Helgaas' 'pci.git' repo.
>
> Changes in version 2:
> - switched from the inte
nts not starting on a tabstop.
>
> Signed-off-by: Sergei Shtylyov
Applied with Simon's ack to pci/host-rcar, thanks!
> ---
> The patch is against the 'next' branch of Bjorn Helgaas' 'pci.git' repo plus
> two patches posted earlier...
>
> dri
eordered the variable declarations, reworded the summary/description.]
>
> Signed-off-by: Grigory Kletsko
> Signed-off-by: Sergei Shtylyov
Applied with Simon's ack to pci/host-rcar for v4.9, thanks!
> ---
> The patch is against the 'next' branch of Bjorn Helgaas' 'p
_rwm32() calls done to set the bits, removed
> unneeded braces, removed non-informative comment, reworded the patch
> summary/description.]
>
> Signed-off-by: Grigory Kletsko
> Signed-off-by: Sergei Shtylyov
>
> ---
> The patch is against the 'next' branch
On Wed, Aug 31, 2016 at 11:28:22AM +0200, Geert Uytterhoeven wrote:
> If clk_prepare_enable() fails, we must not call clk_disable_unprepare()
> in the error path.
>
> Signed-off-by: Geert Uytterhoeven
Applied to pci/host-rcar for v4.9, thanks!
> ---
> drivers/pci/host/pcie-rcar.c | 4 ++--
> 1
Move the devm_ioremap_resource() of R-Car register space next to the
of_address_to_resource() that extracts the resource. No functional change
intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git
i-ow...@vger.kernel.org [mailto:linux-pci-
> > ow...@vger.kernel.org] On Behalf Of Lorenzo Pieralisi
> > Sent: 21 July 2016 10:35
> > To: Phil Edworthy
> > Cc: Bjorn Helgaas ; linux-...@vger.kernel.org; linux-
> > renesas-...@vger.kernel.org
> > Subject: Re: [RFC] pci: Prov
, and we
should not add fictitious spaces for them.
If a caller sets struct hw_pci.io_optional, assume it is responsible for
adding any I/O port resource it desires, and do not add any default I/O
port space.
Signed-off-by: Bjorn Helgaas
---
arch/arm/include/asm/mach/pci.h |1 +
arch/ar
Drop the unused dummy I/O port region and set struct hw_pci.io_optional so
the ARM PCI code doesn't add a default one for us.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-rcar-gen2.c | 11 +--
1 file changed, 1 insertion(+), 10 deletions(-)
diff --git a/drivers/pci/hos
ries at [1], where they should
fix a resource conflict [2] in the R-Car driver.
[1]
http://lkml.kernel.org/r/20160606225630.20936.77349.st...@bhelgaas-glaptop2.roam.corp.google.com
[2]
http://lkml.kernel.org/r/camuhmdvglftd-bjrz+gvkgye8dkdzzq6dnytkus+jvpthjs...@mail.gmail.com
---
Bjorn
On Tue, Jun 21, 2016 at 06:41:00PM +0300, Valentine Barshak wrote:
> On Tue, Jun 21, 2016 at 09:26:23AM -0500, Bjorn Helgaas wrote:
> > [+cc Valentine]
> >
>
> Hi Bjorn,
>
> > Hi Geert,
> >
> > Thanks a lot for testing this, and sorry for the breaka
On Tue, Jun 21, 2016 at 07:58:08PM +0800, wangyijing wrote:
> Hi Bjorn, use devm_request_resource() for host bridge resource is cool,
> what about do the similar change for x86, now we request host bridge resource
> in pci_acpi_root_add_resources() in x86, and we would release the host bridge
> res
On Mon, Jun 20, 2016 at 06:22:40PM +0100, Lorenzo Pieralisi wrote:
> On Mon, Jun 20, 2016 at 09:56:45AM -0700, Tyler Baker wrote:
> > Hi Bjorn,
> >
> > On 6 June 2016 at 16:06, Bjorn Helgaas wrote:
> > > Previously we allocated the PCI resource list in
> > &
[+cc Valentine]
Hi Geert,
Thanks a lot for testing this, and sorry for the breakage.
On Tue, Jun 21, 2016 at 12:41:31PM +0200, Geert Uytterhoeven wrote:
> On Tue, Jun 7, 2016 at 1:07 AM, Bjorn Helgaas wrote:
> > Request host bridge window resources so they appear in ioport_res
On Mon, Jun 06, 2016 at 06:04:44PM -0500, Bjorn Helgaas wrote:
> Several host bridge drivers (designware and all derivatives, iproc,
> xgene, xilinx, and xilinx-nwl) don't request the MMIO and I/O port
> windows they forward downstream to the PCI bus.
>
> That means the PC
On Tue, Jun 07, 2016 at 08:11:05AM -0500, Bjorn Helgaas wrote:
> On Tue, Jun 07, 2016 at 10:21:36AM +0200, Arnd Bergmann wrote:
> > On Monday, June 6, 2016 6:04:44 PM CEST Bjorn Helgaas wrote:
> > > Several host bridge drivers (designware and all derivatives, iproc,
> > >
On Tue, Jun 07, 2016 at 03:25:46PM +0200, Arnd Bergmann wrote:
> On Tuesday, June 7, 2016 8:11:05 AM CEST Bjorn Helgaas wrote:
> > >
> > > What do you think is the correct behavior here, should the driver only
> > > request the PIO range with parent=iopo
On Tue, Jun 07, 2016 at 10:21:36AM +0200, Arnd Bergmann wrote:
> On Monday, June 6, 2016 6:04:44 PM CEST Bjorn Helgaas wrote:
> > Several host bridge drivers (designware and all derivatives, iproc,
> > xgene, xilinx, and xilinx-nwl) don't request the MMIO and I/O port
>
t into -next for a while.
This is on my pci/host-request-windows branch, which you can pull or view
at
https://git.kernel.org/cgit/linux/kernel/git/helgaas/pci.git/log/?h=pci/host-request-windows
---
Bjorn Helgaas (25):
PCI: Add devm_request_pci_bus_resources()
PCI: designwa
of_pci_get_host_bridge_resources() allocates a list of resources for host
bridge windows. If we fail after allocating that list, free it before we
return error.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-designware.c | 24 +---
1 file changed, 17 insertions
Request host bridge window resources so they appear in ioport_resource and
iomem_resource and are reflected in /proc/ioports and /proc/iomem.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-iproc.c |4
1 file changed, 4 insertions(+)
diff --git a/drivers/pci/host/pcie-iproc.c b
Several host bridge drivers iterate through the list of bridge windows to
request resources. Several others don't request the window resources at
all.
Add a devm_request_pci_bus_resources() interface to make it easier for
drivers to request all the window resources.
Signed-off-by: Bjorn He
The switch is the only statement in the resource_list_for_each_entry()
loop, so remove unnecessary "continue" statements in the switch.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-designware.c |6 +-
1 file changed, 1 insertion(+), 5
Request host bridge window resources so they appear in ioport_resource and
iomem_resource and are reflected in /proc/ioports and /proc/iomem.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-designware.c |4
1 file changed, 4 insertions(+)
diff --git a/drivers/pci/host/pcie
of_pci_get_host_bridge_resources() allocates a list of resources for host
bridge windows. If we fail after allocating that list, free it before we
return error.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-xgene.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions
of_pci_get_host_bridge_resources() allocates a list of resources for host
bridge windows. If we fail after allocating that list, free it before we
return error.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-xilinx-nwl.c | 14 ++
1 file changed, 10 insertions(+), 4
Request host bridge window resources so they appear in ioport_resource and
iomem_resource and are reflected in /proc/ioports and /proc/iomem.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-xilinx-nwl.c |4
1 file changed, 4 insertions(+)
diff --git a/drivers/pci/host/pcie
Use devm_request_pci_bus_resources() to request host bridge window
resources instead of doing it by hand in the driver.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-altera.c | 11 +--
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git
Use dev_printk() when possible to make messages more useful.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-xilinx-nwl.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pci/host/pcie-xilinx-nwl.c
b/drivers/pci/host/pcie-xilinx-nwl.c
index 3c16bbf..0b597d9
Request host bridge window resources so they appear in ioport_resource and
iomem_resource and are reflected in /proc/ioports and /proc/iomem.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-xilinx.c |5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/pci/host/pcie
Bus :01
e18000-e181ff : :01:00.0
e18200-e1820f : :01:00.0
e18210-e1821f : :01:00.0
f0-ff : /soc/pcie@1f2b
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-xgene.c |4
1 file changed, 4 insertions
Use devm_request_pci_bus_resources() to request host bridge window
resources instead of doing it by hand in the driver.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-host-common.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff
Previously we allocated the PCI resource list in
gen_pci_parse_request_of_pci_ranges(), but if we had an error, we freed it
on error in gen_pci_init().
Reorder gen_pci_init() so we can take care of error path cleanup in
gen_pci_parse_request_of_pci_ranges() instead.
Signed-off-by: Bjorn Helgaas
Request host bridge window resources so they appear in ioport_resource and
iomem_resource and are reflected in /proc/ioports and /proc/iomem.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-rcar-gen2.c |4
1 file changed, 4 insertions(+)
diff --git a/drivers/pci/host/pci-rcar
functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-altera.c | 24 ++--
1 file changed, 6 insertions(+), 18 deletions(-)
diff --git a/drivers/pci/host/pcie-altera.c b/drivers/pci/host/pcie-altera.c
index b97abbc..cf20c67 100644
--- a/drivers/pci
The switch is the only statement in the resource_list_for_each_entry()
loop, so remove unnecessary cases and "continue" statements in the switch.
Inline rcar_pcie_release_of_pci_ranges(), which is only called once.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
d
The switch is the only statement in the resource_list_for_each_entry()
loop, so remove unnecessary "continue" statements in the switch. Simplify
checking for the required non-prefetchable memory aperture.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pc
pcie-controller@3000/pci@1,0
3000-37ff : /pcie-controller@3000
3000-37ff : pads
3800-39ff : /pcie-controller@3000
3800-39ff : afi
1000-1fff : /pcie-controller@00003000
1000-1fff : cs
...
Signed-off
Use devm_request_pci_bus_resources() to request host bridge window
resources instead of doing it by hand in the driver.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-mvebu.c | 17 +++--
1 file changed, 7 insertions(+), 10 deletions(-)
diff
Use devm_request_pci_bus_resources() to request host bridge window
resources instead of doing it by hand in the driver.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-rcar.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git
The switch is the only statement in the resource_list_for_each_entry()
loop, so remove unnecessary "continue" statements in the switch. Simplify
checking for the required non-prefetchable memory aperture.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pc
Use devm_request_pci_bus_resources() to request host bridge window
resources instead of doing it by hand in the driver.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-tegra.c | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff
Use devm_request_pci_bus_resources() to request host bridge window
resources instead of doing it by hand in the driver.
No functional change intended.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pci-versatile.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff
of_pci_get_host_bridge_resources() allocates a list of resources for host
bridge windows. If we fail after allocating that list, free it before we
return error.
Signed-off-by: Bjorn Helgaas
---
drivers/pci/host/pcie-xilinx.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions
On Tue, Apr 05, 2016 at 01:10:54PM -0500, Bjorn Helgaas wrote:
> Hi Simon,
>
> On Thu, Mar 31, 2016 at 09:35:23AM +0900, Simon Horman wrote:
> > It appears that Gen2 is a misnomer for the R-Car PCIE driver
> > which also supports Gen 1 and Gen 3 SoCs. Accordingly, drop Ge
;t have a bisection problem. If you want to include this patch
and merge through another tree,
Acked-by: Bjorn Helgaas
> v2
> * As suggested by Geert Uytterhoeven:
> - Use PCIE as prefix in Kconfig symbol
> ---
> drivers/pci/host/Kconfig | 4 ++--
> drivers/pci/host/Makefil
On Fri, Mar 11, 2016 at 10:49:11AM +0900, Simon Horman wrote:
> On Tue, Mar 08, 2016 at 12:36:50PM -0600, Bjorn Helgaas wrote:
> > On Thu, Feb 25, 2016 at 09:45:56AM +0900, Simon Horman wrote:
> > > Make use of ARCH_RENESAS in place of ARCH_SHMOBILE.
> > >
> > >
On Thu, Feb 25, 2016 at 09:45:56AM +0900, Simon Horman wrote:
> Make use of ARCH_RENESAS in place of ARCH_SHMOBILE.
>
> This is part of an ongoing process to migrate from ARCH_SHMOBILE to
> ARCH_RENESAS the motivation for which being that RENESAS seems to be a more
> appropriate name than SHMOBILE
1 - 100 of 104 matches
Mail list logo