Commit 9a3eda26 authored by Sam Bobroff's avatar Sam Bobroff Committed by Michael Ellerman

powerpc/eeh: Cleanup logic in eeh_rmv_from_parent_pe()

Move the call to eeh_dev_to_pe() up, so that later it's clear that
"pe" isn't NULL.
Signed-off-by: default avatarSam Bobroff <sbobroff@linux.ibm.com>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent 1c5c533b
...@@ -456,7 +456,8 @@ int eeh_rmv_from_parent_pe(struct eeh_dev *edev) ...@@ -456,7 +456,8 @@ int eeh_rmv_from_parent_pe(struct eeh_dev *edev)
int cnt; int cnt;
struct pci_dn *pdn = eeh_dev_to_pdn(edev); struct pci_dn *pdn = eeh_dev_to_pdn(edev);
if (!edev->pe) { pe = eeh_dev_to_pe(edev);
if (!pe) {
pr_debug("%s: No PE found for device %04x:%02x:%02x.%01x\n", pr_debug("%s: No PE found for device %04x:%02x:%02x.%01x\n",
__func__, pdn->phb->global_number, __func__, pdn->phb->global_number,
pdn->busno, pdn->busno,
...@@ -466,7 +467,6 @@ int eeh_rmv_from_parent_pe(struct eeh_dev *edev) ...@@ -466,7 +467,6 @@ int eeh_rmv_from_parent_pe(struct eeh_dev *edev)
} }
/* Remove the EEH device */ /* Remove the EEH device */
pe = eeh_dev_to_pe(edev);
edev->pe = NULL; edev->pe = NULL;
list_del(&edev->entry); list_del(&edev->entry);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment