diff for duplicates of <1452783597.28109.12.camel@redhat.com> diff --git a/a/1.txt b/N1/1.txt index e85835c..ce90372 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -6,13 +6,13 @@ On Thu, 2016-01-14 at 14:50 +0000, Catalin Marinas wrote: > > > > > From the functionality point of view this series might be split into the > > > > > following logic parts: > > > > > 1. Make MMCONFIG code arch-agnostic which allows all architectures to collect -> > > > > PCI config regions and used when necessary. +> > > > > ???PCI config regions and used when necessary. > > > > > 2. Move non-arch specific bits to the core code. > > > > > 3. Use MMCONFIG code and implement generic ACPI based PCI host controller driver. > > > > > 4. Enable above driver on ARM64 > > > > > > > > > > Patches has been built on top of 4.4 and can be found here: -> > > > > git@github.com:semihalf-nowicki-tomasz/linux.git (pci-acpi-v3) +> > > > > git at github.com:semihalf-nowicki-tomasz/linux.git (pci-acpi-v3) > > > > > > > > > > NOTE, this patch set depends on Matthew's patches: > > > > > http://www.spinics.net/lists/linux-pci/msg45950.html @@ -33,14 +33,14 @@ On Thu, 2016-01-14 at 14:50 +0000, Catalin Marinas wrote: > > > > --- a/drivers/pci/probe.c > > > > +++ b/drivers/pci/probe.c > > > > @@ -1717,7 +1717,7 @@ void pci_device_add(struct pci_dev *dev, struct pci_bus *bus) -> > > > set_dev_node(&dev->dev, pcibus_to_node(bus)); -> > > > dev->dev.dma_mask = &dev->dma_mask; -> > > > dev->dev.dma_parms = &dev->dma_parms; -> > > > - dev->dev.coherent_dma_mask = 0xffffffffull; -> > > > + dev->dev.coherent_dma_mask = DMA_BIT_MASK(64); -> > > > pci_dma_configure(dev); +> > > > ????????set_dev_node(&dev->dev, pcibus_to_node(bus)); +> > > > ????????dev->dev.dma_mask = &dev->dma_mask; +> > > > ????????dev->dev.dma_parms = &dev->dma_parms; +> > > > -???????dev->dev.coherent_dma_mask = 0xffffffffull; +> > > > +???????dev->dev.coherent_dma_mask = DMA_BIT_MASK(64); +> > > > ????????pci_dma_configure(dev); > > > > -> > > > pci_set_dma_max_seg_size(dev, 65536); +> > > > ????????pci_set_dma_max_seg_size(dev, 65536); > > > > > > With OF, we get the coherent_dma_mask set by of_dma_configure(). But I > > > have no idea how you do this with ACPI. @@ -48,8 +48,8 @@ On Thu, 2016-01-14 at 14:50 +0000, Catalin Marinas wrote: > > That doesn't get called for a PCI device (not listed in DT) does it? > > pci_device_add -> pci_dma_configure -> of_dma_configure(&dev->dev, bridge->parent->of_node); +> ? pci_dma_configure +> ????of_dma_configure(&dev->dev, bridge->parent->of_node); > > So it gets configured based on the bridge information the device is > attached to. diff --git a/a/content_digest b/N1/content_digest index dfeef48..e38a8cd 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -3,35 +3,10 @@ "ref\020160114140038.GE2071@e104818-lin.cambridge.arm.com\0" "ref\01452780599.28109.5.camel@redhat.com\0" "ref\020160114145010.GF2071@e104818-lin.cambridge.arm.com\0" - "From\0Mark Salter <msalter@redhat.com>\0" - "Subject\0Re: [PATCH V3 00/21] MMCONFIG refactoring and support for ARM64 PCI hostbridge init based on ACPI\0" + "From\0msalter@redhat.com (Mark Salter)\0" + "Subject\0[PATCH V3 00/21] MMCONFIG refactoring and support for ARM64 PCI hostbridge init based on ACPI\0" "Date\0Thu, 14 Jan 2016 09:59:57 -0500\0" - "To\0Catalin Marinas <catalin.marinas@arm.com>\0" - "Cc\0Graeme Gregory <gg@slimlogic.co.uk>" - linaro-acpi@lists.linaro.org - linux-pci@vger.kernel.org - will.deacon@arm.com - okaya@codeaurora.org - wangyijing@huawei.com - Lorenzo.Pieralisi@arm.com - Tomasz Nowicki <tn@semihalf.com> - ddaney@caviumnetworks.com - linux-acpi@vger.kernel.org - robert.richter@caviumnetworks.com - jcm@redhat.com - arnd@arndb.de - Stefano.Stabellini@eu.citrix.com - Liviu.Dudau@arm.com - bhelgaas@google.com - tglx@linutronix.de - mw@semihalf.com - linux-arm-kernel@lists.infradead.org - jchandra@broadcom.com - rjw@rjwysocki.net - linux-kernel@vger.kernel.org - hanjun.guo@linaro.org - Suravee.Suthikulpanit@amd.com - " jiang.liu@linux.intel.com\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Thu, 2016-01-14 at 14:50 +0000, Catalin Marinas wrote:\n" @@ -42,13 +17,13 @@ "> > > > > From the functionality point of view this series might be split into the\n" "> > > > > following logic parts:\n" "> > > > > 1. Make MMCONFIG code arch-agnostic which allows all architectures to collect\n" - "> > > > > \302\240\302\240\302\240PCI config regions and used when necessary.\n" + "> > > > > ???PCI config regions and used when necessary.\n" "> > > > > 2. Move non-arch specific bits to the core code.\n" "> > > > > 3. Use MMCONFIG code and implement generic ACPI based PCI host controller driver.\n" "> > > > > 4. Enable above driver on ARM64\n" "> > > > > \n" "> > > > > Patches has been built on top of 4.4 and can be found here:\n" - "> > > > > git@github.com:semihalf-nowicki-tomasz/linux.git (pci-acpi-v3)\n" + "> > > > > git at github.com:semihalf-nowicki-tomasz/linux.git (pci-acpi-v3)\n" "> > > > > \n" "> > > > > NOTE, this patch set depends on Matthew's patches:\n" "> > > > > http://www.spinics.net/lists/linux-pci/msg45950.html\n" @@ -69,14 +44,14 @@ "> > > > --- a/drivers/pci/probe.c\n" "> > > > +++ b/drivers/pci/probe.c\n" "> > > > @@ -1717,7 +1717,7 @@ void pci_device_add(struct pci_dev *dev, struct pci_bus *bus)\n" - "> > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240set_dev_node(&dev->dev, pcibus_to_node(bus));\n" - "> > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240dev->dev.dma_mask = &dev->dma_mask;\n" - "> > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240dev->dev.dma_parms = &dev->dma_parms;\n" - "> > > > -\302\240\302\240\302\240\302\240\302\240\302\240\302\240dev->dev.coherent_dma_mask = 0xffffffffull;\n" - "> > > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240dev->dev.coherent_dma_mask = DMA_BIT_MASK(64);\n" - "> > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240pci_dma_configure(dev);\n" + "> > > > ????????set_dev_node(&dev->dev, pcibus_to_node(bus));\n" + "> > > > ????????dev->dev.dma_mask = &dev->dma_mask;\n" + "> > > > ????????dev->dev.dma_parms = &dev->dma_parms;\n" + "> > > > -???????dev->dev.coherent_dma_mask = 0xffffffffull;\n" + "> > > > +???????dev->dev.coherent_dma_mask = DMA_BIT_MASK(64);\n" + "> > > > ????????pci_dma_configure(dev);\n" "> > > > \n" - "> > > > \302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240pci_set_dma_max_seg_size(dev, 65536);\n" + "> > > > ????????pci_set_dma_max_seg_size(dev, 65536);\n" "> > > \n" "> > > With OF, we get the coherent_dma_mask set by of_dma_configure(). But I\n" "> > > have no idea how you do this with ACPI.\n" @@ -84,8 +59,8 @@ "> > That doesn't get called for a PCI device (not listed in DT) does it?\n" "> \n" "> pci_device_add\n" - "> \302\240 pci_dma_configure\n" - "> \302\240\302\240\302\240\302\240of_dma_configure(&dev->dev, bridge->parent->of_node);\n" + "> ? pci_dma_configure\n" + "> ????of_dma_configure(&dev->dev, bridge->parent->of_node);\n" "> \n" "> So it gets configured based on the bridge information the device is\n" "> attached to.\n" @@ -96,4 +71,4 @@ "an r8169 card on Mustang and Seattle with this patch series in a 4.4 kernel\n" and it worked fine with r8169.use_dac=1. -f3e3cf276ef512ceb4d9eec2d25ea793d392c84bed9fe5881c1857c60d2b1ac1 +1a9298e8e10d62224591b9abe538207b94f4538aed7b7e0e8a9d43d12f84b6d9
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.