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 |