On Sun, May 05, 2013 at 03:41:49AM +0100, Benjamin Herrenschmidt wrote:
On Mon, 2013-04-22 at 11:41 +0100, Andrew Murray wrote:
The pci_process_bridge_OF_ranges function, used to parse the ranges
property of a PCI host device, is found in both Microblaze and PowerPC
architectures
This patch converts the pci_load_of_ranges function to use the new common
of_pci_range_parser.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Signed-off-by: Gabor Juhos juh...@openwrt.org
Reviewed-by: Rob Herring rob.herr...@calxeda.com
Reviewed
on
PowerPC, x86_64, MIPS and Microblaze.
Compared to the v8 sent by Andrew Murray, the following changes have been made
(please note that the MIPS patch is unchanged from v8):
* Remove the unification of pci_process_bridge_OF_ranges between PowerPC and
Microblaze. Feedback from Bjorn and Benjamin
with powerpc and microblaze).
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com
Reviewed-by: Rob Herring rob.herr...@calxeda.com
Tested-by: Thomas Petazzoni thomas.petazz...@free
This patch converts the pci_load_of_ranges function to use the new common
of_pci_range_parser.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
---
arch/microblaze/pci/pci-common.c | 106 ++
1 files changed, 38
This patch converts the pci_load_of_ranges function to use the new common
of_pci_range_parser.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Signed-off-by: Gabor Juhos juh...@openwrt.org
Reviewed-by: Rob Herring rob.herr...@calxeda.com
Reviewed
on
PowerPC, x86_64 and MIPS.
Compared to the v7 sent by Andrew Murray, the following changes have been made
(please note that the first patch is unchanged from v7):
* Rename of_pci_range_parser to of_pci_range_parser_init and
of_pci_process_ranges to of_pci_range_parser_one as suggested by Grant
with powerpc and microblaze).
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com
Reviewed-by: Rob Herring rob.herr...@calxeda.com
Tested-by: Thomas Petazzoni thomas.petazz...@free
The pci_process_bridge_OF_ranges function, used to parse the ranges
property of a PCI host device, is found in both Microblaze and PowerPC
architectures. These implementations are nearly identical. This patch
moves this common code to a common place.
Signed-off-by: Andrew Murray andrew.mur
branch the drivers that depend on it
will need to be updated to reflect the new naming of functions as suggested by
Grant.
Thanks,
Andrew Murray
___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo
On Thu, Apr 18, 2013 at 02:44:01PM +0100, Grant Likely wrote:
On Tue, 16 Apr 2013 11:18:27 +0100, Andrew Murray andrew.mur...@arm.com
wrote:
Acked-by: Grant Likely grant.lik...@secretlab.ca
But comments below...
I've updated the patchset (now v8) to reflect your feedback, after
On Wed, Apr 17, 2013 at 04:42:48PM +0100, Linus Walleij wrote:
On Tue, Apr 16, 2013 at 12:18 PM, Andrew Murray andrew.mur...@arm.com wrote:
This patch converts the pci_load_of_ranges function to use the new common
of_pci_range_parser.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
On Thu, Apr 18, 2013 at 02:45:35PM +0100, Grant Likely wrote:
On Tue, 16 Apr 2013 11:18:28 +0100, Andrew Murray andrew.mur...@arm.com
wrote:
This patch converts the pci_load_of_ranges function to use the new common
of_pci_range_parser.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
On Thu, Apr 18, 2013 at 04:29:54PM +0100, Grant Likely wrote:
On Thu, Apr 18, 2013 at 3:24 PM, Andrew Murray andrew.mur...@arm.com wrote:
On Thu, Apr 18, 2013 at 02:44:01PM +0100, Grant Likely wrote:
On Tue, 16 Apr 2013 11:18:27 +0100, Andrew Murray andrew.mur...@arm.com
wrote
The pci_process_bridge_OF_ranges function, used to parse the ranges
property of a PCI host device, is found in both Microblaze and PowerPC
architectures. These implementations are nearly identical. This patch
moves this common code to a common place.
Signed-off-by: Andrew Murray andrew.mur
with powerpc and microblaze).
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com
Reviewed-by: Rob Herring rob.herr...@calxeda.com
Tested-by: Thomas Petazzoni thomas.petazz...@free
This patch converts the pci_load_of_ranges function to use the new common
of_pci_range_parser.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Reviewed-by: Rob Herring rob.herr...@calxeda.com
---
arch/mips/pci/pci.c | 50
On Tue, Apr 16, 2013 at 11:18:26AM +0100, Andrew Murray wrote:
The pci_process_bridge_OF_ranges function, used to parse the ranges
property of a PCI host device, is found in both Microblaze and PowerPC
architectures. These implementations are nearly identical. This patch
moves this common code
...).
Andrew Murray
___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss
for the branch at
mvebu/drivers.
Some other build issues were reported on x86 during the night (thanks
to automated build tests), and Andrew Murray has just sent a new
version v7 of the patch set addressing those issues, as well as the
missing include.
Also, please note that Andrew's
On Wed, Apr 10, 2013 at 02:13:54PM +0100, Rob Herring wrote:
Adding Ben H and Michal...
On 04/10/2013 02:29 AM, Andrew Murray wrote:
The pci_process_bridge_OF_ranges function, used to parse the ranges
property of a PCI host device, is found in both Microblaze and PowerPC
architectures
On Wed, Apr 10, 2013 at 10:33:30PM +0100, Thomas Petazzoni wrote:
Dear Andrew Murray,
On Wed, 10 Apr 2013 08:29:26 +0100, Andrew Murray wrote:
diff --git a/include/linux/of_pci.h b/include/linux/of_pci.h
index bb115de..6852481 100644
--- a/include/linux/of_pci.h
+++ b/include/linux
On Wed, Apr 10, 2013 at 07:26:02PM +0100, Rob Herring wrote:
On 04/10/2013 02:29 AM, Andrew Murray wrote:
This patch factors out common implementation patterns to reduce overall
kernel
code and provide a means for host bridge drivers to directly obtain struct
resources from the DT's
This patch converts the pci_load_of_ranges function to use the new common
of_pci_range_parser.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Reviewed-by: Rob Herring rob.herr...@calxeda.com
---
arch/mips/pci/pci.c | 50
with powerpc and microblaze).
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com
Reviewed-by: Rob Herring rob.herr...@calxeda.com
Tested-by: Thomas Petazzoni thomas.petazz...@free
Petazzoni's Armada 370 and Armada XP SoCs PCIe support by:
Linus Walleij linus.wall...@linaro.org
I've tested that this patchset builds and runs on ARM and that it builds on
PowerPC.
Compared to the v5 sent by Andrew Murray, the following changes have
been made:
* Use of CONFIG_64BIT instead
The pci_process_bridge_OF_ranges function, used to parse the ranges
property of a PCI host device, is found in both Microblaze and PowerPC
architectures. These implementations are nearly identical. This patch
moves this common code to a common place.
Signed-off-by: Andrew Murray andrew.mur
(incorrectly labelled v3) sent by Andrew Murray,
the following changes have been made:
* Split the patch as suggested by Rob Herring
Compared to the v3 sent by Andrew Murray, the following changes have
been made:
* Unify and move duplicate pci_process_bridge_OF_ranges functions to
drivers
with powerpc and microblaze).
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com
---
drivers/of/address.c | 63 +
drivers/of/of_pci.c| 112
This patch converts the pci_load_of_ranges function to use the new common
of_pci_range_parser.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
---
arch/mips/pci/pci.c | 50 --
1 files changed, 16
The pci_process_bridge_OF_ranges function, used to parse the ranges
property of a PCI host device, is found in both Microblaze and PowerPC
architectures. These implementations are nearly identical. This patch
moves this common code to a common place.
Signed-off-by: Andrew Murray andrew.mur
believe pci_alloc_child_bus function would need to be changed to add
b-msi = msi to inherit msi_chip for child buses in the above patch?
Andrew Murray
[1] http://lkml.org/lkml/2013/3/25/67
___
devicetree-discuss mailing list
devicetree-discuss
On Tue, Apr 09, 2013 at 09:22:33AM +0100, Thierry Reding wrote:
On Tue, Apr 09, 2013 at 09:11:19AM +0100, Andrew Murray wrote:
[...]
Also I believe pci_alloc_child_bus function would need to be changed to add
b-msi = msi to inherit msi_chip for child buses in the above patch?
The patch
with powerpc and microblaze).
The modifications to microblaze, mips and powerpc have not been tested.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com
---
Compared to the v3 sent
have (i.e. offset from base
address already provided in DT).
Andrew Murray
+#endif /* CONFIG_PCI_MSI */
+
static int __init mvebu_pcie_probe(struct platform_device *pdev)
{
struct mvebu_pcie *pcie;
@@ -903,6 +1024,13 @@ static int __init mvebu_pcie_probe(struct
platform_device
with powerpc and microblaze).
The modifications to microblaze, mips and powerpc have not been tested.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
Signed-off-by: Thomas Petazzoni thomas.petazz...@free-electrons.com
---
Compared to [PATCH v5 01/17
On Thu, Mar 21, 2013 at 04:06:25PM +, Thomas Petazzoni wrote:
Dear Andrew Murray,
On Fri, 1 Mar 2013 12:23:36 +, Andrew Murray wrote:
This patch factors out common implementations patterns to reduce overall
kernel
code and provide a means for host bridge drivers to directly
On Thu, Mar 21, 2013 at 05:30:09PM +, Thomas Petazzoni wrote:
From: Andrew Murray andrew.mur...@arm.com
This patch factors out common implementations patterns to reduce overall
kernel
code and provide a means for host bridge drivers to directly obtain struct
resources from the DT's
common pci_controller
structures. All we need here is a parser that [at least] ARM can use to support
the pending ARM host-bridge drivers. Is its current form likely to be
acceptable?
I'm happy to update the patch for Thierry's suggestions.
Thanks,
Andrew Murray
On Fri, Mar 01, 2013 at 03:13:34PM +, Rob Herring wrote:
On 03/01/2013 06:23 AM, Andrew Murray wrote:
This patch factors out common implementations patterns to reduce overall
kernel
code and provide a means for host bridge drivers to directly obtain struct
resources from the DT's
an
accepted version for ARM yet but efforts are underway to add one that doesn't
duplicate code.
It would probably make sense for all these new PCIe host drivers to be
consistent where possible.
Andrew Murray
___
devicetree-discuss mailing list
devicetree
On Fri, Mar 01, 2013 at 03:13:34PM +, Rob Herring wrote:
On 03/01/2013 06:23 AM, Andrew Murray wrote:
This patch factors out common implementations patterns to reduce overall
kernel
code and provide a means for host bridge drivers to directly obtain struct
resources from the DT's
)
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
---
arch/microblaze/pci/pci-common.c | 100 +++--
arch/mips/pci/pci.c | 44 -
arch/powerpc/kernel/pci-common.c | 93
:
- This follows on from suggestions made by Grant Likely
(marc.info/?l=linux-kernelm=136079602806328)
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
---
arch/microblaze/pci/pci-common.c | 100 +++--
arch
On Fri, Feb 15, 2013 at 01:16:17PM +, Linus Walleij wrote:
n Thu, Feb 14, 2013 at 8:17 PM, Thierry Reding
thierry.red...@avionic-design.de wrote:
On Thu, Feb 14, 2013 at 04:53:41PM +, Andrew Murray wrote:
Thierry,
If you don't have much bandwidth I'd be quite happy to take
On Wed, Feb 13, 2013 at 10:53:11PM +, Grant Likely wrote:
On Mon, 11 Feb 2013 09:22:17 +0100, Thierry Reding
thierry.red...@avionic-design.de wrote:
From: Andrew Murray andrew.mur...@arm.com
DT bindings for PCI host bridges often use the ranges property to describe
memory and IO
On Tue, Jan 22, 2013 at 07:29:01PM +, Jason Gunthorpe wrote:
On Thu, Jan 17, 2013 at 04:22:18PM +, Andrew Murray wrote:
In either of those cases, does it make sense to use the MSI support
outside the scope of the PCI infrastructure? That is, would devices
other than PCI devices
On Thu, Jan 17, 2013 at 08:30:10PM +, Thierry Reding wrote:
On Thu, Jan 17, 2013 at 04:22:18PM +, Andrew Murray wrote:
On Thu, Jan 17, 2013 at 04:05:02PM +, Thierry Reding wrote:
On Thu, Jan 17, 2013 at 03:42:36PM +, Andrew Murray wrote:
On Wed, Jan 16, 2013 at 06:31:01PM
haven't
implemented hw-scan).
I have a patch for this if you want to fold it into your series? (I see you've
made changes to bios32 for per-controller data).
Andrew Murray
___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https
and MRRS.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
---
arch/arm/include/asm/mach/pci.h |2 ++
arch/arm/kernel/bios32.c| 29 -
2 files changed, 26 insertions(+), 5 deletions(-)
diff --git a/arch/arm
.
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
---
drivers/pci/msi.c | 113 +++
include/linux/msi.h | 16 +++-
2 files changed, 111 insertions(+), 18 deletions(-)
diff --git a/drivers/pci
, but it
doesn't quite solve this problem - perhaps it can be a starting point?
Andrew Murray
___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss
On Thu, Jan 17, 2013 at 04:05:02PM +, Thierry Reding wrote:
On Thu, Jan 17, 2013 at 03:42:36PM +, Andrew Murray wrote:
On Wed, Jan 16, 2013 at 06:31:01PM +, Thierry Reding wrote:
Alright, putting the functions into pci_ops doesn't sound like a very
good idea then. Or perhaps
interrupt handlers. And so the RC / MSI controllers don't need to
be aware of each other.
Andrew Murray
___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss
parts of the
fabric by different MSI controllers (assuming you relate MSI controllers to
part of the fabric and that you'd want to). Perhaps there would be benefits for
virtualisation as well?
Andrew Murray
___
devicetree-discuss mailing list
devicetree
On Thu, Dec 20, 2012 at 08:25:00AM +, Thierry Reding wrote:
On Wed, Dec 12, 2012 at 04:37:50PM +, Andrew Murray wrote:
[...]
diff --git a/drivers/of/address.c b/drivers/of/address.c
[...]
+ start = of_get_property(node, ranges, rlen);
+ if (start == NULL)
+ return
On Sat, Dec 15, 2012 at 01:06:41AM +, Grant Likely wrote:
On Wed, Dec 12, 2012 at 4:37 PM, Andrew Murray andrew.mur...@arm.com wrote:
DT bindings for PCI host bridges often use the ranges property to describe
memory and IO ranges - this binding tends to be the same across
architectures
as a pair or be provided by the same chip.
Perhaps the solution is to support MSI controller drivers and a means to
associate them with PCI host controller drivers?
Andrew Murray
___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
. of_bus_pci_get_flags,
bus-count_cells).
I'm not just ready to post it - but can do before early next week if you can
wait.
Andrew Murray
___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss
request
the obtained resources and pass them on (e.g. pci_add_resource_offset).
Signed-off-by: Andrew Murray andrew.mur...@arm.com
Signed-off-by: Liviu Dudau liviu.du...@arm.com
---
drivers/of/address.c | 53 +++-
include/linux/of_address.h |7
On Wed, Dec 12, 2012 at 01:34:24PM +, Thierry Reding wrote:
On Wed, Dec 12, 2012 at 12:19:12PM +, Andrew Murray wrote:
I've been working on a relatively architecture agnostic PCI host bridge
driver
and also wanted to avoid duplicating more generic DT parsing code for PCI
bindings
another patch for the typo and leave this patch with you for
working into your existing series.
Andrew Murray
___
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss
On Thu, Dec 13, 2012 at 10:03:18AM +, Thierry Reding wrote:
On Thu, Dec 13, 2012 at 09:45:43AM +, Andrew Murray wrote:
On Thu, Dec 13, 2012 at 09:13:33AM +, Thierry Reding wrote:
Hi Andrew,
I don't like iterator interfaces too much, but I can live with that.
Other than
63 matches
Mail list logo