Re: [PATCH] powerpc/pasemi: Seach for PCI root bus by compatible property
Olof Johansson writes: > On Wed, Aug 1, 2018 at 7:36 AM, Michael Ellerman wrote: >> Christian Zigotzky writes: >> >>> Just for info: I tested it on my Nemo board today and it works. >> >> Awesome thanks. >> >> That's probably sufficient to merge it, and if it breaks anything we can >> always revert it. > > Should be fine, all known boards have the properties in question, and > I doubt anyone has anything but Nemo and Electra/Chitra boards out > there. > > It's a virtual root bus, so all boards have it irrespective of what > PCIe is brought out. > > (I should hook up my test system and get it into the CI cycle again, > maybe this fall). > > > Acked-by: Olof Johansson Thanks. cheers
Re: [PATCH] powerpc/pasemi: Seach for PCI root bus by compatible property
On Wed, Aug 1, 2018 at 7:36 AM, Michael Ellerman wrote: > Christian Zigotzky writes: > >> Just for info: I tested it on my Nemo board today and it works. > > Awesome thanks. > > That's probably sufficient to merge it, and if it breaks anything we can > always revert it. Should be fine, all known boards have the properties in question, and I doubt anyone has anything but Nemo and Electra/Chitra boards out there. It's a virtual root bus, so all boards have it irrespective of what PCIe is brought out. (I should hook up my test system and get it into the CI cycle again, maybe this fall). Acked-by: Olof Johansson -Olof
Re: [PATCH] powerpc/pasemi: Seach for PCI root bus by compatible property
Christian Zigotzky writes:
> Just for info: I tested it on my Nemo board today and it works.
Awesome thanks.
That's probably sufficient to merge it, and if it breaks anything we can
always revert it.
cheers
> On 31 July 2018 at 2:04PM, Michael Ellerman wrote:
>> Michael Ellerman writes:
>>> Darren Stevens writes:
>>>
Pasemi arch code finds the root of the PCI-e bus by searching the
device-tree for a node called 'pxp'. But the root bus has a
compatible property of 'pasemi,rootbus' so search for that instead.
Signed-off-by: Darren Stevens
---
This works on the Amigaone X1000, I don't know if this method of
finding the pci bus was there bcause of earlier firmwares.
>>> Does anyone have another pasemi board they can test this on?
>>>
>>> The last time I plugged mine in it popped the power supply and took out
>>> power to half the office :) - I haven't had a chance to try it since.
>> I actually I remembered I have a device tree lying around from an electra.
>>
>> It has:
>>
>>[I] home:pxp@0,8000(7)(I)> lsprop name compatible
>>name "pxp"
>>compatible "pasemi,rootbus"
>> "pa-pxp"
>>
>>
>> So it looks like the patch would work fine on it at least.
>>
>> cheers
>>
diff --git a/arch/powerpc/platforms/pasemi/pci.c
b/arch/powerpc/platforms/pasemi/pci.c
index c7c8607..be62380 100644
--- a/arch/powerpc/platforms/pasemi/pci.c
+++ b/arch/powerpc/platforms/pasemi/pci.c
@@ -216,6 +216,7 @@ static int __init pas_add_bridge(struct device_node
*dev)
void __init pas_pci_init(void)
{
struct device_node *np, *root;
+ int res;
root = of_find_node_by_path("/");
if (!root) {
@@ -226,11 +227,11 @@ void __init pas_pci_init(void)
pci_set_flags(PCI_SCAN_ALL_PCIE_DEVS);
- for (np = NULL; (np = of_get_next_child(root, np)) != NULL;)
- if (np->name && !strcmp(np->name, "pxp") && !pas_add_bridge(np))
- of_node_get(np);
-
- of_node_put(root);
+ np = of_find_compatible_node(root, NULL, "pasemi,rootbus");
+ if (np) {
+ res = pas_add_bridge(np);
+ of_node_put(np);
+ }
}
void __iomem *pasemi_pci_getcfgaddr(struct pci_dev *dev, int offset)
[PATCH] powerpc/pasemi: Seach for PCI root bus by compatible property
Just for info: I tested it on my Nemo board today and it works.
-- Christian
On 31 July 2018 at 2:04PM, Michael Ellerman wrote:
Michael Ellerman writes:
Darren Stevens writes:
Pasemi arch code finds the root of the PCI-e bus by searching the
device-tree for a node called 'pxp'. But the root bus has a
compatible property of 'pasemi,rootbus' so search for that instead.
Signed-off-by: Darren Stevens
---
This works on the Amigaone X1000, I don't know if this method of
finding the pci bus was there bcause of earlier firmwares.
Does anyone have another pasemi board they can test this on?
The last time I plugged mine in it popped the power supply and took out
power to half the office :) - I haven't had a chance to try it since.
I actually I remembered I have a device tree lying around from an electra.
It has:
[I] home:pxp@0,8000(7)(I)> lsprop name compatible
name "pxp"
compatible "pasemi,rootbus"
"pa-pxp"
So it looks like the patch would work fine on it at least.
cheers
diff --git a/arch/powerpc/platforms/pasemi/pci.c
b/arch/powerpc/platforms/pasemi/pci.c
index c7c8607..be62380 100644
--- a/arch/powerpc/platforms/pasemi/pci.c
+++ b/arch/powerpc/platforms/pasemi/pci.c
@@ -216,6 +216,7 @@ static int __init pas_add_bridge(struct device_node *dev)
void __init pas_pci_init(void)
{
struct device_node *np, *root;
+ int res;
root = of_find_node_by_path("/");
if (!root) {
@@ -226,11 +227,11 @@ void __init pas_pci_init(void)
pci_set_flags(PCI_SCAN_ALL_PCIE_DEVS);
- for (np = NULL; (np = of_get_next_child(root, np)) != NULL;)
- if (np->name && !strcmp(np->name, "pxp") && !pas_add_bridge(np))
- of_node_get(np);
-
- of_node_put(root);
+ np = of_find_compatible_node(root, NULL, "pasemi,rootbus");
+ if (np) {
+ res = pas_add_bridge(np);
+ of_node_put(np);
+ }
}
void __iomem *pasemi_pci_getcfgaddr(struct pci_dev *dev, int offset)
Re: [PATCH] powerpc/pasemi: Seach for PCI root bus by compatible property
Michael Ellerman writes:
> Darren Stevens writes:
>
>> Pasemi arch code finds the root of the PCI-e bus by searching the
>> device-tree for a node called 'pxp'. But the root bus has a
>> compatible property of 'pasemi,rootbus' so search for that instead.
>>
>> Signed-off-by: Darren Stevens
>> ---
>>
>> This works on the Amigaone X1000, I don't know if this method of
>> finding the pci bus was there bcause of earlier firmwares.
>
> Does anyone have another pasemi board they can test this on?
>
> The last time I plugged mine in it popped the power supply and took out
> power to half the office :) - I haven't had a chance to try it since.
I actually I remembered I have a device tree lying around from an electra.
It has:
[I] home:pxp@0,8000(7)(I)> lsprop name compatible
name "pxp"
compatible "pasemi,rootbus"
"pa-pxp"
So it looks like the patch would work fine on it at least.
cheers
>> diff --git a/arch/powerpc/platforms/pasemi/pci.c
>> b/arch/powerpc/platforms/pasemi/pci.c
>> index c7c8607..be62380 100644
>> --- a/arch/powerpc/platforms/pasemi/pci.c
>> +++ b/arch/powerpc/platforms/pasemi/pci.c
>> @@ -216,6 +216,7 @@ static int __init pas_add_bridge(struct device_node *dev)
>> void __init pas_pci_init(void)
>> {
>> struct device_node *np, *root;
>> + int res;
>>
>> root = of_find_node_by_path("/");
>> if (!root) {
>> @@ -226,11 +227,11 @@ void __init pas_pci_init(void)
>>
>> pci_set_flags(PCI_SCAN_ALL_PCIE_DEVS);
>>
>> - for (np = NULL; (np = of_get_next_child(root, np)) != NULL;)
>> - if (np->name && !strcmp(np->name, "pxp") && !pas_add_bridge(np))
>> - of_node_get(np);
>> -
>> - of_node_put(root);
>> + np = of_find_compatible_node(root, NULL, "pasemi,rootbus");
>> + if (np) {
>> + res = pas_add_bridge(np);
>> + of_node_put(np);
>> + }
>> }
>>
>> void __iomem *pasemi_pci_getcfgaddr(struct pci_dev *dev, int offset)
Re: [PATCH] powerpc/pasemi: Seach for PCI root bus by compatible property
Darren Stevens writes:
> Pasemi arch code finds the root of the PCI-e bus by searching the
> device-tree for a node called 'pxp'. But the root bus has a
> compatible property of 'pasemi,rootbus' so search for that instead.
>
> Signed-off-by: Darren Stevens
> ---
>
> This works on the Amigaone X1000, I don't know if this method of
> finding the pci bus was there bcause of earlier firmwares.
Does anyone have another pasemi board they can test this on?
The last time I plugged mine in it popped the power supply and took out
power to half the office :) - I haven't had a chance to try it since.
cheers
> diff --git a/arch/powerpc/platforms/pasemi/pci.c
> b/arch/powerpc/platforms/pasemi/pci.c
> index c7c8607..be62380 100644
> --- a/arch/powerpc/platforms/pasemi/pci.c
> +++ b/arch/powerpc/platforms/pasemi/pci.c
> @@ -216,6 +216,7 @@ static int __init pas_add_bridge(struct device_node *dev)
> void __init pas_pci_init(void)
> {
> struct device_node *np, *root;
> + int res;
>
> root = of_find_node_by_path("/");
> if (!root) {
> @@ -226,11 +227,11 @@ void __init pas_pci_init(void)
>
> pci_set_flags(PCI_SCAN_ALL_PCIE_DEVS);
>
> - for (np = NULL; (np = of_get_next_child(root, np)) != NULL;)
> - if (np->name && !strcmp(np->name, "pxp") && !pas_add_bridge(np))
> - of_node_get(np);
> -
> - of_node_put(root);
> + np = of_find_compatible_node(root, NULL, "pasemi,rootbus");
> + if (np) {
> + res = pas_add_bridge(np);
> + of_node_put(np);
> + }
> }
>
> void __iomem *pasemi_pci_getcfgaddr(struct pci_dev *dev, int offset)
[PATCH] powerpc/pasemi: Seach for PCI root bus by compatible property
Pasemi arch code finds the root of the PCI-e bus by searching the
device-tree for a node called 'pxp'. But the root bus has a
compatible property of 'pasemi,rootbus' so search for that instead.
Signed-off-by: Darren Stevens
---
This works on the Amigaone X1000, I don't know if this method of
finding the pci bus was there bcause of earlier firmwares.
diff --git a/arch/powerpc/platforms/pasemi/pci.c
b/arch/powerpc/platforms/pasemi/pci.c
index c7c8607..be62380 100644
--- a/arch/powerpc/platforms/pasemi/pci.c
+++ b/arch/powerpc/platforms/pasemi/pci.c
@@ -216,6 +216,7 @@ static int __init pas_add_bridge(struct device_node *dev)
void __init pas_pci_init(void)
{
struct device_node *np, *root;
+ int res;
root = of_find_node_by_path("/");
if (!root) {
@@ -226,11 +227,11 @@ void __init pas_pci_init(void)
pci_set_flags(PCI_SCAN_ALL_PCIE_DEVS);
- for (np = NULL; (np = of_get_next_child(root, np)) != NULL;)
- if (np->name && !strcmp(np->name, "pxp") && !pas_add_bridge(np))
- of_node_get(np);
-
- of_node_put(root);
+ np = of_find_compatible_node(root, NULL, "pasemi,rootbus");
+ if (np) {
+ res = pas_add_bridge(np);
+ of_node_put(np);
+ }
}
void __iomem *pasemi_pci_getcfgaddr(struct pci_dev *dev, int offset)
