It will only ever be set so another callback, and the pointer to
this callback is available on all locations. So kill it.

Signed-off-by: Hannes Reinecke <h...@suse.com>
---
 drivers/scsi/libfc/fc_rport.c | 13 ++++++-------
 include/scsi/libfc.h          |  3 ---
 2 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/drivers/scsi/libfc/fc_rport.c b/drivers/scsi/libfc/fc_rport.c
index 0da34c7a6866..255e6568be66 100644
--- a/drivers/scsi/libfc/fc_rport.c
+++ b/drivers/scsi/libfc/fc_rport.c
@@ -155,10 +155,9 @@ struct fc_rport_priv *fc_rport_create(struct fc_lport 
*lport, u32 port_id)
        rdata->maxframe_size = FC_MIN_MAX_PAYLOAD;
        INIT_DELAYED_WORK(&rdata->retry_work, fc_rport_timeout);
        INIT_WORK(&rdata->event_work, fc_rport_work);
-       if (port_id != FC_FID_DIR_SERV) {
-               rdata->lld_event_callback = lport->tt.rport_event_callback;
+       if (port_id != FC_FID_DIR_SERV)
                list_add_rcu(&rdata->peers, &lport->disc.rports);
-       }
+
        return rdata;
 }
 EXPORT_SYMBOL(fc_rport_create);
@@ -308,9 +307,9 @@ static void fc_rport_work(struct work_struct *work)
                        FC_RPORT_DBG(rdata, "callback ev %d\n", event);
                        rport_ops->event_callback(lport, rdata, event);
                }
-               if (rdata->lld_event_callback) {
+               if (lport->tt.rport_event_callback) {
                        FC_RPORT_DBG(rdata, "lld callback ev %d\n", event);
-                       rdata->lld_event_callback(lport, rdata, event);
+                       lport->tt.rport_event_callback(lport, rdata, event);
                }
                kref_put(&rdata->kref, fc_rport_destroy);
                break;
@@ -334,9 +333,9 @@ static void fc_rport_work(struct work_struct *work)
                        FC_RPORT_DBG(rdata, "callback ev %d\n", event);
                        rport_ops->event_callback(lport, rdata, event);
                }
-               if (rdata->lld_event_callback) {
+               if (lport->tt.rport_event_callback) {
                        FC_RPORT_DBG(rdata, "lld callback ev %d\n", event);
-                       rdata->lld_event_callback(lport, rdata, event);
+                       lport->tt.rport_event_callback(lport, rdata, event);
                }
                if (cancel_delayed_work_sync(&rdata->retry_work))
                        kref_put(&rdata->kref, fc_rport_destroy);
diff --git a/include/scsi/libfc.h b/include/scsi/libfc.h
index 76cb9192319a..2c3c5b9e7cc6 100644
--- a/include/scsi/libfc.h
+++ b/include/scsi/libfc.h
@@ -212,9 +212,6 @@ struct fc_rport_priv {
        struct rcu_head             rcu;
        u16                         sp_features;
        u8                          spp_type;
-       void                        (*lld_event_callback)(struct fc_lport *,
-                                                     struct fc_rport_priv *,
-                                                     enum fc_rport_event);
 };
 
 /**
-- 
2.16.4

Reply via email to