[PATCH 3/3] staging: gasket: sysfs: remove legacy_device field

[Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>]

This field is only ever checked, never actually set, and looks to be
left-over from some old interface of some sort.  As it's not being used
at all here, and is just adding to the complexity, delete it.

Cc: Rob Springer <rspringer@xxxxxxxxxx>
Cc: John Joseph <jnjoseph@xxxxxxxxxx>
Cc: Ben Chan <benchan@xxxxxxxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
 drivers/staging/gasket/gasket_sysfs.c | 29 ++-------------------------
 1 file changed, 2 insertions(+), 27 deletions(-)

diff --git a/drivers/staging/gasket/gasket_sysfs.c b/drivers/staging/gasket/gasket_sysfs.c
index 39f9595c85e8..e3d770630961 100644
--- a/drivers/staging/gasket/gasket_sysfs.c
+++ b/drivers/staging/gasket/gasket_sysfs.c
@@ -17,9 +17,6 @@ struct gasket_sysfs_mapping {
 	 */
 	struct device *device;
 
-	/* Legacy device struct, if used by this mapping's driver. */
-	struct device *legacy_device;
-
 	/* The Gasket descriptor for this device. */
 	struct gasket_dev *gasket_dev;
 
@@ -75,8 +72,7 @@ static struct gasket_sysfs_mapping *get_mapping(struct device *device)
 
 	for (i = 0; i < GASKET_SYSFS_NUM_MAPPINGS; i++) {
 		mutex_lock(&dev_mappings[i].mutex);
-		if (dev_mappings[i].device == device ||
-		    dev_mappings[i].legacy_device == device) {
+		if (dev_mappings[i].device == device) {
 			kref_get(&dev_mappings[i].refcount);
 			mutex_unlock(&dev_mappings[i].mutex);
 			return &dev_mappings[i];
@@ -105,7 +101,6 @@ static void put_mapping(struct gasket_sysfs_mapping *mapping)
 	int num_files_to_remove = 0;
 	struct device_attribute *files_to_remove;
 	struct device *device;
-	struct device *legacy_device;
 
 	if (!mapping) {
 		gasket_nodev_info("Mapping should not be NULL.");
@@ -126,7 +121,6 @@ static void put_mapping(struct gasket_sysfs_mapping *mapping)
 		 * sysfs nodes are removed outside the lock.
 		 */
 		device = mapping->device;
-		legacy_device = mapping->legacy_device;
 		num_files_to_remove = mapping->attribute_count;
 		files_to_remove = kcalloc(num_files_to_remove,
 					  sizeof(*files_to_remove),
@@ -143,12 +137,8 @@ static void put_mapping(struct gasket_sysfs_mapping *mapping)
 	mutex_unlock(&mapping->mutex);
 
 	if (num_files_to_remove != 0) {
-		for (i = 0; i < num_files_to_remove; ++i) {
+		for (i = 0; i < num_files_to_remove; ++i)
 			device_remove_file(device, &files_to_remove[i]);
-			if (legacy_device)
-				device_remove_file(
-					legacy_device, &files_to_remove[i]);
-		}
 		kfree(files_to_remove);
 	}
 }
@@ -282,21 +272,6 @@ int gasket_sysfs_create_entries(
 			return ret;
 		}
 
-		if (mapping->legacy_device) {
-			ret = device_create_file(mapping->legacy_device,
-						 &attrs[i].attr);
-			if (ret) {
-				gasket_log_error(
-					mapping->gasket_dev,
-					"Unable to create legacy sysfs entries;"
-					" rc: %d",
-					ret);
-				mutex_unlock(&mapping->mutex);
-				put_mapping(mapping);
-				return ret;
-			}
-		}
-
 		mapping->attributes[mapping->attribute_count] = attrs[i];
 		++mapping->attribute_count;
 	}
-- 
2.18.0

_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel