aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/scsi/fcoe
diff options
context:
space:
mode:
authorRobert Love <robert.w.love@intel.com>2009-02-27 13:55:50 -0500
committerJames Bottomley <James.Bottomley@HansenPartnership.com>2009-03-10 10:05:09 -0400
commitb2ab99c9a300e572105d6db7f6efe0a4d1572167 (patch)
tree1cdb9bd208e0cdb317b62f92c2c7879e3bc61335 /drivers/scsi/fcoe
parent34f42a070fc98f5dc07e9fa2338b7b8d1dc347eb (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.c11
-rw-r--r--drivers/scsi/fcoe/libfcoe.c15
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 */
65static struct fcoe_transport_internal *fcoe_transport_device_lookup( 65static struct fcoe_transport_internal *
66 struct fcoe_transport *t, struct net_device *netdev) 66fcoe_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 */
186static bool fcoe_transport_match(struct fcoe_transport *t, 187static 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 */
220static struct fcoe_transport *fcoe_transport_lookup( 221static struct fcoe_transport *
221 struct net_device *netdev) 222fcoe_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 */
952static struct module *fcoe_netdev_to_module_owner( 952static struct module *
953 const struct net_device *netdev) 953fcoe_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 */
1208static inline struct Scsi_Host *libfc_host_alloc( 1208static inline struct Scsi_Host *
1209 struct scsi_host_template *sht, int priv_size) 1209libfc_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 */
1288static struct fcoe_softc *fcoe_hostlist_lookup_softc( 1288static struct fcoe_softc *
1289 const struct net_device *dev) 1289fcoe_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