drm/tegra: sor: Root debugfs files at the connector
authorThierry Reding <treding@nvidia.com>
Thu, 12 Oct 2017 17:07:54 +0000 (19:07 +0200)
committerThierry Reding <treding@nvidia.com>
Wed, 13 Dec 2017 12:42:08 +0000 (13:42 +0100)
Rather create new files within the top-level DRM device's debugfs node,
add the SOR specific files to the connector's debugfs node. This avoids
the need to come up with subdirectory names and is also more intuitive.

Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/gpu/drm/tegra/sor.c

index 75b21dbaa8f09e4b9c0b0fe98cf321c7780d583f..f40fc987900c82bbe40ba9b8b628705a3611cb38 100644 (file)
@@ -184,7 +184,6 @@ struct tegra_sor {
 
        struct drm_info_list *debugfs_files;
        struct drm_minor *minor;
-       struct dentry *debugfs;
 
        const struct tegra_sor_ops *ops;
 
@@ -1260,14 +1259,10 @@ static const struct drm_info_list debugfs_files[] = {
 static int tegra_sor_debugfs_init(struct tegra_sor *sor,
                                  struct drm_minor *minor)
 {
-       const char *name = sor->soc->supports_dp ? "sor1" : "sor";
+       struct dentry *root = sor->output.connector.debugfs_entry;
        unsigned int i;
        int err;
 
-       sor->debugfs = debugfs_create_dir(name, minor->debugfs_root);
-       if (!sor->debugfs)
-               return -ENOMEM;
-
        sor->debugfs_files = kmemdup(debugfs_files, sizeof(debugfs_files),
                                     GFP_KERNEL);
        if (!sor->debugfs_files) {
@@ -1280,7 +1275,7 @@ static int tegra_sor_debugfs_init(struct tegra_sor *sor,
 
        err = drm_debugfs_create_files(sor->debugfs_files,
                                       ARRAY_SIZE(debugfs_files),
-                                      sor->debugfs, minor);
+                                      root, minor);
        if (err < 0)
                goto free;
 
@@ -1292,13 +1287,14 @@ free:
        kfree(sor->debugfs_files);
        sor->debugfs_files = NULL;
 remove:
-       debugfs_remove_recursive(sor->debugfs);
-       sor->debugfs = NULL;
+       debugfs_remove_recursive(root);
        return err;
 }
 
 static void tegra_sor_debugfs_exit(struct tegra_sor *sor)
 {
+       struct dentry *root = sor->output.connector.debugfs_entry;
+
        drm_debugfs_remove_files(sor->debugfs_files, ARRAY_SIZE(debugfs_files),
                                 sor->minor);
        sor->minor = NULL;
@@ -1306,8 +1302,7 @@ static void tegra_sor_debugfs_exit(struct tegra_sor *sor)
        kfree(sor->debugfs_files);
        sor->debugfs_files = NULL;
 
-       debugfs_remove_recursive(sor->debugfs);
-       sor->debugfs = NULL;
+       debugfs_remove_recursive(root);
 }
 
 static void tegra_sor_connector_reset(struct drm_connector *connector)