* [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips
@ 2020-08-24 19:30 Jim Quinlan
2020-08-25 17:40 ` Florian Fainelli
0 siblings, 1 reply; 9+ messages in thread
From: Jim Quinlan @ 2020-08-24 19:30 UTC (permalink / raw)
To: linux-pci, Nicolas Saenz Julienne, Christoph Hellwig,
Robin Murphy, bcm-kernel-feedback-list, james.quinlan
Cc: Alan Stern, Andy Shevchenko, Bartosz Golaszewski, Dan Williams,
open list:STAGING SUBSYSTEM,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE,
open list:DRM DRIVERS FOR ALLWINNER A10, Felipe Balbi,
Greg Kroah-Hartman, H. Peter Anvin, open list:IOMMU DRIVERS,
Joerg Roedel, Julien Grall, open list:ACPI FOR ARM64 (ACPI/arm64),
moderated list:ARM PORT, open list,
open list:ALLWINNER A10 CSI DRIVER,
open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM,
moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE,
open list:SUPERH, open list:USB SUBSYSTEM, Rafael J. Wysocki,
Rob Herring, Saravana Kannan, Stefano Stabellini
Patchset Summary:
Enhance a PCIe host controller driver. Because of its unusual design
we are foced to change dev->dma_pfn_offset into a more general role
allowing multiple offsets. See the 'v1' notes below for more info.
v11:
Commit: "device-mapping: Introduce DMA range map, supplanting ..."
-- Rebased to latest torvalds, Aug 20, 2020.
-- Minor change in of_dma_get_range() to satisfy the kernel's
robot tester.
-- Use of PFN_DOWN(), PFN_PHYS() instead of explicit shifts (Andy)
-- Eliminate extra return in dma_offset_from_xxx_addr() (Andy)
-- Change dma_set_offset_range() to correctly handle the case
of pre-existing DMA map and zero offset.
v10:
Commit: "device-mapping: Introduce DMA range map, supplanting ..."
-- change title of commit; "bus core:" => "device-mapping:"
-- instead of allocating the DMA map with devm, use kcalloc
and call kfree() during device_release(). (RobH) Also,
for three cases that want to use the same DMA map, copy
the dma_range_map using a helper function.
-- added a missing 'return = 0;' to of_dma_get_range(). (Nicolas)
-- removed dma_range_overlaps(); instead return error if there
is an existing DMA map. (Christoph).
Commit: "PCI: brcmstb: Set additional internal memory DMA ..."
-- Changed constant 1 to 1ULL. (Nicolas)
Commit: "ata: ahci_brcm: Fix use of BCM7216 reset controller"
This commit has been removed from this patchset and will be
submitted on its own.
v9:
Commit: "device core: Introduce DMA range map, supplanting ..."
-- A number of code improvements were implemented as suggested by
ChristophH. Unfortunately, some of these changes reversed the
implemented suggestions of other reviewers; for example, the new
macros PFN_DMA_ADDR(), DMA_ADDR_PFN() have been pulled.
v8:
Commit: "device core: Introduce DMA range map, supplanting ..."
-- To satisfy a specific m68 compile configuration, I moved the 'struct
bus_dma_region; definition out of #ifdef CONFIG_HAS_DMA and also defined
three inline functions for !CONFIG_HAS_DMA (kernel test robot).
-- The sunXi drivers -- suc4i_csi, sun6i_csi, cedrus_hw -- set
a pfn_offset outside of_dma_configure() but the code offers no
insight on the size of the translation window. V7 had me using
SIZE_MAX as the size. I have since contacted the sunXi maintainer and
he said that using a size of SZ_4G would cover sunXi configurations.
v7:
Commit: "device core: Introduce DMA range map, supplanting ..."
-- remove second kcalloc/copy in device.c (AndyS)
-- use PTR_ERR_OR_ZERO() and PHYS_PFN() (AndyS)
-- indentation, sizeof(struct ...) => sizeof(*r) (AndyS)
-- add pfn.h definitions: PFN_DMA_ADDR(), DMA_ADDR_PFN() (AndyS)
-- Fixed compile error in "sun6i_csi.c" (kernel test robot)
Commit "ata: ahci_brcm: Fix use of BCM7216 reset controller"
-- correct name of function in the commit msg (SergeiS)
v6:
Commit "device core: Introduce DMA range map":
-- of_dma_get_range() now takes a single argument and returns either
NULL, a valid map, or an ERR_PTR. (Robin)
-- offsets are no longer a PFN value but an actual address. (Robin)
-- the bus_dma_region struct stores the range size instead of
the cpu_end and pci_end values. (Robin)
-- devices that were setting a single offset with no boundaries
have been modified to have boundaries; in a few places
where this information was unavilable a /* FIXME: ... */
comment was added. (Robin)
-- dma_attach_offset_range() can be called when an offset
map already exists; if it's range is already present
nothing is done and success is returned. (Robin)
All commits:
-- Man name/style/corrections/etc changed (Bjorn)
-- rebase to Torvalds master
v5:
Commit "device core: Introduce multiple dma pfn offsets"
-- in of/address.c: "map_size = 0" => "*map_size = 0"
-- use kcalloc instead of kzalloc (AndyS)
-- use PHYS_ADDR_MAX instead of "~(phys_addr_t)0"
Commit "PCI: brcmstb: Set internal memory viewport sizes"
-- now gives error on missing dma-ranges property.
Commit "dt-bindings: PCI: Add bindings for more Brcmstb chips"
-- removed "Allof:" from brcm,scb-sizes definition (RobH)
All Commits:
-- indentation style, use max chars 100 (AndyS)
-- rebased to torvalds master
v4:
Commit "device core: Introduce multiple dma pfn offsets"
-- of_dma_get_range() does not take a dev param but instead
takes two "out" params: map and map_size. We do this so
that the code that parses dma-ranges is separate from
the code that modifies 'dev'. (Nicolas)
-- the separate case of having a single pfn offset has
been removed and is now processed by going through the
map array. (Nicolas)
-- move attach_uniform_dma_pfn_offset() from of/address.c to
dma/mapping.c so that it does not depend on CONFIG_OF. (Nicolas)
-- devm_kcalloc => devm_kzalloc (DanC)
-- add/fix assignment to dev->dma_pfn_offset_map for func
attach_uniform_dma_pfn_offset() (DanC, Nicolas)
-- s/struct dma_pfn_offset_region/struct bus_dma_region/ (Nicolas)
-- s/attach_uniform_dma_pfn_offset/dma_attach_uniform_pfn_offset/
-- s/attach_dma_pfn_offset_map/dma_attach_pfn_offset_map/
-- More use of PFN_{PHYS,DOWN,UP}. (AndyS)
Commit "of: Include a dev param in of_dma_get_range()"
-- this commit was sqaushed with "device core: Introduce ..."
v3:
Commit "device core: Introduce multiple dma pfn offsets"
Commit "arm: dma-mapping: Invoke dma offset func if needed"
-- The above two commits have been squashed. More importantly,
the code has been modified so that the functionality for
multiple pfn offsets subsumes the use of dev->dma_pfn_offset.
In fact, dma_pfn_offset is removed and supplanted by
dma_pfn_offset_map, which is a pointer to an array. The
more common case of a uniform offset is now handled as
a map with a single entry, while cases requiring multiple
pfn offsets use a map with multiple entries. Code paths
that used to do this:
dev->dma_pfn_offset = mydrivers_pfn_offset;
have been changed to do this:
attach_uniform_dma_pfn_offset(dev, pfn_offset);
Commit "dt-bindings: PCI: Add bindings for more Brcmstb chips"
-- Add if/then clause for required props: resets, reset-names (RobH)
-- Change compatible list from const to enum (RobH)
-- Change list of u32-tuples to u64 (RobH)
Commit "of: Include a dev param in of_dma_get_range()"
-- modify of/unittests.c to add NULL param in of_dma_get_range() call.
Commit "device core: Add ability to handle multiple dma offsets"
-- align comment in device.h (AndyS).
-- s/cpu_beg/cpu_start/ and s/dma_beg/dma_start/ in struct
dma_pfn_offset_region (AndyS).
v2:
Commit: "device core: Add ability to handle multiple dma offsets"
o Added helper func attach_dma_pfn_offset_map() in address.c (Chistoph)
o Helpers funcs added to __phys_to_dma() & __dma_to_phys() (Christoph)
o Added warning when multiple offsets are needed and !DMA_PFN_OFFSET_MAP
o dev->dma_pfn_map => dev->dma_pfn_offset_map
o s/frm/from/ for dma_pfn_offset_frm_{phys,dma}_addr() (Christoph)
o In device.h: s/const void */const struct dma_pfn_offset_region */
o removed 'unlikely' from unlikely(dev->dma_pfn_offset_map) since
guarded by CONFIG_DMA_PFN_OFFSET_MAP (Christoph)
o Since dev->dma_pfn_offset is copied in usb/core/{usb,message}.c, now
dev->dma_pfn_offset_map is copied as well.
o Merged two of the DMA commits into one (Christoph).
Commit "arm: dma-mapping: Invoke dma offset func if needed":
o Use helper functions instead of #if CONFIG_DMA_PFN_OFFSET
Other commits' changes:
o Removed need for carrying of_id var in priv (Nicolas)
o Commit message rewordings (Bjorn)
o Commit log messages filled to 75 chars (Bjorn)
o devm_reset_control_get_shared())
=> devm_reset_control_get_optional_shared (Philipp)
o Add call to reset_control_assert() in PCIe remove routines (Philipp)
v1:
This patchset expands the usefulness of the Broadcom Settop Box PCIe
controller by building upon the PCIe driver used currently by the
Raspbery Pi. Other forms of this patchset were submitted by me years
ago and not accepted; the major sticking point was the code required
for the DMA remapping needed for the PCIe driver to work [1].
There have been many changes to the DMA and OF subsystems since that
time, making a cleaner and less intrusive patchset possible. This
patchset implements a generalization of "dev->dma_pfn_offset", except
that instead of a single scalar offset it provides for multiple
offsets via a function which depends upon the "dma-ranges" property of
the PCIe host controller. This is required for proper functionality
of the BrcmSTB PCIe controller and possibly some other devices.
[1] https://lore.kernel.org/linux-arm-kernel/1516058925-46522-5-git-send-email-jim2101024@gmail.com/
Jim Quinlan (11):
PCI: brcmstb: PCIE_BRCMSTB depends on ARCH_BRCMSTB
dt-bindings: PCI: Add bindings for more Brcmstb chips
PCI: brcmstb: Add bcm7278 register info
PCI: brcmstb: Add suspend and resume pm_ops
PCI: brcmstb: Add bcm7278 PERST# support
PCI: brcmstb: Add control of rescal reset
device-mapping: Introduce DMA range map, supplanting dma_pfn_offset
PCI: brcmstb: Set additional internal memory DMA viewport sizes
PCI: brcmstb: Accommodate MSI for older chips
PCI: brcmstb: Set bus max burst size by chip type
PCI: brcmstb: Add bcm7211, bcm7216, bcm7445, bcm7278 to match list
.../bindings/pci/brcm,stb-pcie.yaml | 56 ++-
arch/arm/include/asm/dma-mapping.h | 10 +-
arch/arm/mach-keystone/keystone.c | 17 +-
arch/sh/drivers/pci/pcie-sh7786.c | 9 +-
arch/x86/pci/sta2x11-fixup.c | 7 +-
drivers/acpi/arm64/iort.c | 5 +-
drivers/base/core.c | 2 +
drivers/gpu/drm/sun4i/sun4i_backend.c | 5 +-
drivers/iommu/io-pgtable-arm.c | 2 +-
.../platform/sunxi/sun4i-csi/sun4i_csi.c | 5 +-
.../platform/sunxi/sun6i-csi/sun6i_csi.c | 4 +-
drivers/of/address.c | 72 ++-
drivers/of/device.c | 43 +-
drivers/of/of_private.h | 10 +-
drivers/of/unittest.c | 34 +-
drivers/pci/controller/Kconfig | 3 +-
drivers/pci/controller/pcie-brcmstb.c | 409 +++++++++++++++---
drivers/remoteproc/remoteproc_core.c | 8 +-
.../staging/media/sunxi/cedrus/cedrus_hw.c | 7 +-
drivers/usb/core/message.c | 9 +-
drivers/usb/core/usb.c | 7 +-
include/linux/device.h | 4 +-
include/linux/dma-direct.h | 8 +-
include/linux/dma-mapping.h | 36 ++
kernel/dma/coherent.c | 10 +-
kernel/dma/mapping.c | 66 +++
26 files changed, 668 insertions(+), 180 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips
2020-08-24 19:30 [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips Jim Quinlan
@ 2020-08-25 17:40 ` Florian Fainelli
2020-08-27 6:35 ` Christoph Hellwig
0 siblings, 1 reply; 9+ messages in thread
From: Florian Fainelli @ 2020-08-25 17:40 UTC (permalink / raw)
To: Jim Quinlan, linux-pci, Nicolas Saenz Julienne,
Christoph Hellwig, Robin Murphy, bcm-kernel-feedback-list
Cc: Alan Stern, Andy Shevchenko, Bartosz Golaszewski, Dan Williams,
open list:STAGING SUBSYSTEM,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE,
open list:DRM DRIVERS FOR ALLWINNER A10, Felipe Balbi,
Greg Kroah-Hartman, H. Peter Anvin, open list:IOMMU DRIVERS,
Joerg Roedel, Julien Grall, open list:ACPI FOR ARM64 (ACPI/arm64),
moderated list:ARM PORT, open list,
open list:ALLWINNER A10 CSI DRIVER,
open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM,
moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE,
open list:SUPERH, open list:USB SUBSYSTEM, Rafael J. Wysocki,
Rob Herring, Saravana Kannan, Stefano Stabellini
Hi,
On 8/24/2020 12:30 PM, Jim Quinlan wrote:
>
> Patchset Summary:
> Enhance a PCIe host controller driver. Because of its unusual design
> we are foced to change dev->dma_pfn_offset into a more general role
> allowing multiple offsets. See the 'v1' notes below for more info.
We are version 11 and counting, and it is not clear to me whether there
is any chance of getting these patches reviewed and hopefully merged for
the 5.10 merge window.
There are a lot of different files being touched, so what would be the
ideal way of routing those changes towards inclusion?
Thanks!
--
Florian
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips
2020-08-25 17:40 ` Florian Fainelli
@ 2020-08-27 6:35 ` Christoph Hellwig
2020-08-27 13:29 ` Jim Quinlan
0 siblings, 1 reply; 9+ messages in thread
From: Christoph Hellwig @ 2020-08-27 6:35 UTC (permalink / raw)
To: Florian Fainelli
Cc: Jim Quinlan, linux-pci, Nicolas Saenz Julienne,
Christoph Hellwig, Robin Murphy, bcm-kernel-feedback-list,
Alan Stern, Andy Shevchenko, Bartosz Golaszewski, Dan Williams,
open list:STAGING SUBSYSTEM,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE,
open list:DRM DRIVERS FOR ALLWINNER A10, Felipe Balbi,
Greg Kroah-Hartman, H. Peter Anvin, open list:IOMMU DRIVERS,
Joerg Roedel, Julien Grall, open list:ACPI FOR ARM64 (ACPI/arm64),
moderated list:ARM PORT, open list,
open list:ALLWINNER A10 CSI DRIVER,
open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM,
moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE,
open list:SUPERH, open list:USB SUBSYSTEM, Rafael J. Wysocki,
Rob Herring, Saravana Kannan, Stefano Stabellini
On Tue, Aug 25, 2020 at 10:40:27AM -0700, Florian Fainelli wrote:
> Hi,
>
> On 8/24/2020 12:30 PM, Jim Quinlan wrote:
>>
>> Patchset Summary:
>> Enhance a PCIe host controller driver. Because of its unusual design
>> we are foced to change dev->dma_pfn_offset into a more general role
>> allowing multiple offsets. See the 'v1' notes below for more info.
>
> We are version 11 and counting, and it is not clear to me whether there is
> any chance of getting these patches reviewed and hopefully merged for the
> 5.10 merge window.
>
> There are a lot of different files being touched, so what would be the
> ideal way of routing those changes towards inclusion?
FYI, I offered to take the dma-mapping bits through the dma-mapping tree.
I have a bit of a backlog, but plan to review and if Jim is ok with that
apply the current version.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips
2020-08-27 6:35 ` Christoph Hellwig
@ 2020-08-27 13:29 ` Jim Quinlan
2020-09-07 9:16 ` Lorenzo Pieralisi
0 siblings, 1 reply; 9+ messages in thread
From: Jim Quinlan @ 2020-08-27 13:29 UTC (permalink / raw)
To: Christoph Hellwig
Cc: Florian Fainelli,
open list:PCI NATIVE HOST BRIDGE AND ENDPOINT DRIVERS,
Nicolas Saenz Julienne, Robin Murphy,
maintainer:BROADCOM BCM7XXX ARM ARCHITECTURE, Alan Stern,
Andy Shevchenko, Bartosz Golaszewski, Dan Williams,
open list:STAGING SUBSYSTEM,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE,
open list:DRM DRIVERS FOR ALLWINNER A10, Felipe Balbi,
Greg Kroah-Hartman, H. Peter Anvin, open list:IOMMU DRIVERS,
Joerg Roedel, Julien Grall, open list:ACPI FOR ARM64 (ACPI/arm64),
moderated list:ARM PORT, open list,
open list:ALLWINNER A10 CSI DRIVER,
open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM,
moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE,
open list:SUPERH, open list:USB SUBSYSTEM, Rafael J. Wysocki,
Rob Herring, Saravana Kannan, Stefano Stabellini
On Thu, Aug 27, 2020 at 2:35 AM Christoph Hellwig <hch@lst.de> wrote:
>
> On Tue, Aug 25, 2020 at 10:40:27AM -0700, Florian Fainelli wrote:
> > Hi,
> >
> > On 8/24/2020 12:30 PM, Jim Quinlan wrote:
> >>
> >> Patchset Summary:
> >> Enhance a PCIe host controller driver. Because of its unusual design
> >> we are foced to change dev->dma_pfn_offset into a more general role
> >> allowing multiple offsets. See the 'v1' notes below for more info.
> >
> > We are version 11 and counting, and it is not clear to me whether there is
> > any chance of getting these patches reviewed and hopefully merged for the
> > 5.10 merge window.
> >
> > There are a lot of different files being touched, so what would be the
> > ideal way of routing those changes towards inclusion?
>
> FYI, I offered to take the dma-mapping bits through the dma-mapping tree.
> I have a bit of a backlog, but plan to review and if Jim is ok with that
> apply the current version.
Sounds good to me.
Thanks, Jim
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips
2020-08-27 13:29 ` Jim Quinlan
@ 2020-09-07 9:16 ` Lorenzo Pieralisi
2020-09-07 17:43 ` Jim Quinlan
0 siblings, 1 reply; 9+ messages in thread
From: Lorenzo Pieralisi @ 2020-09-07 9:16 UTC (permalink / raw)
To: Jim Quinlan
Cc: Christoph Hellwig, Florian Fainelli,
open list:PCI NATIVE HOST BRIDGE AND ENDPOINT DRIVERS,
Nicolas Saenz Julienne, Robin Murphy,
maintainer:BROADCOM BCM7XXX ARM ARCHITECTURE, Alan Stern,
Andy Shevchenko, Bartosz Golaszewski, Dan Williams,
open list:STAGING SUBSYSTEM,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE,
open list:DRM DRIVERS FOR ALLWINNER A10, Felipe Balbi,
Greg Kroah-Hartman, H. Peter Anvin, open list:IOMMU DRIVERS,
Joerg Roedel, Julien Grall, open list:ACPI FOR ARM64 (ACPI/arm64),
moderated list:ARM PORT, open list,
open list:ALLWINNER A10 CSI DRIVER,
open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM,
moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE,
open list:SUPERH, open list:USB SUBSYSTEM, Rafael J. Wysocki,
Rob Herring, Saravana Kannan, Stefano Stabellini
On Thu, Aug 27, 2020 at 09:29:59AM -0400, Jim Quinlan wrote:
> On Thu, Aug 27, 2020 at 2:35 AM Christoph Hellwig <hch@lst.de> wrote:
> >
> > On Tue, Aug 25, 2020 at 10:40:27AM -0700, Florian Fainelli wrote:
> > > Hi,
> > >
> > > On 8/24/2020 12:30 PM, Jim Quinlan wrote:
> > >>
> > >> Patchset Summary:
> > >> Enhance a PCIe host controller driver. Because of its unusual design
> > >> we are foced to change dev->dma_pfn_offset into a more general role
> > >> allowing multiple offsets. See the 'v1' notes below for more info.
> > >
> > > We are version 11 and counting, and it is not clear to me whether there is
> > > any chance of getting these patches reviewed and hopefully merged for the
> > > 5.10 merge window.
> > >
> > > There are a lot of different files being touched, so what would be the
> > > ideal way of routing those changes towards inclusion?
> >
> > FYI, I offered to take the dma-mapping bits through the dma-mapping tree.
> > I have a bit of a backlog, but plan to review and if Jim is ok with that
> > apply the current version.
> Sounds good to me.
Hi Jim,
is the dependency now solved ? Should we review/take this series as
is for v5.10 through the PCI tree ?
Thanks,
Lorenzo
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips
2020-09-07 9:16 ` Lorenzo Pieralisi
@ 2020-09-07 17:43 ` Jim Quinlan
2020-09-07 18:29 ` Florian Fainelli
0 siblings, 1 reply; 9+ messages in thread
From: Jim Quinlan @ 2020-09-07 17:43 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Christoph Hellwig, Florian Fainelli,
open list:PCI NATIVE HOST BRIDGE AND ENDPOINT DRIVERS,
Nicolas Saenz Julienne, Robin Murphy,
maintainer:BROADCOM BCM7XXX ARM ARCHITECTURE, Alan Stern,
Andy Shevchenko, Bartosz Golaszewski, Dan Williams,
open list:STAGING SUBSYSTEM,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE,
open list:DRM DRIVERS FOR ALLWINNER A10, Felipe Balbi,
Greg Kroah-Hartman, H. Peter Anvin, open list:IOMMU DRIVERS,
Joerg Roedel, Julien Grall, open list:ACPI FOR ARM64 (ACPI/arm64),
moderated list:ARM PORT, open list,
open list:ALLWINNER A10 CSI DRIVER,
open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM,
moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE,
open list:SUPERH, open list:USB SUBSYSTEM, Rafael J. Wysocki,
Rob Herring, Saravana Kannan, Stefano Stabellini
On Mon, Sep 7, 2020 at 5:16 AM Lorenzo Pieralisi
<lorenzo.pieralisi@arm.com> wrote:
>
> On Thu, Aug 27, 2020 at 09:29:59AM -0400, Jim Quinlan wrote:
> > On Thu, Aug 27, 2020 at 2:35 AM Christoph Hellwig <hch@lst.de> wrote:
> > >
> > > On Tue, Aug 25, 2020 at 10:40:27AM -0700, Florian Fainelli wrote:
> > > > Hi,
> > > >
> > > > On 8/24/2020 12:30 PM, Jim Quinlan wrote:
> > > >>
> > > >> Patchset Summary:
> > > >> Enhance a PCIe host controller driver. Because of its unusual design
> > > >> we are foced to change dev->dma_pfn_offset into a more general role
> > > >> allowing multiple offsets. See the 'v1' notes below for more info.
> > > >
> > > > We are version 11 and counting, and it is not clear to me whether there is
> > > > any chance of getting these patches reviewed and hopefully merged for the
> > > > 5.10 merge window.
> > > >
> > > > There are a lot of different files being touched, so what would be the
> > > > ideal way of routing those changes towards inclusion?
> > >
> > > FYI, I offered to take the dma-mapping bits through the dma-mapping tree.
> > > I have a bit of a backlog, but plan to review and if Jim is ok with that
> > > apply the current version.
> > Sounds good to me.
>
> Hi Jim,
>
> is the dependency now solved ? Should we review/take this series as
> is for v5.10 through the PCI tree ?
Hello Lorenzo,
We are still working out a regression with the DMA offset commit on
the RaspberryPi. Nicolas has found the root cause and we are now
devising a solution.
Thanks,
Jim Quinlan
Broadcom STB
>
> Thanks,
> Lorenzo
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips
2020-09-07 17:43 ` Jim Quinlan
@ 2020-09-07 18:29 ` Florian Fainelli
2020-09-08 10:42 ` Lorenzo Pieralisi
0 siblings, 1 reply; 9+ messages in thread
From: Florian Fainelli @ 2020-09-07 18:29 UTC (permalink / raw)
To: Jim Quinlan, Lorenzo Pieralisi
Cc: Christoph Hellwig, Florian Fainelli,
open list:PCI NATIVE HOST BRIDGE AND ENDPOINT DRIVERS,
Nicolas Saenz Julienne, Robin Murphy,
maintainer:BROADCOM BCM7XXX ARM ARCHITECTURE, Alan Stern,
Andy Shevchenko, Bartosz Golaszewski, Dan Williams,
open list:STAGING SUBSYSTEM,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE,
open list:DRM DRIVERS FOR ALLWINNER A10, Felipe Balbi,
Greg Kroah-Hartman, H. Peter Anvin, open list:IOMMU DRIVERS,
Joerg Roedel, Julien Grall, open list:ACPI FOR ARM64 (ACPI/arm64),
moderated list:ARM PORT, open list,
open list:ALLWINNER A10 CSI DRIVER,
open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM,
moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE,
open list:SUPERH, open list:USB SUBSYSTEM, Rafael J. Wysocki,
Rob Herring, Saravana Kannan, Stefano Stabellini
On 9/7/2020 10:43 AM, Jim Quinlan wrote:
> On Mon, Sep 7, 2020 at 5:16 AM Lorenzo Pieralisi
> <lorenzo.pieralisi@arm.com> wrote:
>>
>> On Thu, Aug 27, 2020 at 09:29:59AM -0400, Jim Quinlan wrote:
>>> On Thu, Aug 27, 2020 at 2:35 AM Christoph Hellwig <hch@lst.de> wrote:
>>>>
>>>> On Tue, Aug 25, 2020 at 10:40:27AM -0700, Florian Fainelli wrote:
>>>>> Hi,
>>>>>
>>>>> On 8/24/2020 12:30 PM, Jim Quinlan wrote:
>>>>>>
>>>>>> Patchset Summary:
>>>>>> Enhance a PCIe host controller driver. Because of its unusual design
>>>>>> we are foced to change dev->dma_pfn_offset into a more general role
>>>>>> allowing multiple offsets. See the 'v1' notes below for more info.
>>>>>
>>>>> We are version 11 and counting, and it is not clear to me whether there is
>>>>> any chance of getting these patches reviewed and hopefully merged for the
>>>>> 5.10 merge window.
>>>>>
>>>>> There are a lot of different files being touched, so what would be the
>>>>> ideal way of routing those changes towards inclusion?
>>>>
>>>> FYI, I offered to take the dma-mapping bits through the dma-mapping tree.
>>>> I have a bit of a backlog, but plan to review and if Jim is ok with that
>>>> apply the current version.
>>> Sounds good to me.
>>
>> Hi Jim,
>>
>> is the dependency now solved ? Should we review/take this series as
>> is for v5.10 through the PCI tree ?
> Hello Lorenzo,
>
> We are still working out a regression with the DMA offset commit on
> the RaspberryPi. Nicolas has found the root cause and we are now
> devising a solution.
Maybe we can parallelize the PCIe driver review while the DMA changes
are being worked on in Christoph's branch. Lorenzo, are you fine with
the PCIe changes proper?
--
Florian
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips
2020-09-07 18:29 ` Florian Fainelli
@ 2020-09-08 10:42 ` Lorenzo Pieralisi
2020-09-08 12:20 ` Christoph Hellwig
0 siblings, 1 reply; 9+ messages in thread
From: Lorenzo Pieralisi @ 2020-09-08 10:42 UTC (permalink / raw)
To: Florian Fainelli
Cc: Jim Quinlan, Christoph Hellwig,
open list:PCI NATIVE HOST BRIDGE AND ENDPOINT DRIVERS,
Nicolas Saenz Julienne, Robin Murphy,
maintainer:BROADCOM BCM7XXX ARM ARCHITECTURE, Alan Stern,
Andy Shevchenko, Bartosz Golaszewski, Dan Williams,
open list:STAGING SUBSYSTEM,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE,
open list:DRM DRIVERS FOR ALLWINNER A10, Felipe Balbi,
Greg Kroah-Hartman, H. Peter Anvin, open list:IOMMU DRIVERS,
Joerg Roedel, Julien Grall, open list:ACPI FOR ARM64 (ACPI/arm64),
moderated list:ARM PORT, open list,
open list:ALLWINNER A10 CSI DRIVER,
open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM,
moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE,
open list:SUPERH, open list:USB SUBSYSTEM, Rafael J. Wysocki,
Rob Herring, Saravana Kannan, Stefano Stabellini
On Mon, Sep 07, 2020 at 11:29:06AM -0700, Florian Fainelli wrote:
>
>
> On 9/7/2020 10:43 AM, Jim Quinlan wrote:
> > On Mon, Sep 7, 2020 at 5:16 AM Lorenzo Pieralisi
> > <lorenzo.pieralisi@arm.com> wrote:
> > >
> > > On Thu, Aug 27, 2020 at 09:29:59AM -0400, Jim Quinlan wrote:
> > > > On Thu, Aug 27, 2020 at 2:35 AM Christoph Hellwig <hch@lst.de> wrote:
> > > > >
> > > > > On Tue, Aug 25, 2020 at 10:40:27AM -0700, Florian Fainelli wrote:
> > > > > > Hi,
> > > > > >
> > > > > > On 8/24/2020 12:30 PM, Jim Quinlan wrote:
> > > > > > >
> > > > > > > Patchset Summary:
> > > > > > > Enhance a PCIe host controller driver. Because of its unusual design
> > > > > > > we are foced to change dev->dma_pfn_offset into a more general role
> > > > > > > allowing multiple offsets. See the 'v1' notes below for more info.
> > > > > >
> > > > > > We are version 11 and counting, and it is not clear to me whether there is
> > > > > > any chance of getting these patches reviewed and hopefully merged for the
> > > > > > 5.10 merge window.
> > > > > >
> > > > > > There are a lot of different files being touched, so what would be the
> > > > > > ideal way of routing those changes towards inclusion?
> > > > >
> > > > > FYI, I offered to take the dma-mapping bits through the dma-mapping tree.
> > > > > I have a bit of a backlog, but plan to review and if Jim is ok with that
> > > > > apply the current version.
> > > > Sounds good to me.
> > >
> > > Hi Jim,
> > >
> > > is the dependency now solved ? Should we review/take this series as
> > > is for v5.10 through the PCI tree ?
> > Hello Lorenzo,
> >
> > We are still working out a regression with the DMA offset commit on
> > the RaspberryPi. Nicolas has found the root cause and we are now
> > devising a solution.
>
> Maybe we can parallelize the PCIe driver review while the DMA changes
> are being worked on in Christoph's branch. Lorenzo, are you fine with
> the PCIe changes proper?
I will have a look - the main contentious point was about the DMA
changes - if Christoph is happy with them I am OK with them
too - I hope there is not anything controversial in the host
bridge driver itself but I will look into it.
Lorenzo
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips
2020-09-08 10:42 ` Lorenzo Pieralisi
@ 2020-09-08 12:20 ` Christoph Hellwig
0 siblings, 0 replies; 9+ messages in thread
From: Christoph Hellwig @ 2020-09-08 12:20 UTC (permalink / raw)
To: Lorenzo Pieralisi
Cc: Florian Fainelli, Jim Quinlan, Christoph Hellwig,
open list:PCI NATIVE HOST BRIDGE AND ENDPOINT DRIVERS,
Nicolas Saenz Julienne, Robin Murphy,
maintainer:BROADCOM BCM7XXX ARM ARCHITECTURE, Alan Stern,
Andy Shevchenko, Bartosz Golaszewski, Dan Williams,
open list:STAGING SUBSYSTEM,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE,
open list:DRM DRIVERS FOR ALLWINNER A10, Felipe Balbi,
Greg Kroah-Hartman, H. Peter Anvin, open list:IOMMU DRIVERS,
Joerg Roedel, Julien Grall, open list:ACPI FOR ARM64 (ACPI/arm64),
moderated list:ARM PORT, open list,
open list:ALLWINNER A10 CSI DRIVER,
open list:REMOTE PROCESSOR (REMOTEPROC) SUBSYSTEM,
moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE,
open list:SUPERH, open list:USB SUBSYSTEM, Rafael J. Wysocki,
Rob Herring, Saravana Kannan, Stefano Stabellini
On Tue, Sep 08, 2020 at 11:42:26AM +0100, Lorenzo Pieralisi wrote:
> > Maybe we can parallelize the PCIe driver review while the DMA changes
> > are being worked on in Christoph's branch. Lorenzo, are you fine with
> > the PCIe changes proper?
>
> I will have a look - the main contentious point was about the DMA
> changes - if Christoph is happy with them I am OK with them
> too - I hope there is not anything controversial in the host
> bridge driver itself but I will look into it.
I'm pretty happy with the overall shape. Now we just need to squeeze
out the regressions..
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2020-09-08 16:45 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-24 19:30 [PATCH v11 00/11] PCI: brcmstb: enable PCIe for STB chips Jim Quinlan
2020-08-25 17:40 ` Florian Fainelli
2020-08-27 6:35 ` Christoph Hellwig
2020-08-27 13:29 ` Jim Quinlan
2020-09-07 9:16 ` Lorenzo Pieralisi
2020-09-07 17:43 ` Jim Quinlan
2020-09-07 18:29 ` Florian Fainelli
2020-09-08 10:42 ` Lorenzo Pieralisi
2020-09-08 12:20 ` Christoph Hellwig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).