diff options
| author | Bart Van Assche <bvanassche@acm.org> | 2013-06-25 13:49:40 -0400 |
|---|---|---|
| committer | Robert Love <robert.w.love@intel.com> | 2013-07-09 14:19:00 -0400 |
| commit | 7a5ed75a782a1f2de7fc773981ecd88bfa7595b1 (patch) | |
| tree | eff7771c4bdf5321d8c745abf5916b1c2bbde845 | |
| parent | d17efa001aacaa774581972230b1a9db1e618ded (diff) | |
fcoe: Reduce number of sparse warnings
Declare local variables and functions 'static'. This patch does not
change any functionality.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Robert Love <robert.w.love@intel.com>
| -rw-r--r-- | drivers/scsi/fcoe/fcoe_sysfs.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/drivers/scsi/fcoe/fcoe_sysfs.c b/drivers/scsi/fcoe/fcoe_sysfs.c index 8c05ae017f5b..c9382d6eee78 100644 --- a/drivers/scsi/fcoe/fcoe_sysfs.c +++ b/drivers/scsi/fcoe/fcoe_sysfs.c | |||
| @@ -507,7 +507,7 @@ static const struct attribute_group *fcoe_fcf_attr_groups[] = { | |||
| 507 | NULL, | 507 | NULL, |
| 508 | }; | 508 | }; |
| 509 | 509 | ||
| 510 | struct bus_type fcoe_bus_type; | 510 | static struct bus_type fcoe_bus_type; |
| 511 | 511 | ||
| 512 | static int fcoe_bus_match(struct device *dev, | 512 | static int fcoe_bus_match(struct device *dev, |
| 513 | struct device_driver *drv) | 513 | struct device_driver *drv) |
| @@ -541,25 +541,25 @@ static void fcoe_fcf_device_release(struct device *dev) | |||
| 541 | kfree(fcf); | 541 | kfree(fcf); |
| 542 | } | 542 | } |
| 543 | 543 | ||
| 544 | struct device_type fcoe_ctlr_device_type = { | 544 | static struct device_type fcoe_ctlr_device_type = { |
| 545 | .name = "fcoe_ctlr", | 545 | .name = "fcoe_ctlr", |
| 546 | .groups = fcoe_ctlr_attr_groups, | 546 | .groups = fcoe_ctlr_attr_groups, |
| 547 | .release = fcoe_ctlr_device_release, | 547 | .release = fcoe_ctlr_device_release, |
| 548 | }; | 548 | }; |
| 549 | 549 | ||
| 550 | struct device_type fcoe_fcf_device_type = { | 550 | static struct device_type fcoe_fcf_device_type = { |
| 551 | .name = "fcoe_fcf", | 551 | .name = "fcoe_fcf", |
| 552 | .groups = fcoe_fcf_attr_groups, | 552 | .groups = fcoe_fcf_attr_groups, |
| 553 | .release = fcoe_fcf_device_release, | 553 | .release = fcoe_fcf_device_release, |
| 554 | }; | 554 | }; |
| 555 | 555 | ||
| 556 | struct bus_attribute fcoe_bus_attr_group[] = { | 556 | static struct bus_attribute fcoe_bus_attr_group[] = { |
| 557 | __ATTR(ctlr_create, S_IWUSR, NULL, fcoe_ctlr_create_store), | 557 | __ATTR(ctlr_create, S_IWUSR, NULL, fcoe_ctlr_create_store), |
| 558 | __ATTR(ctlr_destroy, S_IWUSR, NULL, fcoe_ctlr_destroy_store), | 558 | __ATTR(ctlr_destroy, S_IWUSR, NULL, fcoe_ctlr_destroy_store), |
| 559 | __ATTR_NULL | 559 | __ATTR_NULL |
| 560 | }; | 560 | }; |
| 561 | 561 | ||
| 562 | struct bus_type fcoe_bus_type = { | 562 | static struct bus_type fcoe_bus_type = { |
| 563 | .name = "fcoe", | 563 | .name = "fcoe", |
| 564 | .match = &fcoe_bus_match, | 564 | .match = &fcoe_bus_match, |
| 565 | .bus_attrs = fcoe_bus_attr_group, | 565 | .bus_attrs = fcoe_bus_attr_group, |
| @@ -569,7 +569,7 @@ struct bus_type fcoe_bus_type = { | |||
| 569 | * fcoe_ctlr_device_flush_work() - Flush a FIP ctlr's workqueue | 569 | * fcoe_ctlr_device_flush_work() - Flush a FIP ctlr's workqueue |
| 570 | * @ctlr: Pointer to the FIP ctlr whose workqueue is to be flushed | 570 | * @ctlr: Pointer to the FIP ctlr whose workqueue is to be flushed |
| 571 | */ | 571 | */ |
| 572 | void fcoe_ctlr_device_flush_work(struct fcoe_ctlr_device *ctlr) | 572 | static void fcoe_ctlr_device_flush_work(struct fcoe_ctlr_device *ctlr) |
| 573 | { | 573 | { |
| 574 | if (!fcoe_ctlr_work_q(ctlr)) { | 574 | if (!fcoe_ctlr_work_q(ctlr)) { |
| 575 | printk(KERN_ERR | 575 | printk(KERN_ERR |
| @@ -590,8 +590,8 @@ void fcoe_ctlr_device_flush_work(struct fcoe_ctlr_device *ctlr) | |||
| 590 | * Return value: | 590 | * Return value: |
| 591 | * 1 on success / 0 already queued / < 0 for error | 591 | * 1 on success / 0 already queued / < 0 for error |
| 592 | */ | 592 | */ |
| 593 | int fcoe_ctlr_device_queue_work(struct fcoe_ctlr_device *ctlr, | 593 | static int fcoe_ctlr_device_queue_work(struct fcoe_ctlr_device *ctlr, |
| 594 | struct work_struct *work) | 594 | struct work_struct *work) |
| 595 | { | 595 | { |
| 596 | if (unlikely(!fcoe_ctlr_work_q(ctlr))) { | 596 | if (unlikely(!fcoe_ctlr_work_q(ctlr))) { |
| 597 | printk(KERN_ERR | 597 | printk(KERN_ERR |
| @@ -609,7 +609,7 @@ int fcoe_ctlr_device_queue_work(struct fcoe_ctlr_device *ctlr, | |||
| 609 | * fcoe_ctlr_device_flush_devloss() - Flush a FIP ctlr's devloss workqueue | 609 | * fcoe_ctlr_device_flush_devloss() - Flush a FIP ctlr's devloss workqueue |
| 610 | * @ctlr: Pointer to FIP ctlr whose workqueue is to be flushed | 610 | * @ctlr: Pointer to FIP ctlr whose workqueue is to be flushed |
| 611 | */ | 611 | */ |
| 612 | void fcoe_ctlr_device_flush_devloss(struct fcoe_ctlr_device *ctlr) | 612 | static void fcoe_ctlr_device_flush_devloss(struct fcoe_ctlr_device *ctlr) |
| 613 | { | 613 | { |
| 614 | if (!fcoe_ctlr_devloss_work_q(ctlr)) { | 614 | if (!fcoe_ctlr_devloss_work_q(ctlr)) { |
| 615 | printk(KERN_ERR | 615 | printk(KERN_ERR |
| @@ -631,9 +631,9 @@ void fcoe_ctlr_device_flush_devloss(struct fcoe_ctlr_device *ctlr) | |||
| 631 | * Return value: | 631 | * Return value: |
| 632 | * 1 on success / 0 already queued / < 0 for error | 632 | * 1 on success / 0 already queued / < 0 for error |
| 633 | */ | 633 | */ |
| 634 | int fcoe_ctlr_device_queue_devloss_work(struct fcoe_ctlr_device *ctlr, | 634 | static int fcoe_ctlr_device_queue_devloss_work(struct fcoe_ctlr_device *ctlr, |
| 635 | struct delayed_work *work, | 635 | struct delayed_work *work, |
| 636 | unsigned long delay) | 636 | unsigned long delay) |
| 637 | { | 637 | { |
| 638 | if (unlikely(!fcoe_ctlr_devloss_work_q(ctlr))) { | 638 | if (unlikely(!fcoe_ctlr_devloss_work_q(ctlr))) { |
| 639 | printk(KERN_ERR | 639 | printk(KERN_ERR |
