Commit 6498cbc5 authored by Nathan Chancellor's avatar Nathan Chancellor Committed by Martin K. Petersen
Browse files

scsi: bfa: Remove unused functions 

Clang warns when a variable is assigned to itself.

drivers/scsi/bfa/bfa_fcbuild.c:199:6: warning: explicitly assigning
value of variable of type 'int' to itself [-Wself-assign]
        len = len;
        ~~~ ^ ~~~
drivers/scsi/bfa/bfa_fcbuild.c:838:6: warning: explicitly assigning
value of variable of type 'int' to itself [-Wself-assign]
        len = len;
        ~~~ ^ ~~~
drivers/scsi/bfa/bfa_fcbuild.c:917:6: warning: explicitly assigning
value of variable of type 'int' to itself [-Wself-assign]
        len = len;
        ~~~ ^ ~~~
drivers/scsi/bfa/bfa_fcbuild.c:981:6: warning: explicitly assigning
value of variable of type 'int' to itself [-Wself-assign]
        len = len;
        ~~~ ^ ~~~
drivers/scsi/bfa/bfa_fcbuild.c:1008:6: warning: explicitly assigning
value of variable of type 'int' to itself [-Wself-assign]
        len = len;
        ~~~ ^ ~~~
5 warnings generated.

This construct is usually used to avoid unused variable warnings, which
I assume is the case here. -Wunused-parameter is hidden behind -Wextra
with GCC 4.6, which is the minimum version to compile the kernel as of
commit cafa0010 ("Raise the minimum required gcc version to 4.6").

However, upon further inspection, these functions aren't actually used
anywhere; they're just defined. Rather than just removing the self
assignments, remove all of this dead code.

Link: https://github.com/ClangBuiltLinux/linux/issues/148


Signed-off-by: default avatarNathan Chancellor <natechancellor@gmail.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 3e59790e

drivers/scsi/bfa/bfa_fcbuild.c

+0 −108
Original line number Diff line number Diff line
@@ -190,27 +190,6 @@ fc_els_rsp_build(struct fchs_s *fchs, u32 d_id, u32 s_id, __be16 ox_id) 
	fchs->ox_id = ox_id;

}



enum fc_parse_status

fc_els_rsp_parse(struct fchs_s *fchs, int len)

{

	struct fc_els_cmd_s *els_cmd = (struct fc_els_cmd_s *) (fchs + 1);

	struct fc_ls_rjt_s *ls_rjt = (struct fc_ls_rjt_s *) els_cmd;



	len = len;



	switch (els_cmd->els_code) {

	case FC_ELS_LS_RJT:

		if (ls_rjt->reason_code == FC_LS_RJT_RSN_LOGICAL_BUSY)

			return FC_PARSE_BUSY;

		else

			return FC_PARSE_FAILURE;



	case FC_ELS_ACC:

		return FC_PARSE_OK;

	}

	return FC_PARSE_OK;

}



static void

fc_bls_rsp_build(struct fchs_s *fchs, u32 d_id, u32 s_id, __be16 ox_id)

