diff options
author | Robert Love <robert.w.love@intel.com> | 2009-02-27 13:55:50 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2009-03-10 10:05:09 -0400 |
commit | b2ab99c9a300e572105d6db7f6efe0a4d1572167 (patch) | |
tree | 1cdb9bd208e0cdb317b62f92c2c7879e3bc61335 /drivers/scsi/libfc/fc_exch.c | |
parent | 34f42a070fc98f5dc07e9fa2338b7b8d1dc347eb (diff) |
[SCSI] libfc, fcoe: Cleanup function formatting and minor typos
1) There were a few functions with a strange layout, i.e. all
arguments on the second line, when not necessary.
Where ever possible I moved the return value to the same line
as the function name. However, when the line was too long
to have a single argument on the same line I moved the
return value to above line. For example:
<short return> <function name>(<arg 1>, <arg2>)
and
<very long return value>
<function name>(<arg1>,
<arg2>)
2) Removed one extra whitespace line
3) Fixed two typos
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/scsi/libfc/fc_exch.c')
-rw-r--r-- | drivers/scsi/libfc/fc_exch.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/scsi/libfc/fc_exch.c b/drivers/scsi/libfc/fc_exch.c index 8c4018956d4c..7b933954c107 100644 --- a/drivers/scsi/libfc/fc_exch.c +++ b/drivers/scsi/libfc/fc_exch.c | |||
@@ -675,8 +675,8 @@ static struct fc_exch *fc_exch_resp(struct fc_exch_mgr *mp, struct fc_frame *fp) | |||
675 | * If fc_pf_rjt_reason is FC_RJT_NONE then this function will have a hold | 675 | * If fc_pf_rjt_reason is FC_RJT_NONE then this function will have a hold |
676 | * on the ep that should be released by the caller. | 676 | * on the ep that should be released by the caller. |
677 | */ | 677 | */ |
678 | static enum fc_pf_rjt_reason | 678 | static enum fc_pf_rjt_reason fc_seq_lookup_recip(struct fc_exch_mgr *mp, |
679 | fc_seq_lookup_recip(struct fc_exch_mgr *mp, struct fc_frame *fp) | 679 | struct fc_frame *fp) |
680 | { | 680 | { |
681 | struct fc_frame_header *fh = fc_frame_header_get(fp); | 681 | struct fc_frame_header *fh = fc_frame_header_get(fp); |
682 | struct fc_exch *ep = NULL; | 682 | struct fc_exch *ep = NULL; |
@@ -994,9 +994,9 @@ static void fc_seq_send_ack(struct fc_seq *sp, const struct fc_frame *rx_fp) | |||
994 | * Send BLS Reject. | 994 | * Send BLS Reject. |
995 | * This is for rejecting BA_ABTS only. | 995 | * This is for rejecting BA_ABTS only. |
996 | */ | 996 | */ |
997 | static void | 997 | static void fc_exch_send_ba_rjt(struct fc_frame *rx_fp, |
998 | fc_exch_send_ba_rjt(struct fc_frame *rx_fp, enum fc_ba_rjt_reason reason, | 998 | enum fc_ba_rjt_reason reason, |
999 | enum fc_ba_rjt_explan explan) | 999 | enum fc_ba_rjt_explan explan) |
1000 | { | 1000 | { |
1001 | struct fc_frame *fp; | 1001 | struct fc_frame *fp; |
1002 | struct fc_frame_header *rx_fh; | 1002 | struct fc_frame_header *rx_fh; |