isci: kill isci_host.shost
authorDan Williams <dan.j.williams@intel.com>
Sat, 25 Feb 2012 22:29:49 +0000 (14:29 -0800)
committerDan Williams <dan.j.williams@intel.com>
Thu, 17 May 2012 19:27:13 +0000 (12:27 -0700)
We can retrieve the shost from the sas_ha like the rest of libsas and
drop this out of our local data structure.

Acked-by: Jacek Danecki <jacek.danecki@intel.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
drivers/scsi/isci/host.h
drivers/scsi/isci/init.c

index 9701c1d..7272a0a 100644 (file)
@@ -203,7 +203,6 @@ struct isci_host {
        #define IHOST_IRQ_ENABLED 2
        unsigned long flags;
        wait_queue_head_t eventq;
-       struct Scsi_Host *shost;
        struct tasklet_struct completion_tasklet;
        struct list_head requests_to_complete;
        struct list_head requests_to_errorback;
@@ -308,6 +307,11 @@ static inline struct isci_pci_info *to_pci_info(struct pci_dev *pdev)
        return pci_get_drvdata(pdev);
 }
 
+static inline struct Scsi_Host *to_shost(struct isci_host *ihost)
+{
+       return ihost->sas_ha.core.shost;
+}
+
 #define for_each_isci_host(id, ihost, pdev) \
        for (id = 0, ihost = to_pci_info(pdev)->hosts[id]; \
             id < ARRAY_SIZE(to_pci_info(pdev)->hosts) && ihost; \
index eda4385..fdae42f 100644 (file)
@@ -271,13 +271,12 @@ static void isci_unregister(struct isci_host *isci_host)
        if (!isci_host)
                return;
 
-       shost = isci_host->shost;
-
        sas_unregister_ha(&isci_host->sas_ha);
 
-       sas_remove_host(isci_host->shost);
-       scsi_remove_host(isci_host->shost);
-       scsi_host_put(isci_host->shost);
+       shost = to_shost(isci_host);
+       sas_remove_host(shost);
+       scsi_remove_host(shost);
+       scsi_host_put(shost);
 }
 
 static int __devinit isci_pci_init(struct pci_dev *pdev)
@@ -578,7 +577,6 @@ static struct isci_host *isci_host_alloc(struct pci_dev *pdev, int id)
        shost = scsi_host_alloc(&isci_sht, sizeof(void *));
        if (!shost)
                return NULL;
-       ihost->shost = shost;
 
        dev_info(&pdev->dev, "%sSCU controller %d: phy 3-0 cables: "
                 "{%s, %s, %s, %s}\n",
@@ -690,11 +688,11 @@ static int __devinit isci_pci_probe(struct pci_dev *pdev, const struct pci_devic
                pci_info->hosts[i] = h;
 
                /* turn on DIF support */
-               scsi_host_set_prot(h->shost,
+               scsi_host_set_prot(to_shost(h),
                                   SHOST_DIF_TYPE1_PROTECTION |
                                   SHOST_DIF_TYPE2_PROTECTION |
                                   SHOST_DIF_TYPE3_PROTECTION);
-               scsi_host_set_guard(h->shost, SHOST_DIX_GUARD_CRC);
+               scsi_host_set_guard(to_shost(h), SHOST_DIX_GUARD_CRC);
        }
 
        err = isci_setup_interrupts(pdev);
@@ -702,7 +700,7 @@ static int __devinit isci_pci_probe(struct pci_dev *pdev, const struct pci_devic
                goto err_host_alloc;
 
        for_each_isci_host(i, isci_host, pdev)
-               scsi_scan_host(isci_host->shost);
+               scsi_scan_host(to_shost(isci_host));
 
        return 0;