Commit a8c6da90 authored by Nicholas Bellinger's avatar Nicholas Bellinger

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: default avatarChristophe Fergeau <cfergeau@gmail.com>
Signed-off-by: default avatarNicholas A. Bellinger <nab@linux-iscsi.org>
parent efa4988d
...@@ -533,7 +533,6 @@ static int target_fabric_tf_ops_check( ...@@ -533,7 +533,6 @@ static int target_fabric_tf_ops_check(
int target_fabric_configfs_register( int target_fabric_configfs_register(
struct target_fabric_configfs *tf) struct target_fabric_configfs *tf)
{ {
struct config_group *su_group;
int ret; int ret;
if (!(tf)) { if (!(tf)) {
...@@ -546,12 +545,6 @@ int target_fabric_configfs_register( ...@@ -546,12 +545,6 @@ int target_fabric_configfs_register(
" pointer\n"); " pointer\n");
return -EINVAL; return -EINVAL;
} }
su_group = &tf->tf_subsys->su_group;
if (!(su_group)) {
printk(KERN_ERR "Unable to locate target struct config_group"
" pointer\n");
return -EINVAL;
}
ret = target_fabric_tf_ops_check(tf); ret = target_fabric_tf_ops_check(tf);
if (ret < 0) if (ret < 0)
return ret; return ret;
...@@ -565,7 +558,6 @@ EXPORT_SYMBOL(target_fabric_configfs_register); ...@@ -565,7 +558,6 @@ EXPORT_SYMBOL(target_fabric_configfs_register);
void target_fabric_configfs_deregister( void target_fabric_configfs_deregister(
struct target_fabric_configfs *tf) struct target_fabric_configfs *tf)
{ {
struct config_group *su_group;
struct configfs_subsystem *su; struct configfs_subsystem *su;
if (!(tf)) { if (!(tf)) {
...@@ -579,13 +571,6 @@ void target_fabric_configfs_deregister( ...@@ -579,13 +571,6 @@ void target_fabric_configfs_deregister(
" pointer\n"); " pointer\n");
return; return;
} }
su_group = &tf->tf_subsys->su_group;
if (!(su_group)) {
printk(KERN_ERR "Unable to locate target struct config_group"
" pointer\n");
return;
}
printk(KERN_INFO "<<<<<<<<<<<<<<<<<<<<<< BEGIN FABRIC API >>>>>>>>>>" printk(KERN_INFO "<<<<<<<<<<<<<<<<<<<<<< BEGIN FABRIC API >>>>>>>>>>"
">>>>>>>>>>>>\n"); ">>>>>>>>>>>>\n");
mutex_lock(&g_tf_lock); mutex_lock(&g_tf_lock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment