Commit 3f7d67da authored by Baoyou Xie's avatar Baoyou Xie Committed by Martin K. Petersen

scsi: bnx2fc: Mark symbols static where possible

We get a few warnings when building kernel with W=1:
drivers/scsi/bnx2fc/bnx2fc_els.c:257:6: warning: no previous prototype for 'bnx2fc_srr_compl' [-Wmissing-prototypes]
drivers/scsi/bnx2fc/bnx2fc_els.c:367:6: warning: no previous prototype for 'bnx2fc_rec_compl' [-Wmissing-prototypes]
drivers/scsi/bnx2fc/bnx2fc_fcoe.c:628:5: warning: no previous prototype for 'bnx2fc_percpu_io_thread' [-Wmissing-prototypes]
drivers/scsi/bnx2fc/bnx2fc_fcoe.c:1413:26: warning: no previous prototype for 'bnx2fc_interface_create' [-Wmissing-prototypes]
drivers/scsi/bnx2fc/bnx2fc_hwi.c:997:21: warning: no previous prototype for 'bnx2fc_alloc_work' [-Wmissing-prototypes]
drivers/scsi/bnx2fc/bnx2fc_io.c:1082:5: warning: no previous prototype for 'bnx2fc_abts_cleanup' [-Wmissing-prototypes]
....

In fact, these functions are only used in the file in which they are
declared and don't need a declaration, but can be made static.  so this
patch marks these functions with 'static'.
Signed-off-by: default avatarBaoyou Xie <baoyou.xie@linaro.org>
Acked-by: default avatarArnd Bergmann <arnd@arndb.de>
Acked-by: default avatarChad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent a44199ee
...@@ -254,7 +254,7 @@ int bnx2fc_send_rls(struct bnx2fc_rport *tgt, struct fc_frame *fp) ...@@ -254,7 +254,7 @@ int bnx2fc_send_rls(struct bnx2fc_rport *tgt, struct fc_frame *fp)
return rc; return rc;
} }
void bnx2fc_srr_compl(struct bnx2fc_els_cb_arg *cb_arg) static void bnx2fc_srr_compl(struct bnx2fc_els_cb_arg *cb_arg)
{ {
struct bnx2fc_mp_req *mp_req; struct bnx2fc_mp_req *mp_req;
struct fc_frame_header *fc_hdr, *fh; struct fc_frame_header *fc_hdr, *fh;
...@@ -364,7 +364,7 @@ void bnx2fc_srr_compl(struct bnx2fc_els_cb_arg *cb_arg) ...@@ -364,7 +364,7 @@ void bnx2fc_srr_compl(struct bnx2fc_els_cb_arg *cb_arg)
kref_put(&orig_io_req->refcount, bnx2fc_cmd_release); kref_put(&orig_io_req->refcount, bnx2fc_cmd_release);
} }
void bnx2fc_rec_compl(struct bnx2fc_els_cb_arg *cb_arg) static void bnx2fc_rec_compl(struct bnx2fc_els_cb_arg *cb_arg)
{ {
struct bnx2fc_cmd *orig_io_req, *new_io_req; struct bnx2fc_cmd *orig_io_req, *new_io_req;
struct bnx2fc_cmd *rec_req; struct bnx2fc_cmd *rec_req;
......
...@@ -625,7 +625,7 @@ static void bnx2fc_recv_frame(struct sk_buff *skb) ...@@ -625,7 +625,7 @@ static void bnx2fc_recv_frame(struct sk_buff *skb)
* *
* @arg: ptr to bnx2fc_percpu_info structure * @arg: ptr to bnx2fc_percpu_info structure
*/ */
int bnx2fc_percpu_io_thread(void *arg) static int bnx2fc_percpu_io_thread(void *arg)
{ {
struct bnx2fc_percpu_s *p = arg; struct bnx2fc_percpu_s *p = arg;
struct bnx2fc_work *work, *tmp; struct bnx2fc_work *work, *tmp;
...@@ -1410,9 +1410,10 @@ static struct bnx2fc_hba *bnx2fc_hba_create(struct cnic_dev *cnic) ...@@ -1410,9 +1410,10 @@ static struct bnx2fc_hba *bnx2fc_hba_create(struct cnic_dev *cnic)
return NULL; return NULL;
} }
struct bnx2fc_interface *bnx2fc_interface_create(struct bnx2fc_hba *hba, static struct bnx2fc_interface *
struct net_device *netdev, bnx2fc_interface_create(struct bnx2fc_hba *hba,
enum fip_state fip_mode) struct net_device *netdev,
enum fip_state fip_mode)
{ {
struct fcoe_ctlr_device *ctlr_dev; struct fcoe_ctlr_device *ctlr_dev;
struct bnx2fc_interface *interface; struct bnx2fc_interface *interface;
......
...@@ -994,7 +994,7 @@ void bnx2fc_arm_cq(struct bnx2fc_rport *tgt) ...@@ -994,7 +994,7 @@ void bnx2fc_arm_cq(struct bnx2fc_rport *tgt)
} }
struct bnx2fc_work *bnx2fc_alloc_work(struct bnx2fc_rport *tgt, u16 wqe) static struct bnx2fc_work *bnx2fc_alloc_work(struct bnx2fc_rport *tgt, u16 wqe)
{ {
struct bnx2fc_work *work; struct bnx2fc_work *work;
work = kzalloc(sizeof(struct bnx2fc_work), GFP_ATOMIC); work = kzalloc(sizeof(struct bnx2fc_work), GFP_ATOMIC);
......
...@@ -1079,7 +1079,7 @@ int bnx2fc_eh_device_reset(struct scsi_cmnd *sc_cmd) ...@@ -1079,7 +1079,7 @@ int bnx2fc_eh_device_reset(struct scsi_cmnd *sc_cmd)
return bnx2fc_initiate_tmf(sc_cmd, FCP_TMF_LUN_RESET); return bnx2fc_initiate_tmf(sc_cmd, FCP_TMF_LUN_RESET);
} }
int bnx2fc_abts_cleanup(struct bnx2fc_cmd *io_req) static int bnx2fc_abts_cleanup(struct bnx2fc_cmd *io_req)
{ {
struct bnx2fc_rport *tgt = io_req->tgt; struct bnx2fc_rport *tgt = io_req->tgt;
int rc = SUCCESS; int rc = SUCCESS;
......
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