s390/pci: do not modify function handles

Don't modify function handles to get a disabled handle - call
clp_disable_fh. With this change we also do no longer deconfigure
enabled functions.

Reviewed-by: Gerald Schaefer <gerald.schaefer@de.ibm.com>
Signed-off-by: Sebastian Ott <sebott@linux.vnet.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>

1 files changed, 4 insertions, 5 deletions

diff --git a/drivers/pci/hotplug/s390_pci_hpc.c b/drivers/pci/hotplug/s390_pci_hpc.c
index 6053e7221b51..46a7b738f61f 100644
--- a/drivers/pci/hotplug/s390_pci_hpc.c
+++ b/drivers/pci/hotplug/s390_pci_hpc.c
@@ -68,17 +68,16 @@ static int disable_slot(struct hotplug_slot *hotplug_slot)
         if (!zpci_fn_configured(slot->zdev->state))
                 return -EIO;
 
+        rc = zpci_disable_device(slot->zdev);
+        if (rc)
+                return rc;
         /* TODO: we rely on the user to unbind/remove the device, is that plausible
          *       or do we need to trigger that here?
          */
         rc = sclp_pci_deconfigure(slot->zdev->fid);
         zpci_dbg(3, "deconf fid:%x, rc:%d\n", slot->zdev->fid, rc);
-        if (!rc) {
-                /* Fixme: better call List-PCI to find the disabled FH
-                   for the FID since the FH should be opaque... */
-                slot->zdev->fh &= 0x7fffffff;
+        if (!rc)
                 slot->zdev->state = ZPCI_FN_STATE_STANDBY;
-        }
         return rc;
 }