{
@@ -830,18 +809,6 @@ fc_rpsc_acc_build(struct fchs_s *fchs, struct fc_rpsc_acc_s *rpsc_acc, 
	return sizeof(struct fc_rpsc_acc_s);

}



u16

fc_logo_rsp_parse(struct fchs_s *fchs, int len)

{

	struct fc_els_cmd_s *els_cmd = (struct fc_els_cmd_s *) (fchs + 1);



	len = len;

	if (els_cmd->els_code != FC_ELS_ACC)

		return FC_PARSE_FAILURE;



	return FC_PARSE_OK;

}



u16

fc_pdisc_build(struct fchs_s *fchs, u32 d_id, u32 s_id, u16 ox_id,

	       wwn_t port_name, wwn_t node_name, u16 pdu_size)
@@ -907,40 +874,6 @@ fc_prlo_build(struct fchs_s *fchs, u32 d_id, u32 s_id, u16 ox_id, 
	return be16_to_cpu(prlo->payload_len);

}



u16

fc_prlo_rsp_parse(struct fchs_s *fchs, int len)

{

	struct fc_prlo_acc_s *prlo = (struct fc_prlo_acc_s *) (fchs + 1);

	int             num_pages = 0;

	int             page = 0;



	len = len;



	if (prlo->command != FC_ELS_ACC)

		return FC_PARSE_FAILURE;



	num_pages = ((be16_to_cpu(prlo->payload_len)) - 4) / 16;



	for (page = 0; page < num_pages; page++) {

		if (prlo->prlo_acc_params[page].type != FC_TYPE_FCP)

			return FC_PARSE_FAILURE;



		if (prlo->prlo_acc_params[page].opa_valid != 0)

			return FC_PARSE_FAILURE;



		if (prlo->prlo_acc_params[page].rpa_valid != 0)

			return FC_PARSE_FAILURE;



		if (prlo->prlo_acc_params[page].orig_process_assc != 0)

			return FC_PARSE_FAILURE;



		if (prlo->prlo_acc_params[page].resp_process_assc != 0)

			return FC_PARSE_FAILURE;

	}

	return FC_PARSE_OK;



}



u16

fc_tprlo_build(struct fchs_s *fchs, u32 d_id, u32 s_id, u16 ox_id,

	       int num_pages, enum fc_tprlo_type tprlo_type, u32 tpr_id)
@@ -971,47 +904,6 @@ fc_tprlo_build(struct fchs_s *fchs, u32 d_id, u32 s_id, u16 ox_id, 
	return be16_to_cpu(tprlo->payload_len);

}



u16

fc_tprlo_rsp_parse(struct fchs_s *fchs, int len)

{

	struct fc_tprlo_acc_s *tprlo = (struct fc_tprlo_acc_s *) (fchs + 1);

	int             num_pages = 0;

	int             page = 0;



	len = len;



	if (tprlo->command != FC_ELS_ACC)

		return FC_PARSE_ACC_INVAL;



	num_pages = (be16_to_cpu(tprlo->payload_len) - 4) / 16;



	for (page = 0; page < num_pages; page++) {

		if (tprlo->tprlo_acc_params[page].type != FC_TYPE_FCP)

			return FC_PARSE_NOT_FCP;

		if (tprlo->tprlo_acc_params[page].opa_valid != 0)

			return FC_PARSE_OPAFLAG_INVAL;

		if (tprlo->tprlo_acc_params[page].rpa_valid != 0)

			return FC_PARSE_RPAFLAG_INVAL;

		if (tprlo->tprlo_acc_params[page].orig_process_assc != 0)

			return FC_PARSE_OPA_INVAL;

		if (tprlo->tprlo_acc_params[page].resp_process_assc != 0)

			return FC_PARSE_RPA_INVAL;

	}

	return FC_PARSE_OK;

}



enum fc_parse_status

fc_rrq_rsp_parse(struct fchs_s *fchs, int len)

{

	struct fc_els_cmd_s *els_cmd = (struct fc_els_cmd_s *) (fchs + 1);



	len = len;

	if (els_cmd->els_code != FC_ELS_ACC)

		return FC_PARSE_FAILURE;



	return FC_PARSE_OK;

}



u16

fc_ba_rjt_build(struct fchs_s *fchs, u32 d_id, u32 s_id, __be16 ox_id,

		u32 reason_code, u32 reason_expl)

drivers/scsi/bfa/bfa_fcbuild.h

+0 −9
Original line number Diff line number Diff line
@@ -163,7 +163,6 @@ enum fc_parse_status fc_abts_rsp_parse(struct fchs_s *buf, int len); 


u16        fc_rrq_build(struct fchs_s *buf, struct fc_rrq_s *rrq, u32 d_id,

			     u32 s_id, u16 ox_id, u16 rrq_oxid);

enum fc_parse_status fc_rrq_rsp_parse(struct fchs_s *buf, int len);



u16        fc_rspnid_build(struct fchs_s *fchs, void *pld, u32 s_id,

				u16 ox_id, u8 *name);
@@ -276,8 +275,6 @@ void fc_get_fc4type_bitmask(u8 fc4_type, u8 *bit_mask); 
void		fc_els_req_build(struct fchs_s *fchs, u32 d_id, u32 s_id,

					 __be16 ox_id);



enum fc_parse_status	fc_els_rsp_parse(struct fchs_s *fchs, int len);



enum fc_parse_status	fc_plogi_rsp_parse(struct fchs_s *fchs, int len,

					wwn_t port_name);
@@ -297,8 +294,6 @@ u16 fc_tprlo_acc_build(struct fchs_s *fchs, struct fc_tprlo_acc_s *tprlo_acc, 
u16 fc_prlo_acc_build(struct fchs_s *fchs, struct fc_prlo_acc_s *prlo_acc,

		u32 d_id, u32 s_id, __be16 ox_id, int num_pages);



u16 fc_logo_rsp_parse(struct fchs_s *fchs, int len);



u16 fc_pdisc_build(struct fchs_s *fchs, u32 d_id, u32 s_id,

		u16 ox_id, wwn_t port_name, wwn_t node_name,

		u16 pdu_size);
@@ -308,14 +303,10 @@ u16 fc_pdisc_rsp_parse(struct fchs_s *fchs, int len, wwn_t port_name); 
u16 fc_prlo_build(struct fchs_s *fchs, u32 d_id, u32 s_id,

		u16 ox_id, int num_pages);



u16 fc_prlo_rsp_parse(struct fchs_s *fchs, int len);



u16 fc_tprlo_build(struct fchs_s *fchs, u32 d_id, u32 s_id,

		u16 ox_id, int num_pages, enum fc_tprlo_type tprlo_type,

		u32 tpr_id);



u16 fc_tprlo_rsp_parse(struct fchs_s *fchs, int len);



u16 fc_ba_rjt_build(struct fchs_s *fchs, u32 d_id, u32 s_id,

		__be16 ox_id, u32 reason_code, u32 reason_expl);

CRA Git | Maintained and supported by SUSTech CRA and CCSE