| Message ID | 1449033440-26213-1-git-send-email-gwshan@linux.vnet.ibm.com |
|---|---|
| State | Not Applicable |
| Headers | show |
On Wed, Dec 02, 2015 at 04:17:20PM +1100, Gavin Shan wrote: Please ignore the noise as it should have been sent to linux-ppc (not linux-pci) maillist. Thanks, Gavin >In eeh_pe_loc_get(), the PE location code is retrieved from the >"ibm,loc-code" property of the device node for the bridge of the >PE's primary bus. It's not correct because the property indicates >the parent PE's location code. > >This reads the correct PE location code from "ibm,io-base-loc-code" >or "ibm,slot-location-code" property of PE parent bus's device node. > >Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> >--- > arch/powerpc/kernel/eeh_pe.c | 33 +++++++++++++++------------------ > 1 file changed, 15 insertions(+), 18 deletions(-) > >diff --git a/arch/powerpc/kernel/eeh_pe.c b/arch/powerpc/kernel/eeh_pe.c >index 8654cb1..ca9e537 100644 >--- a/arch/powerpc/kernel/eeh_pe.c >+++ b/arch/powerpc/kernel/eeh_pe.c >@@ -883,32 +883,29 @@ void eeh_pe_restore_bars(struct eeh_pe *pe) > const char *eeh_pe_loc_get(struct eeh_pe *pe) > { > struct pci_bus *bus = eeh_pe_bus_get(pe); >- struct device_node *dn = pci_bus_to_OF_node(bus); >+ struct device_node *dn; > const char *loc = NULL; > >- if (!dn) >- goto out; >+ while (bus) { >+ dn = pci_bus_to_OF_node(bus); >+ if (!dn) { >+ bus = bus->parent; >+ continue; >+ } > >- /* PHB PE or root PE ? */ >- if (pci_is_root_bus(bus)) { >- loc = of_get_property(dn, "ibm,loc-code", NULL); >- if (!loc) >+ if (pci_is_root_bus(bus)) > loc = of_get_property(dn, "ibm,io-base-loc-code", NULL); >+ else >+ loc = of_get_property(dn, "ibm,slot-location-code", >+ NULL); >+ > if (loc) >- goto out; >+ return loc; > >- /* Check the root port */ >- dn = dn->child; >- if (!dn) >- goto out; >+ bus = bus->parent; > } > >- loc = of_get_property(dn, "ibm,loc-code", NULL); >- if (!loc) >- loc = of_get_property(dn, "ibm,slot-location-code", NULL); >- >-out: >- return loc ? loc : "N/A"; >+ return "N/A"; > } > > /** >-- >2.1.0 > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/powerpc/kernel/eeh_pe.c b/arch/powerpc/kernel/eeh_pe.c index 8654cb1..ca9e537 100644 --- a/arch/powerpc/kernel/eeh_pe.c +++ b/arch/powerpc/kernel/eeh_pe.c @@ -883,32 +883,29 @@ void eeh_pe_restore_bars(struct eeh_pe *pe) const char *eeh_pe_loc_get(struct eeh_pe *pe) { struct pci_bus *bus = eeh_pe_bus_get(pe); - struct device_node *dn = pci_bus_to_OF_node(bus); + struct device_node *dn; const char *loc = NULL; - if (!dn) - goto out; + while (bus) { + dn = pci_bus_to_OF_node(bus); + if (!dn) { + bus = bus->parent; + continue; + } - /* PHB PE or root PE ? */ - if (pci_is_root_bus(bus)) { - loc = of_get_property(dn, "ibm,loc-code", NULL); - if (!loc) + if (pci_is_root_bus(bus)) loc = of_get_property(dn, "ibm,io-base-loc-code", NULL); + else + loc = of_get_property(dn, "ibm,slot-location-code", + NULL); + if (loc) - goto out; + return loc; - /* Check the root port */ - dn = dn->child; - if (!dn) - goto out; + bus = bus->parent; } - loc = of_get_property(dn, "ibm,loc-code", NULL); - if (!loc) - loc = of_get_property(dn, "ibm,slot-location-code", NULL); - -out: - return loc ? loc : "N/A"; + return "N/A"; } /**
In eeh_pe_loc_get(), the PE location code is retrieved from the "ibm,loc-code" property of the device node for the bridge of the PE's primary bus. It's not correct because the property indicates the parent PE's location code. This reads the correct PE location code from "ibm,io-base-loc-code" or "ibm,slot-location-code" property of PE parent bus's device node. Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> --- arch/powerpc/kernel/eeh_pe.c | 33 +++++++++++++++------------------ 1 file changed, 15 insertions(+), 18 deletions(-)