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/fcoe | |
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/fcoe')
-rw-r--r-- | drivers/scsi/fcoe/fc_transport_fcoe.c | 11 | ||||
-rw-r--r-- | drivers/scsi/fcoe/libfcoe.c | 15 |
2 files changed, 13 insertions, 13 deletions
diff --git a/drivers/scsi/fcoe/fc_transport_fcoe.c b/drivers/scsi/fcoe/fc_transport_fcoe.c index 0eea4c434c31..847453a70c62 100644 --- a/drivers/scsi/fcoe/fc_transport_fcoe.c +++ b/drivers/scsi/fcoe/fc_transport_fcoe.c | |||
@@ -62,8 +62,9 @@ struct pci_dev *fcoe_transport_pcidev(const struct net_device *netdev) | |||
62 | * | 62 | * |
63 | * Returns: 0 for success | 63 | * Returns: 0 for success |
64 | */ | 64 | */ |
65 | static struct fcoe_transport_internal *fcoe_transport_device_lookup( | 65 | static struct fcoe_transport_internal * |
66 | struct fcoe_transport *t, struct net_device *netdev) | 66 | fcoe_transport_device_lookup(struct fcoe_transport *t, |
67 | struct net_device *netdev) | ||
67 | { | 68 | { |
68 | struct fcoe_transport_internal *ti; | 69 | struct fcoe_transport_internal *ti; |
69 | 70 | ||
@@ -184,7 +185,7 @@ static void fcoe_transport_device_remove_all(struct fcoe_transport *t) | |||
184 | * Returns: true for match up | 185 | * Returns: true for match up |
185 | */ | 186 | */ |
186 | static bool fcoe_transport_match(struct fcoe_transport *t, | 187 | static bool fcoe_transport_match(struct fcoe_transport *t, |
187 | struct net_device *netdev) | 188 | struct net_device *netdev) |
188 | { | 189 | { |
189 | /* match transport by vendor and device id */ | 190 | /* match transport by vendor and device id */ |
190 | struct pci_dev *pci; | 191 | struct pci_dev *pci; |
@@ -217,8 +218,8 @@ static bool fcoe_transport_match(struct fcoe_transport *t, | |||
217 | * | 218 | * |
218 | * TODO: return default sw transport if no other transport is found | 219 | * TODO: return default sw transport if no other transport is found |
219 | */ | 220 | */ |
220 | static struct fcoe_transport *fcoe_transport_lookup( | 221 | static struct fcoe_transport * |
221 | struct net_device *netdev) | 222 | fcoe_transport_lookup(struct net_device *netdev) |
222 | { | 223 | { |
223 | struct fcoe_transport *t; | 224 | struct fcoe_transport *t; |
224 | 225 | ||
diff --git a/drivers/scsi/fcoe/libfcoe.c b/drivers/scsi/fcoe/libfcoe.c index 5e68652985b2..d006d6576817 100644 --- a/drivers/scsi/fcoe/libfcoe.c +++ b/drivers/scsi/fcoe/libfcoe.c | |||
@@ -949,8 +949,8 @@ static struct net_device *fcoe_if_to_netdev(const char *buffer) | |||
949 | * | 949 | * |
950 | * Returns: ptr to the struct module, NULL for failure | 950 | * Returns: ptr to the struct module, NULL for failure |
951 | */ | 951 | */ |
952 | static struct module *fcoe_netdev_to_module_owner( | 952 | static struct module * |
953 | const struct net_device *netdev) | 953 | fcoe_netdev_to_module_owner(const struct net_device *netdev) |
954 | { | 954 | { |
955 | struct device *dev; | 955 | struct device *dev; |
956 | 956 | ||
@@ -1205,8 +1205,8 @@ EXPORT_SYMBOL_GPL(fcoe_clean_pending_queue); | |||
1205 | * Returns: ptr to Scsi_Host | 1205 | * Returns: ptr to Scsi_Host |
1206 | * TODO: to libfc? | 1206 | * TODO: to libfc? |
1207 | */ | 1207 | */ |
1208 | static inline struct Scsi_Host *libfc_host_alloc( | 1208 | static inline struct Scsi_Host * |
1209 | struct scsi_host_template *sht, int priv_size) | 1209 | libfc_host_alloc(struct scsi_host_template *sht, int priv_size) |
1210 | { | 1210 | { |
1211 | return scsi_host_alloc(sht, sizeof(struct fc_lport) + priv_size); | 1211 | return scsi_host_alloc(sht, sizeof(struct fc_lport) + priv_size); |
1212 | } | 1212 | } |
@@ -1285,8 +1285,8 @@ EXPORT_SYMBOL_GPL(fcoe_wwn_from_mac); | |||
1285 | * | 1285 | * |
1286 | * Returns: NULL or the located fcoe_softc | 1286 | * Returns: NULL or the located fcoe_softc |
1287 | */ | 1287 | */ |
1288 | static struct fcoe_softc *fcoe_hostlist_lookup_softc( | 1288 | static struct fcoe_softc * |
1289 | const struct net_device *dev) | 1289 | fcoe_hostlist_lookup_softc(const struct net_device *dev) |
1290 | { | 1290 | { |
1291 | struct fcoe_softc *fc; | 1291 | struct fcoe_softc *fc; |
1292 | 1292 | ||
@@ -1426,7 +1426,6 @@ static int __init fcoe_init(void) | |||
1426 | } else { | 1426 | } else { |
1427 | fcoe_percpu[cpu] = NULL; | 1427 | fcoe_percpu[cpu] = NULL; |
1428 | kfree(p); | 1428 | kfree(p); |
1429 | |||
1430 | } | 1429 | } |
1431 | } | 1430 | } |
1432 | } | 1431 | } |
@@ -1476,7 +1475,7 @@ static void __exit fcoe_exit(void) | |||
1476 | */ | 1475 | */ |
1477 | del_timer_sync(&fcoe_timer); | 1476 | del_timer_sync(&fcoe_timer); |
1478 | 1477 | ||
1479 | /* releases the assocaited fcoe transport for each lport */ | 1478 | /* releases the associated fcoe transport for each lport */ |
1480 | list_for_each_entry_safe(fc, tmp, &fcoe_hostlist, list) | 1479 | list_for_each_entry_safe(fc, tmp, &fcoe_hostlist, list) |
1481 | fcoe_transport_release(fc->real_dev); | 1480 | fcoe_transport_release(fc->real_dev); |
1482 | 1481 | ||