diff options
author | Nicholas Bellinger <nab@linux-iscsi.org> | 2011-03-06 14:49:36 -0500 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2011-07-22 05:37:43 -0400 |
commit | a8c6da90b823fb94ca76ca0df6bb44e6e205dc87 (patch) | |
tree | 7954ec28c34e5115805338502665263f8f2ed499 /drivers/target | |
parent | efa4988d72c69d3024ee25ad1ae87c83b9f8267e (diff) |
target: Remove unused su_group usage in fabric register/dergister
This patch removes two instances of left over v3.x code performing local
scope access to struct target_core_fabric_ops->tf_subsys->su_group in
target_fabric_configfs_register() and target_fabric_configfs_deregister().
Reported-by: Christophe Fergeau <cfergeau@gmail.com>
Signed-off-by: Nicholas A. Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target')
-rw-r--r-- | drivers/target/target_core_configfs.c | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/drivers/target/target_core_configfs.c b/drivers/target/target_core_configfs.c index 25c1f49a7d8..c6140004307 100644 --- a/drivers/target/target_core_configfs.c +++ b/drivers/target/target_core_configfs.c | |||
@@ -533,7 +533,6 @@ static int target_fabric_tf_ops_check( | |||
533 | int target_fabric_configfs_register( | 533 | int target_fabric_configfs_register( |
534 | struct target_fabric_configfs *tf) | 534 | struct target_fabric_configfs *tf) |
535 | { | 535 | { |
536 | struct config_group *su_group; | ||
537 | int ret; | 536 | int ret; |
538 | 537 | ||
539 | if (!(tf)) { | 538 | if (!(tf)) { |
@@ -546,12 +545,6 @@ int target_fabric_configfs_register( | |||
546 | " pointer\n"); | 545 | " pointer\n"); |
547 | return -EINVAL; | 546 | return -EINVAL; |
548 | } | 547 | } |
549 | su_group = &tf->tf_subsys->su_group; | ||
550 | if (!(su_group)) { | ||
551 | printk(KERN_ERR "Unable to locate target struct config_group" | ||
552 | " pointer\n"); | ||
553 | return -EINVAL; | ||
554 | } | ||
555 | ret = target_fabric_tf_ops_check(tf); | 548 | ret = target_fabric_tf_ops_check(tf); |
556 | if (ret < 0) | 549 | if (ret < 0) |
557 | return ret; | 550 | return ret; |
@@ -565,7 +558,6 @@ EXPORT_SYMBOL(target_fabric_configfs_register); | |||
565 | void target_fabric_configfs_deregister( | 558 | void target_fabric_configfs_deregister( |
566 | struct target_fabric_configfs *tf) | 559 | struct target_fabric_configfs *tf) |
567 | { | 560 | { |
568 | struct config_group *su_group; | ||
569 | struct configfs_subsystem *su; | 561 | struct configfs_subsystem *su; |
570 | 562 | ||
571 | if (!(tf)) { | 563 | if (!(tf)) { |
@@ -579,13 +571,6 @@ void target_fabric_configfs_deregister( | |||
579 | " pointer\n"); | 571 | " pointer\n"); |
580 | return; | 572 | return; |
581 | } | 573 | } |
582 | su_group = &tf->tf_subsys->su_group; | ||
583 | if (!(su_group)) { | ||
584 | printk(KERN_ERR "Unable to locate target struct config_group" | ||
585 | " pointer\n"); | ||
586 | return; | ||
587 | } | ||
588 | |||
589 | printk(KERN_INFO "<<<<<<<<<<<<<<<<<<<<<< BEGIN FABRIC API >>>>>>>>>>" | 574 | printk(KERN_INFO "<<<<<<<<<<<<<<<<<<<<<< BEGIN FABRIC API >>>>>>>>>>" |
590 | ">>>>>>>>>>>>\n"); | 575 | ">>>>>>>>>>>>\n"); |
591 | mutex_lock(&g_tf_lock); | 576 | mutex_lock(&g_tf_lock); |