[PATCH RFT 1/2] drm/bridge: it6505: Use per-client debugfs entry

[Wolfram Sang]

The I2C core now offers a debugfs-directory per client. Use it and
remove the custom handling. This also fixes the problem of a static
debugfs dir name which would break if two of these devices would be on
one system.

Signed-off-by: Wolfram Sang <wsa+rene...@sang-engineering.com>
---
 drivers/gpu/drm/bridge/ite-it6505.c | 28 +++-------------------------
 1 file changed, 3 insertions(+), 25 deletions(-)

diff --git a/drivers/gpu/drm/bridge/ite-it6505.c 
b/drivers/gpu/drm/bridge/ite-it6505.c
index 88ef76a37fe6..f32e3691a9ea 100644
--- a/drivers/gpu/drm/bridge/ite-it6505.c
+++ b/drivers/gpu/drm/bridge/ite-it6505.c
@@ -302,7 +302,6 @@
 #define WORD_LENGTH_18BIT 1
 #define WORD_LENGTH_20BIT 2
 #define WORD_LENGTH_24BIT 3
-#define DEBUGFS_DIR_NAME "it6505-debugfs"
 #define READ_BUFFER_SIZE 400
 
 /* Vendor option */
@@ -478,7 +477,6 @@ struct it6505 {
        struct device *codec_dev;
        struct delayed_work delayed_audio;
        struct it6505_audio_data audio;
-       struct dentry *debugfs;
 
        /* it6505 driver hold option */
        bool enable_drv_hold;
@@ -3574,37 +3572,18 @@ static const struct debugfs_entries debugfs_entry[] = {
        { NULL, NULL },
 };
 
-static void debugfs_create_files(struct it6505 *it6505)
+static void it6505_debugfs_create_files(struct i2c_client *client, struct 
it6505 *it6505)
 {
        int i = 0;
 
        while (debugfs_entry[i].name && debugfs_entry[i].fops) {
                debugfs_create_file(debugfs_entry[i].name, 0644,
-                                   it6505->debugfs, it6505,
+                                   client->debugfs, it6505,
                                    debugfs_entry[i].fops);
                i++;
        }
 }
 
-static void debugfs_init(struct it6505 *it6505)
-{
-       struct device *dev = it6505->dev;
-
-       it6505->debugfs = debugfs_create_dir(DEBUGFS_DIR_NAME, NULL);
-
-       if (IS_ERR(it6505->debugfs)) {
-               dev_err(dev, "failed to create debugfs root");
-               return;
-       }
-
-       debugfs_create_files(it6505);
-}
-
-static void it6505_debugfs_remove(struct it6505 *it6505)
-{
-       debugfs_remove_recursive(it6505->debugfs);
-}
-
 static void it6505_shutdown(struct i2c_client *client)
 {
        struct it6505 *it6505 = dev_get_drvdata(&client->dev);
@@ -3689,7 +3668,7 @@ static int it6505_i2c_probe(struct i2c_client *client)
                it6505_poweron(it6505);
 
        DRM_DEV_DEBUG_DRIVER(dev, "it6505 device name: %s", dev_name(dev));
-       debugfs_init(it6505);
+       it6505_debugfs_create_files(client, it6505);
        pm_runtime_enable(dev);
 
        it6505->aux.name = "DP-AUX";
@@ -3712,7 +3691,6 @@ static void it6505_i2c_remove(struct i2c_client *client)
 
        drm_bridge_remove(&it6505->bridge);
        drm_dp_aux_unregister(&it6505->aux);
-       it6505_debugfs_remove(it6505);
        it6505_poweroff(it6505);
        it6505_remove_edid(it6505);
 }
-- 
2.45.2