USB: typec: fusb302: create debugfs subdir for the driver

The single debugfs file for this driver really is a log file, so make a
subdir and call it "log" to make it obvious this is what it is for.
This makes cleanup simpler as we just remove the whole directory, no
need to handle individual files anymore.

Cc: Guenter Roeck <linux@roeck-us.net>
Cc: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Cc: linux-usb@vger.kernel.org
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Reviewed-by: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Link: https://lore.kernel.org/r/20210216144645.3813043-5-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/usb/typec/tcpm/fusb302.c b/drivers/usb/typec/tcpm/fusb302.c
index ebc46b9f776cdb02952e22a4509ff4343255cdbf..7a2a17866a823b36d5e11bce7e31517fd19c1284 100644 (file)
--- a/drivers/usb/typec/tcpm/fusb302.c
+++ b/drivers/usb/typec/tcpm/fusb302.c
@@ -213,8 +213,9 @@ static void fusb302_debugfs_init(struct fusb302_chip *chip)
 
        mutex_init(&chip->logbuffer_lock);
        snprintf(name, NAME_MAX, "fusb302-%s", dev_name(chip->dev));
-       chip->dentry = debugfs_create_file(name, S_IFREG | 0444, usb_debug_root,
-                                          chip, &fusb302_debug_fops);
+       chip->dentry = debugfs_create_dir(name, usb_debug_root);
+       debugfs_create_file("log", S_IFREG | 0444, chip->dentry, chip,
+                           &fusb302_debug_fops);
 }
 
 static void fusb302_debugfs_exit(struct fusb302_chip *chip)